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

Existing experiment disabled, the new Settings experiment activated #2801

Merged
merged 3 commits into from May 1, 2024

Conversation

tomasstrba
Copy link
Contributor

@tomasstrba tomasstrba commented Apr 29, 2024

Task/Issue URL: https://app.asana.com/0/0/1206933032583646/f
CC:

Description:
New Settings experiment

Steps to test this PR:

  1. On the first run, make sure the PixelExperimentLogic allocates user to a cohort based on the random number
  2. Verify the cohort is saved and loaded on the next run.
  3. Manually set the cohort to newSettings and make sure the new Settings view is presented to user
  4. Smoke test the new Settings and verify pixels have the cohort as a parameter

Device Testing:

  • iPhone SE (1st Gen)

OS Testing:

  • iOS 16

Internal references:

Software Engineering Expectations
Technical Design Template

…atest implementation of subscription adjusted based on the old Settings view.
@tomasstrba tomasstrba requested a review from brindy April 29, 2024 06:54
@github-actions github-actions bot added the bot: not in app board Added by automation for pull requests with tasks not added to iOS App Board Asana project label Apr 29, 2024
@tomasstrba
Copy link
Contributor Author

@afterxleep, could you please smoke test the Privacy Pro onboarding flow? I have noticed there was a change since I merged the new Settings. I matched the code with the old Settings in this PR.

Copy link
Contributor

@brindy brindy left a comment

Choose a reason for hiding this comment

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

LGTM

@brindy brindy removed the bot: not in app board Added by automation for pull requests with tasks not added to iOS App Board Asana project label Apr 29, 2024
@afterxleep
Copy link
Collaborator

looks on @tomasstrba.

@tomasstrba tomasstrba merged commit 985efac into main May 1, 2024
14 checks passed
@tomasstrba tomasstrba deleted the tom/new-settings-experiment branch May 1, 2024 07:36
samsymons added a commit that referenced this pull request May 1, 2024
* main:
  Release 7.118.0-1 (#2812)
  [Release PR] Update VPN metadata reporter (#2808)
  fix address bar weirdness (#2810)
  Fix RMF button styling for "big_two_action" format (#2811)
  Existing experiment disabled, the new Settings experiment activated (#2801)
  Create Asana Subtask on PR requested (#2803)
  Remove ATB from default params (#2430)
  Fix Kingfisher deprecation warnings (#2799)
  VPN server failure detection recovery (#2779)
  Disable the feedback send button when there’s no text (#2800)
samsymons added a commit that referenced this pull request May 1, 2024
# By Sam Symons (3) and others
# Via Chris Brind (1) and GitHub (1)
* main:
  Release 7.118.0-1 (#2812)
  [Release PR] Update VPN metadata reporter (#2808)
  fix address bar weirdness (#2810)
  Fix RMF button styling for "big_two_action" format (#2811)
  Existing experiment disabled, the new Settings experiment activated (#2801)
  Create Asana Subtask on PR requested (#2803)
  Remove ATB from default params (#2430)
  Fix Kingfisher deprecation warnings (#2799)
  VPN server failure detection recovery (#2779)

# Conflicts:
#	DuckDuckGo.xcodeproj/project.pbxproj
#	DuckDuckGo.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved
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.

None yet

3 participants