notif cleanup

[chrisnojima-zoom]

move config delegation out of notif

[Copilot]

Pull request overview

Copilot reviewed 9 out of 9 changed files in this pull request and generated no new comments.

Comments suppressed due to low confidence (1)

shared/constants/init/shared.tsx:555

• The same condition (leaving the People tab) is checked twice in a row for two different side effects. This duplication makes it easier for the behaviors to drift if the condition ever changes. Consider combining these into a single if block that performs both clearSignupEmail() and people.markViewed() under the same guard.

      if (prev && Util.getTab(prev) === Tabs.peopleTab && next && Util.getTab(next) !== Tabs.peopleTab) {
        clearSignupEmail()
      }

      if (prev && Util.getTab(prev) === Tabs.peopleTab && next && Util.getTab(next) !== Tabs.peopleTab) {
        storeRegistry.getState('people').dispatch.markViewed()
      }

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.