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

Add notifications feature #1402

Closed
rafalp opened this issue Dec 21, 2022 · 1 comment · Fixed by #1536
Closed

Add notifications feature #1402

rafalp opened this issue Dec 21, 2022 · 1 comment · Fixed by #1536
Assignees
Labels
area: backend This issue involves Python, Django or dependency (eg. database) area: frontend This issue involves JavaScript, React.js and Node new feature New feature
Milestone

Comments

@rafalp
Copy link
Owner

rafalp commented Dec 21, 2022

Misago does plenty of things for notifications feature like subscription levels, following and mentions. We could do in-app notifications for those.

Facebook separates new notifications from unread ones. Digit next to notifications icon is number of new notifications you haven't seen yet. When you open the window, this number is zeroed, but notifications are still there, with unread status. When you click on notification, its marked as read and you are redirected to its target. Look at the console, Facebook fires notifications_update_seen_or_read GraphQL mutation on click.

@rafalp rafalp added this to the Misago 0.28 milestone Dec 21, 2022
@rafalp rafalp self-assigned this Dec 21, 2022
@rafalp rafalp modified the milestones: Misago 0.28, Misago 0.29 Jan 1, 2023
@rafalp rafalp modified the milestones: Misago 0.29, Misago 0.x Jan 10, 2023
@rafalp rafalp modified the milestones: Misago 1.0, Misago 0.32 Mar 27, 2023
@rafalp rafalp changed the title Add alerts feature Add notifications feature Mar 27, 2023
@rafalp rafalp added area: frontend This issue involves JavaScript, React.js and Node area: backend This issue involves Python, Django or dependency (eg. database) new feature New feature labels Mar 27, 2023
@rafalp rafalp modified the milestones: Misago 0.32, Misago 1.0, 0.33 Mar 28, 2023
@rafalp
Copy link
Owner Author

rafalp commented Mar 30, 2023

I can't rely on firing an event on clicking the alert because Misago is not SPA, but I can have all alerts link to view that will set alert as read and redirect user to alert's target.

Misago would use redirecting view anyway, in order to be able to have a url to the post. Except this one will prepend it's URL resolution logic with extra steps to retrieve alert, mark it as read and then redirect user to alert's target.

@rafalp rafalp modified the milestones: 0.33, 0.34 Apr 4, 2023
@rafalp rafalp modified the milestones: 0.34, 0.35 Apr 29, 2023
@rafalp rafalp closed this as completed May 15, 2023
@rafalp rafalp mentioned this issue May 15, 2023
65 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: backend This issue involves Python, Django or dependency (eg. database) area: frontend This issue involves JavaScript, React.js and Node new feature New feature
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant