-
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
Configure/Disable certain notifications for a user #2696
Comments
@sleidig We can design the UI for Notification Types similar to the layout shown in the below-attached screenshot. On the Profile page, we can add a new tab next to the User Account tab, named Notification Settings. When the user clicks on the Notification Settings tab, a new page will open. This page will display a list of notification types, such as Children, Student, etc., as shown in the screenshot. Under the label Where you receive notifications, we will specify the platforms where notifications are available, such as through the browser and Email (we can implement this functionality in future). Also, If notification permissions are not granted by the user, there should be an option to enable the necessary permissions. This page can also be accessed directly from the Notification List. Please let me know your thoughts and suggestions on this. |
@Ayush8923 |
As per discussion on call, here is the todos for Notification setting UI Updates Descriptions
Dynamic Saving
Editing Notification Rule block
Default Settings
Test Trigger Button
Section Updates
Delete Option
@sleidig @george-neha I have added the todos for notification UI updates, please take a look and let me know if I missed anything. Also, I will update the notification setting logic issue description based on this discussion. |
@sleidig @george-neha I have updated the notification settings UI according to the suggestions mentioned above. Could you please have a look and let me know if any updates or changes are needed? For reference, here is the screenshot.
Also, @george-neha, could you please share the updated descriptions for the fields, so that I can update them here? |
@sleidig @Ayush8923 |
|
To allow users to enable or disable various notification types in the UI:
For details on the required database updates, here is the discussion:
#2690 (comment)
The text was updated successfully, but these errors were encountered: