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 Settings: Update feature flag #5480

Conversation

mikescamell
Copy link
Contributor

@mikescamell mikescamell commented Jan 16, 2025

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

Description

Updates the feature toggle to settingsPage and add two new sub-flags for controlling the rollout:

  • newSettingsPage for showing the screen and all changes except the privacy pro section
  • newPrivacyProSection for showing/hiding the new privacy pro section items

Steps to test this PR

New Settings

  • Enable newSettingsPage in the Feature Flag Inventory
  • Press back
  • Open Settings again
  • The new settings screen should be displayed
  • Privacy Pro should remain unchanged

New Privacy Pro Section

  • Enable newPrivacyProSection in the Feature Flag Inventory
  • Press back
  • Open Settings again
  • The new settings screen should be displayed
  • The new Privacy Pro section should be displayed

New Privacy Pro Section and legacy settings

  • Disable newSettingsPage in the Feature Flag Inventory
  • Press back
  • Open Settings again
  • The legacy settings screen should be displayed
  • The new Privacy Pro section should be displayed

UI changes

N/A

Copy link
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

We need this to be able to do the rollout, it cannot be done at top level

the top level feature name is now "settingsScreen"

I will rename NewSettingsFeature to SettingsScreenFeature next
@mikescamell mikescamell force-pushed the feature/mike/update-settings/convert-to-remote-feature-toggle branch from 935b191 to e246e53 Compare January 17, 2025 12:23
Copy link
Contributor

@malmstein malmstein left a comment

Choose a reason for hiding this comment

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

Good idea changing the flags!

@mikescamell mikescamell merged commit f95f022 into develop Jan 17, 2025
6 checks passed
@mikescamell mikescamell deleted the feature/mike/update-settings/convert-to-remote-feature-toggle branch January 17, 2025 15:22
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