Skip to content

Conversation

@alessandroboron
Copy link
Contributor

@alessandroboron alessandroboron commented Nov 4, 2025

Asana Task/Github Issue: https://app.asana.com/1/137249556945/project/72649045549333/task/1211817439010293

Description

Adds a remote settings to define the cooldown interval between prompts in iOS.

Screenshot with test value:

Screenshot 2025-11-04 at 5 17 57 pm

Feature change process:

  • I have added a schema to validate this feature change.
  • I have tested this change locally in all supported browsers.
  • This code for the config change is ready to merge.
  • This feature was covered by a tech design.

Site breakage mitigation process:

Brief explanation

  • Reported URL:
  • Problems experienced:
  • Platforms affected:
    • iOS
    • Android
    • Windows
    • MacOS
    • Extensions
  • Tracker(s) being unblocked:
  • Feature being disabled/modified:
  • This change is a speculative mitigation to fix reported breakage.

github-actions bot pushed a commit that referenced this pull request Nov 4, 2025
@github-actions
Copy link

github-actions bot commented Nov 4, 2025

Generated file outputs:

Time updated: Tue, 04 Nov 2025 09:32:42 GMT

legacy
27 files identical
  • trackers-unprotected-temporary.txt
  • v3/android-config.json
  • v3/extension-brave-config.json
  • v3/extension-bravemv3-config.json
  • v3/extension-chrome-config.json
  • v3/extension-chromemv3-config.json
  • v3/extension-config.json
  • v3/extension-edg-config.json
  • v3/extension-edge-config.json
  • v3/extension-edgmv3-config.json
  • v3/extension-firefox-config.json
  • v3/extension-safarimv3-config.json
  • v3/macos-config.json
  • v3/windows-config.json
  • v4/android-config.json
  • v4/extension-brave-config.json
  • v4/extension-bravemv3-config.json
  • v4/extension-chrome-config.json
  • v4/extension-chromemv3-config.json
  • v4/extension-config.json
  • v4/extension-edg-config.json
  • v4/extension-edge-config.json
  • v4/extension-edgmv3-config.json
  • v4/extension-firefox-config.json
  • v4/extension-safarimv3-config.json
  • v4/macos-config.json
  • v4/windows-config.json

⚠️ File is identical

2 files changed
  • v3/ios-config.json
  • v4/ios-config.json
--- v4/ios-config.json (and 1 other files)
+++ v4/ios-config.json
@@ -95266,8 +95266,11 @@
         },
         "iOSBrowserConfig": {
             "exceptions": [],
             "state": "enabled",
+            "settings": {
+                "promptCooldownInterval": 24
+            },
             "features": {
                 "widgetReporting": {
                     "state": "disabled"
                 },
latest
13 files identical
  • v5/android-config.json
  • v5/extension-brave-config.json
  • v5/extension-bravemv3-config.json
  • v5/extension-chrome-config.json
  • v5/extension-chromemv3-config.json
  • v5/extension-config.json
  • v5/extension-edg-config.json
  • v5/extension-edge-config.json
  • v5/extension-edgmv3-config.json
  • v5/extension-firefox-config.json
  • v5/extension-safarimv3-config.json
  • v5/macos-config.json
  • v5/windows-config.json

⚠️ File is identical

v5/ios-config.json
--- v5/ios-config.json
+++ v5/ios-config.json
@@ -95266,8 +95266,11 @@
         },
         "iOSBrowserConfig": {
             "exceptions": [],
             "state": "enabled",
+            "settings": {
+                "promptCooldownInterval": 24
+            },
             "features": {
                 "widgetReporting": {
                     "state": "disabled"
                 },

@github-actions
Copy link

github-actions bot commented Nov 4, 2025

JSON approval analysis:

Time updated: Tue, 04 Nov 2025 09:32:51 GMT

legacy

❌ Manual Review Required

  • v3/ios-config.json (1 total changes)
    Disallowed paths that require review:
    • /features/iOSBrowserConfig/settings (add)
  • v4/ios-config.json (1 total changes)
    Disallowed paths that require review:
    • /features/iOSBrowserConfig/settings (add)
latest

❌ Manual Review Required

  • v5/ios-config.json (1 total changes)
    Disallowed paths that require review:
    • /features/iOSBrowserConfig/settings (add)

🎯 OVERALL APPROVAL STATUS

❌ MANUAL REVIEW REQUIRED

Copy link
Contributor

@rachelmcr rachelmcr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Just one suggestion, not blocking: What do you think of naming the setting so it’s clear that the interval is in hours? (e.g. promptCooldownIntervalHours)

@alessandroboron alessandroboron added this pull request to the merge queue Nov 5, 2025
Merged via the queue into main with commit 3c3519c Nov 5, 2025
16 of 26 checks passed
@alessandroboron alessandroboron deleted the alessandro/prompt-coordination-cooldown-interval branch November 5, 2025 07:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants