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

Sent messages not shown after narrowing from a notification #2539

Closed
borisyankov opened this issue May 20, 2018 · 7 comments
Closed

Sent messages not shown after narrowing from a notification #2539

borisyankov opened this issue May 20, 2018 · 7 comments
Assignees
Labels
a-data-sync Zulip's event system, event queues, staleness/liveness bug zz-in progress

Comments

@borisyankov
Copy link
Contributor

@rishig and @timabbott reported an issue where after narrowing to a PM conversation from a notification, sent messages were not displayed.

I was not able to reproduce the issue trying it in several different configurations and states, but it remains high-priority.

@zulipbot
Copy link
Member

Hello @zulip/server-notifications members, this issue was labeled with the "area: notifications" label, so you may want to check it out!

@gnprice
Copy link
Member

gnprice commented May 24, 2018

FWIW I also saw this several times a couple of weeks ago. I saw it on 13.2.86, and not sure if I did on 13.3.87.

Rishi, Tim, and I all use Android phones, so it's possible this only repros on Android.

@borisyankov
Copy link
Contributor Author

There was a significant bug fix related to this 4-5 weeks ago, but this is on a newer version.
I have a good guess for a specific behavior that I do not consider a bug as we intended it to work that way, but might result in this.
Surprisingly, testing on several different configurations and states I was not able to reproduce.

The reason a user might not be seeing message updates is that he is not 'caught up' to the newest messages. This can happen if:

  • he is missing the latest messages
  • we do not know if he is missing the latest messages (app was in the background, polling was interrupted)

Other behavior that might feel buggy (but is intended and we might want to change):

  • on narrow from notification we show the latest unread message
  • several users reported as bug and expected to be shown the message they tapped on (which in case is not the latest unread is not shown)

@gnprice
Copy link
Member

gnprice commented May 24, 2018 via email

@gnprice
Copy link
Member

gnprice commented Jul 20, 2018

Given our diagnosis of #2800 , it strikes me that this might be a symptom of the same underlying bug: we lose our event queue and don't realize it, so although our initial fetch for the narrow works fine, we never see new messages come in thereafter.

@gnprice
Copy link
Member

gnprice commented Jul 30, 2018

I'd be very interested to hear from anyone seeing this symptom on 15.0.92 (which went out to all Android users yesterday, and into Apple's review process to get it to iOS users), which has the fix for #2800. That bug definitely would cause this behavior, so I'm hoping it was the cause.

@gnprice
Copy link
Member

gnprice commented Sep 18, 2018

Haven't seen or heard of this after that release; so it looks like this was indeed caused by #2800 and is now fixed.

@gnprice gnprice closed this as completed Sep 18, 2018
@gnprice gnprice added a-data-sync Zulip's event system, event queues, staleness/liveness and removed a-notifications labels Nov 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a-data-sync Zulip's event system, event queues, staleness/liveness bug zz-in progress
Projects
None yet
Development

No branches or pull requests

3 participants