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.

Sign up for GitHub

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

Jump to bottom

Suppress notification count for forum users in /ping #12399

Closed
MrPetovan opened this issue Dec 13, 2022 · 0 comments · Fixed by #12427
Closed

Suppress notification count for forum users in /ping #12399

MrPetovan opened this issue Dec 13, 2022 · 0 comments · Fixed by #12427
Assignees
@MrPetovan
Labels
Enhancement Junior Jobs These are issues we think are good for starting to work with the Friendica code base UX
Milestone
2022.12

Comments

@MrPetovan
Copy link
Collaborator

MrPetovan commented Dec 13, 2022 •
edited

See https://poliverso.org/display/0477a01e-1263-9765-a03e-c56668278916

Forum users have the bell icon removed, but the /ping endpoint keep returning a notification count which appears in the title bar. Without the notification menu, there's no way of marking them as read, and the /notifications page doesn't show them either.
@MrPetovan MrPetovan added UX Enhancement Junior Jobs These are issues we think are good for starting to work with the Friendica code base labels Dec 13, 2022
@MrPetovan MrPetovan self-assigned this Dec 15, 2022
@MrPetovan MrPetovan added this to the 2022.12 milestone Dec 15, 2022
@MrPetovan MrPetovan mentioned this issue Dec 15, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@MrPetovan MrPetovan

Labels
Enhancement Junior Jobs These are issues we think are good for starting to work with the Friendica code base UX
Projects
None yet
Milestone
2022.12
Development

Successfully merging a pull request may close this issue.

Suppress notifications for forum users in Module\Notifications\Ping MrPetovan/friendica
1 participant
@MrPetovan