Skip to content

fix: notification read and snooze bugs - #2639

Merged
sriramveeraghanta merged 3 commits into
developfrom
refactor/notifications
Nov 3, 2023
Merged

fix: notification read and snooze bugs#2639
sriramveeraghanta merged 3 commits into
developfrom
refactor/notifications

Conversation

@dakshesh14

Copy link
Copy Markdown
Contributor

fix:

  • marking notification as read doesn't remove it from the un-read notification list
  • past snooze notifications coming in the snooze tab

@vercel

vercel Bot commented Nov 3, 2023

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

6 Ignored Deployments
Name Status Preview Updated (UTC)
plane ⬜️ Ignored (Inspect) Nov 3, 2023 11:57am
plane-dev ⬜️ Ignored (Inspect) Nov 3, 2023 11:57am
plane-sh ⬜️ Ignored (Inspect) Nov 3, 2023 11:57am
plane-sh-dev ⬜️ Ignored (Inspect) Nov 3, 2023 11:57am
plane-sh-stage ⬜️ Ignored (Inspect) Nov 3, 2023 11:57am
plane-staging ⬜️ Ignored (Inspect) Nov 3, 2023 11:57am

@sriramveeraghanta
sriramveeraghanta merged commit 91878fb into develop Nov 3, 2023
@sriramveeraghanta
sriramveeraghanta deleted the refactor/notifications branch November 3, 2023 13:51
aaryan610 pushed a commit that referenced this pull request Apr 9, 2025
Co-authored-by: Anmol Singh Bhatia <121005188+anmolsinghbhatia@users.noreply.github.com>
DoctorFogarty pushed a commit to DoctorFogarty/plane that referenced this pull request Aug 7, 2026
* fix: marking notification as read doesn't remove it from un-read list

* refactor: arranged imports

* fix: past snooze notifications coming in snooze tab
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants