-
Notifications
You must be signed in to change notification settings - Fork 21
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
Store and access notifications & mark them "read" #2690
Comments
According to me, Notifications can be considered a new entity type within the system. Therefore, it makes sense to create a dedicated entity in CouchDB to handle them. For reference, I’ve included an example below of how we could structure a notification document in CouchDB:
@sleidig @tomwwinter Please let me know what you think/suggest? |
As far as I understand, event records are user-specific. Each user will have the ability to select the types of notifications they wish to receive. A new sidebar tab will be introduced, allowing users to define their preferences for different event types. From this tab, users can opt-in to notifications for specific event types, ensuring they are only notified about the events they have selected. In the CouchDB user document, we can to add the following fields to manage this customization:
@sleidig @tomwwinter Please let me know what you think/suggest? |
@sleidig @tomwwinter, Here's an updated schema and approach for updating an existing CouchDB user document to include the new keys
|
@sleidig We need to update the user document to include |
I think this can become a part of the NotificationConfig entity that you outlined here: #2723 ? |
Yes this is part of the |
For the "notification centre" UI, it is necessary to store notification events in the same way as the user-facing functionalities:
DISCUSS:
We need to implement the below-mentioned functionality in the notification list.
All
tab.Unread
tab.readStatus
should be updated to true for that user.Delete Notification
, the notification should be removed and no longer displayed in the notification list.Enable notifications
should activate the notification for the user.Notification Settings
should redirect the user to the Notifications tab within the Profile section. This is handled in this PR.The text was updated successfully, but these errors were encountered: