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

Likes/Favs not properly streamed #18907

Open
lfuelling opened this issue Jul 30, 2022 · 4 comments
Open

Likes/Favs not properly streamed #18907

lfuelling opened this issue Jul 30, 2022 · 4 comments
Labels
bug Something isn't working streaming Streaming server

Comments

@lfuelling
Copy link
Contributor

Steps to reproduce the problem

How To Reproduce

  1. Open Mastodon in a Browser
  2. Get a reply to a post (or mentioned)
    • This will cause the notification to be streamed into the notifications
  3. Open Mastodon in an App and like the status
  4. Go back to the browser
    • The status in the notifications is not liked/faved until the page is refreshed

Expected behaviour

The like should be synced using the stream as well.

Actual behaviour

The like was not synced via streaming, I had to refresh the page.

Specifications

Mastodon: v3.5.3
Chrome: 103.0.5060.114
macOS: 12.4 (21F79)

@lfuelling lfuelling added the bug Something isn't working label Jul 30, 2022
@ThisIsMissEm
Copy link
Contributor

This should be working, as Mastodon's Web UI connects to the user timeline in streaming. Could you re-test on a newer version of Mastodon? e.g., v4.1.x or even v4.2.0-rc1 ?

@ThisIsMissEm
Copy link
Contributor

Actually, I think I've worked out what's happening: we're only sending the notification to the author of the status, not to the person performing the action. For reblogs you get this sync because it does a FeedManager#push_to_home which publishes the update via PushUpdateWorker; For favorites and others we'd need to do something similar where we push to the person doing the action.

@lfuelling
Copy link
Contributor Author

This should be working, as Mastodon's Web UI connects to the user timeline in streaming. Could you re-test on a newer version of Mastodon? e.g., v4.1.x or even v4.2.0-rc1 ?

I can't. I'm not using Mastodon anymore, sorry.

@ThisIsMissEm
Copy link
Contributor

@ClearlyClaire is this something we should fix? Feels like a maybe?

@renchap renchap added the streaming Streaming server label Oct 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working streaming Streaming server
Projects
None yet
Development

No branches or pull requests

3 participants