Skip to content

Notify#277

Merged
amina69 merged 11 commits intoamina69:mainfrom
abimbolaalabi:notify
Mar 30, 2026
Merged

Notify#277
amina69 merged 11 commits intoamina69:mainfrom
abimbolaalabi:notify

Conversation

@abimbolaalabi
Copy link
Copy Markdown
Contributor

@abimbolaalabi abimbolaalabi commented Mar 29, 2026

Closes #232


@abimbolaalabi
Copy link
Copy Markdown
Contributor Author

  • PATCH /notifications/preferences on each toggle with 500ms debounce
  • Add optimistic toggle update with rollback on error
  • Implement "Reset to defaults" button with confirmation modal
  • Reset sends all-enabled payload and PATCH request
  • Show inline "Saved" confirmation that auto-dismisses after 2 seconds
  • Add unit tests for debounce, optimistic rollback, and reset modal flow
notfypref

@drips-wave
Copy link
Copy Markdown

drips-wave bot commented Mar 29, 2026

@abimbolaalabi Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@amina69
Copy link
Copy Markdown
Owner

amina69 commented Mar 29, 2026

@abimbolaalabi check, build fail

@abimbolaalabi
Copy link
Copy Markdown
Contributor Author

@amina69 This has been updated.

@abimbolaalabi
Copy link
Copy Markdown
Contributor Author

build

@abimbolaalabi
Copy link
Copy Markdown
Contributor Author

@amina69 please can you kindly merge this.

@abimbolaalabi
Copy link
Copy Markdown
Contributor Author

@amina69 please can you kindly merge this.

@amina69
Copy link
Copy Markdown
Owner

amina69 commented Mar 30, 2026

@abimbolaalabi
Screenshot from 2026-03-30 09-56-58

@abimbolaalabi
Copy link
Copy Markdown
Contributor Author

@amina69 please check!!

@abimbolaalabi
Copy link
Copy Markdown
Contributor Author

@amina69 please check again Maintainer!

@amina69 amina69 merged commit 779febb into amina69:main Mar 30, 2026
1 check passed
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.

[Frontend · Notification UI] Create NotificationPreferencesPage — save and reset

2 participants