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

Muting someone may remove existing notifications from them #11087

Open
ClearlyClaire opened this issue Jun 15, 2019 · 3 comments
Open

Muting someone may remove existing notifications from them #11087

ClearlyClaire opened this issue Jun 15, 2019 · 3 comments
Labels
bug Something isn't working

Comments

@ClearlyClaire
Copy link
Contributor

Expected behaviour

When muting someone without disabling notifications from them, existing notifications should not disappear.

Actual behaviour

Existing notifications from muted people disappear from the WebUI if the corresponding toots were also in the Home timeline.

This is because FeedManager#clear_from_timelines (called by MuteWorker) sends a delete to the streaming server, which can't be distinguished from a proper status deletion by the client.

Possible solutions may be:

  • do not send a delete in such cases, as the client already does filtering when muting (but other connected clients wouldn't apply the mute retroactively)
  • have a new version of the delete event (api/v2/streaming?) having a field allowing to differentiate between a toot that is removed from the timeline and a toot that is completely deleted
@Gargron Gargron added the bug Something isn't working label Aug 5, 2019
@stale
Copy link

stale bot commented Oct 26, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the status/wontfix This will not be worked on label Oct 26, 2019
@trwnh
Copy link
Member

trwnh commented Oct 26, 2019

still an issue?

@stale stale bot removed the status/wontfix This will not be worked on label Oct 26, 2019
@ClearlyClaire
Copy link
Contributor Author

yes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants