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.

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

fix: unsubscribe after notification requested #3109

Merged
merged 3 commits into from
Jul 22, 2024
Merged

Conversation

raviteja83
Copy link
Collaborator

Details(context, link the issue, how was the bug fixed, what does the new feature do)

Implementation note, gotchas, related work and Future TODOs (optional)

Copy link

vercel bot commented Jul 22, 2024

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

Name Status Preview Comments Updated (UTC)
dashboard-app ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 22, 2024 7:45am
storybook-app ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 22, 2024 7:45am

@raviteja83 raviteja83 changed the title fix: remove use of subscribe fix: use getState instead of subscribe Jul 22, 2024
@raviteja83 raviteja83 changed the title fix: use getState instead of subscribe fix: use getState instead of subscribe in request notification Jul 22, 2024
@raviteja83 raviteja83 changed the title fix: use getState instead of subscribe in request notification fix: unsubscribe after notification requested Jul 22, 2024
@raviteja83 raviteja83 merged commit 5322549 into dev Jul 22, 2024
4 checks passed
@raviteja83 raviteja83 deleted the fix/WEB-2940-notif branch July 22, 2024 07:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants