-
Notifications
You must be signed in to change notification settings - Fork 149
[BUG] Settings purge should only remove SecuScan localStorage keys #464
Copy link
Copy link
Open
Labels
area:frontendFrontend React/UI workFrontend React/UI workbeginner friendlySmall scoped task suitable for beginner contributorsSmall scoped task suitable for beginner contributorsgood first issueFriendly starter issue for first-time contributorsFriendly starter issue for first-time contributorslevel:beginner20 pts difficulty label for small beginner-friendly PRs20 pts difficulty label for small beginner-friendly PRspriority:lowNice-to-have issue with low urgencyNice-to-have issue with low urgencytype:bugBug fix work category bonus labelBug fix work category bonus label
Metadata
Metadata
Assignees
Labels
area:frontendFrontend React/UI workFrontend React/UI workbeginner friendlySmall scoped task suitable for beginner contributorsSmall scoped task suitable for beginner contributorsgood first issueFriendly starter issue for first-time contributorsFriendly starter issue for first-time contributorslevel:beginner20 pts difficulty label for small beginner-friendly PRs20 pts difficulty label for small beginner-friendly PRspriority:lowNice-to-have issue with low urgencyNice-to-have issue with low urgencytype:bugBug fix work category bonus labelBug fix work category bonus label
Problem
Settings.handleNuclearPurge()currently callslocalStorage.clear(). That removes every key for the origin, including unrelated app state that SecuScan does not own.Expected behavior
The purge action should remove only SecuScan-owned keys, for example keys prefixed with
secuscanand known app keys such assidebar-expanded.Acceptance criteria
localStorage.clear()with targeted removal of SecuScan-owned keys.