Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for local overrides for feature flags #1074

Merged
merged 18 commits into from
Nov 14, 2024

Conversation

ayoy
Copy link
Contributor

@ayoy ayoy commented Nov 13, 2024

Please review the release process for BrowserServicesKit here.

Required:

Task/Issue URL: https://app.asana.com/0/72649045549333/1208716221426945/f
iOS PR: duckduckgo/iOS#3571
macOS PR: duckduckgo/macos-browser#3545
What kind of version bump will this require?: Major

Optional:

Tech Design URL: https://app.asana.com/0/481882893211075/1208716218352496/f
CC: @diegoreymendez

Description:
This change adds FeatureFlagLocalOverrides class that is owned by DefaultFeatureFlagger
and allow for setting local overrides for feature flags. This is supported only for internal users
and is opt-in for all feature flags. Currently disabled for all flags but HTML New Tab Page on macOS.

Steps to test this PR:
See macOS PR for steps to test. iOS PR is good to go as soon as CI is green.

OS Testing:

  • iOS 14
  • iOS 15
  • iOS 16
  • macOS 10.15
  • macOS 11
  • macOS 12

Internal references:

Software Engineering Expectations
Technical Design Template

@ayoy ayoy self-assigned this Nov 13, 2024
@ayoy ayoy marked this pull request as ready for review November 13, 2024 15:13
@ayoy ayoy requested a review from SabrinaTardio November 13, 2024 15:13
@ayoy ayoy requested review from diegoreymendez and removed request for SabrinaTardio November 14, 2024 08:55
@ayoy ayoy assigned diegoreymendez and unassigned ayoy Nov 14, 2024
@ayoy ayoy merged commit cfb1780 into main Nov 14, 2024
7 checks passed
@ayoy ayoy deleted the dominik/feature-flag-overrides branch November 14, 2024 11:38
samsymons added a commit that referenced this pull request Nov 14, 2024
* main:
  add text zoom feature flag (#1057)
  Add support for local overrides for feature flags (#1074)
  Speculative password import prompt crash fix (#1072)
  Roll back VPN Routing change (#1076)
samsymons added a commit that referenced this pull request Nov 18, 2024
* main:
  Upgrade to Xcode 16 and macOS 15 (#1075)
  add text zoom feature flag (#1057)
  Add support for local overrides for feature flags (#1074)
  Speculative password import prompt crash fix (#1072)
mgurgel pushed a commit that referenced this pull request Nov 18, 2024
Task/Issue URL: https://app.asana.com/0/72649045549333/1208716221426945/f
Tech Design URL: https://app.asana.com/0/481882893211075/1208716218352496/f

Description:
This change adds FeatureFlagLocalOverrides class that is owned by DefaultFeatureFlagger
and allow for setting local overrides for feature flags. This is supported only for internal users
and is opt-in for all feature flags. Currently disabled for all flags but HTML New Tab Page on macOS.
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