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

Update BSK version to support userTips NetP Flag #3416

Merged
merged 3 commits into from
Oct 8, 2024
Merged

Update BSK version to support userTips NetP Flag #3416

merged 3 commits into from
Oct 8, 2024

Conversation

Bunn
Copy link
Contributor

@Bunn Bunn commented Oct 7, 2024

Task/Issue URL: https://app.asana.com/0/72649045549333/1208381891916914/f
Tech Design URL:
CC:

Description:
Update BSK version to support userTips NetP Flag

Steps to test this PR:

  1. You can add print("USER ON? \(AppDependencyProvider.shared.featureFlagger.isFeatureOn(.networkProtectionUserTips))") to the App Delegate and check if the flag is returning as true (it should)
  2. You can also change the config URL to this: https://jsonblob.com/api/1292861631544287232 which has the flag set to false and make sure that now the isFeatureOn function returns false

Bunn added a commit to duckduckgo/BrowserServicesKit that referenced this pull request Oct 8, 2024
Task/Issue URL:
https://app.asana.com/0/72649045549333/1208381891916914/f
iOS PR:  duckduckgo/iOS#3416
macOS PR:  duckduckgo/macos-browser#3365
What kind of version bump will this require?: Minor

**Description**:
Add userTips feature flag
@Bunn Bunn merged commit 44d78d6 into main Oct 8, 2024
13 checks passed
@Bunn Bunn deleted the bunn/netp/ff branch October 8, 2024 14:19
samsymons added a commit that referenced this pull request Oct 9, 2024
* main:
  Update BSK version to support userTips NetP Flag (#3416)
  Release 7.140.0-0 (#3418)
  Adds new debug screen to view raw credentials data (#3400)
  Bump rollup from 2.79.1 to 3.29.5 (#3385)
  fix UI tests broken by tabs suggestions (#3402)
  Fix ability to edit Synced device name (#3404)
  Fix failing password-management maestro test (#3413)
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