refactor: remove unused FeatureAnnouncementToggle [GE-244]#30441
Conversation
|
CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes. |
37b5a3a to
273cd0e
Compare
🔍 Smart E2E Test Selection
click to see 🤖 AI reasoning detailsE2E Test Selection: Performance Test Selection: |
|



Description
Removes the Feature Announcements notification setting by deleting FeatureAnnouncementToggle (and its test) and dropping the corresponding useFeatureAnnouncementToggle hook.
Replaced by AUS notification preference in #30106
Changelog
CHANGELOG entry:
Related issues
Fixes: https://consensyssoftware.atlassian.net/browse/GE-244
Manual testing steps
Screenshots/Recordings
Before
After
Pre-merge author checklist
Performance checks (if applicable)
trace()for usage andaddTokenfor an exampleFor performance guidelines and tooling, see the Performance Guide.
Pre-merge reviewer checklist
Note
Low Risk
Low risk removal of unused UI toggle and associated hook/tests; primary risk is any missed call sites leading to missing functionality, but searches indicate no remaining references.
Overview
Removes the Feature Announcements notification setting by deleting
FeatureAnnouncementToggle(and its test) and dropping the correspondinguseFeatureAnnouncementTogglehook.Cleans up
useSwitchNotificationsand its tests by removing the feature-announcement toggle flow (action/selector usage and list-refresh behavior) so only the remaining notification/account toggle hooks are exercised.Reviewed by Cursor Bugbot for commit 1b5ee03. Bugbot is set up for automated code reviews on this repo. Configure here.