-
Notifications
You must be signed in to change notification settings - Fork 35
[iOS][Feature Flags] Remove Onboarding Highlights flag #2678
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
[iOS][Feature Flags] Remove Onboarding Highlights flag #2678
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR removes the deprecated onboardingHighlights feature flag that was released to all users and is no longer needed in the codebase.
- Removed the
onboardingHighlightsenum case fromFeatureFlag - Removed the associated debug key
debugOnboardingHighlightsEnabledKeyfromUserDefaultsPropertyWrapper - Cleaned up all references including the flag's source implementation and its entry in the local override list
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| iOS/Core/FeatureFlag.swift | Removed the onboardingHighlights feature flag enum case, its source implementation, and its entry in the local override list |
| iOS/Core/UserDefaultsPropertyWrapper.swift | Removed the debug key debugOnboardingHighlightsEnabledKey that was associated with the feature flag |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
SabrinaTardio
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
dd6d3a5 to
6ef2fa4
Compare
Task/Issue URL: https://app.asana.com/1/137249556945/project/1211834678943996/task/1211866468468677?focus=true Tech Design URL: CC: ### Description Removed the unused feature flag for onboarding highlights from the codebase. The onboarding highlights were released to all users a long time ago, so the feature flag is no longer needed. ### Testing Steps 1. The feature flag was not used anywhere in the codebase so ensure only CI is green. <!— ### Testability Challenges If you encountered issues writing tests due to any class in the codebase, please report it in the [Testability Challenges Document](https://app.asana.com/1/137249556945/project/1204186595873227/task/1211703869786699) 1. **Check the Document:** First, check the **Testability Challenges Table** to see if the class you encountered is listed. 3. **If the Class Exists:** - Update the **Encounter Count** by increasing it by 1. 4. **If the Class Does Not Exist:** - Add a new entry —> ### Impact and Risks <!— What's the impact on users if something goes wrong? High: Could affect user privacy, lose user data, break core functionality Medium: Could disrupt specific features or user flows Low: Minor visual changes, small bug fixes, improvement to existing features None: Internal tooling, documentation —> #### What could go wrong? <!-- Describe specific scenarios and how you've addressed them —> ### Quality Considerations <!— Focus on what matters for your changes: - What edge cases exist? - How does this affect performance? - What monitoring have you added? - What documentation needs updating? - How does this impact privacy/security? —> ### Notes to Reviewer <!-- Anything specific you want reviewers to focus on —> — ###### Internal references: [Definition of Done](https://app.asana.com/0/1202500774821704/1207634633537039/f) | [Engineering Expectations](https://app.asana.com/0/59792373528535/199064865822552) | [Tech Design Template](https://app.asana.com/0/59792373528535/184709971311943) <!-- CURSOR_SUMMARY --> --- > [!NOTE] > Removes the unused onboardingHighlights feature flag and its debug UserDefaults key. > > - **Core** > - **Feature Flags**: Remove `onboardingHighlights` from `FeatureFlag`, including its entries in `supportsLocalOverriding` and `source`. > - **Storage** > - **UserDefaults**: Remove `debugOnboardingHighlightsEnabledKey` from `UserDefaultsWrapper.Key`. > > <sup>Written by [Cursor Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit 6ef2fa4. This will update automatically on new commits. Configure [here](https://cursor.com/dashboard?tab=bugbot).</sup> <!-- /CURSOR_SUMMARY -->
Task/Issue URL: https://app.asana.com/1/137249556945/project/1211834678943996/task/1211866468468677?focus=true
Tech Design URL:
CC:
Description
Removed the unused feature flag for onboarding highlights from the codebase. The onboarding highlights were released to all users a long time ago, so the feature flag is no longer needed.
Testing Steps
<!—
Testability Challenges
If you encountered issues writing tests due to any class in the codebase, please report it in the Testability Challenges Document
—>
Impact and Risks
<!—
What's the impact on users if something goes wrong?
High: Could affect user privacy, lose user data, break core functionality
Medium: Could disrupt specific features or user flows
Low: Minor visual changes, small bug fixes, improvement to existing features
None: Internal tooling, documentation
—>
What could go wrong?
Note
Removes the unused onboardingHighlights feature flag and its debug UserDefaults key.
onboardingHighlightsfromFeatureFlag, including its entries insupportsLocalOverridingandsource.debugOnboardingHighlightsEnabledKeyfromUserDefaultsWrapper.Key.Written by Cursor Bugbot for commit 6ef2fa4. This will update automatically on new commits. Configure here.