feat: implement in-app notification system with bell icon#218
Conversation
…n system endpoints and components
|
@Krish-Makadiya is attempting to deploy a commit to the Karan Mani Tripathi 's projects Team on Vercel. A member of the Team first needs to authorize it. |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
knoxiboy
added
type:feature
|
Done broo!! I think this issue should was a bit advanced, as it had lot of inbuild functionalities, which had to be refactored and been monitored and tested.. just my opinion. |
knoxiboy
added
gssoc:approved
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
There was a problem hiding this comment.
Hi @Krish-Makadiya! This PR currently has merge conflicts with the main branch. Please pull the latest changes, resolve the conflicts locally, and push again so we can merge it. Thank you!
knoxiboy
added
level:advanced
knoxiboy
left a comment
knoxiboy
left a comment
There was a problem hiding this comment.
Automated review: Thank you for starring the repository! The PR is approved.
2 participants
Description
This PR introduces a complete in-app notification system to DoubtDesk to improve user engagement. Users will now receive real-time alerts when someone replies to their doubts without having to refresh or manually check for updates.
Closes #190
Key Changes
notificationsTableto the schema and configured database migrations.GETandPATCHAPI routes at/api/notificationsfor fetching and managing unread states.NotificationBellUI component using Radix UI and SWR for polling.DashboardLayout./api/repliesroute to automatically insert a notification for the doubt author when a reply is posted.