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.

Sign up for GitHub

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

Jump to bottom

refactor(notifications): Centralize notification state update logic #606

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

refactor(notifications): Centralize notification state update logic #606

wants to merge 2 commits into from

Commits on Sep 24, 2024

  1. Update notifications.ts 

    The main improvement is the removal of repeated logic for filtering and updating notifications. This is achieved by centralizing it in the 'getUpdatedTopicState' function, making the reducer more readable and reducing potential errors from code duplication.
    @cypherpepe
    cypherpepe authored Sep 24, 2024
    Configuration menu
    Copy the full SHA
    67d6a89 View commit details
    Browse the repository at this point in the history

Commits on Oct 18, 2024

  1. Added comments 

    Added comments to notificationsReducer for clarity
    @cypherpepe
    cypherpepe authored Oct 18, 2024
    Configuration menu
    Copy the full SHA
    543e0be View commit details
    Browse the repository at this point in the history