-
Notifications
You must be signed in to change notification settings - Fork 3k
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
[Guides] Chats that are already read show up as unread in LHN #22603
Comments
Triggered auto assignment to @zanyrenney ( |
Triggered auto assignment to @MariaHCD ( |
Unable to auto-create job on Upwork. The BZ team member should create it manually for this issue. cc @thienlnam |
Triggered auto assignment to Contributor Plus for review of internal employee PR - @mananjadhav ( |
Not able to 100% reproduce the issue in dev. But I managed to see it with:
Screen.Recording.2023-07-12.at.1.00.51.PM.mov
Screen.Recording.2023-07-12.at.1.38.07.PM.mov |
Not sure if related to #21766 because it seems happens for a non-high-traffic account in dev as well (although not 100% reproducible for me) Seems like there are two scenarios in #focus mode:
|
More discussions happening here: https://expensify.slack.com/archives/C04JD99283A/p1689202761085489 |
Update:
Next steps:
Co-assigning you @MonilBhavsar! Let me know if I missed anything above. |
Sounds good 👍 David thinks this issue is reproducible when using same account on different devices, pointing to unreliable pusher delivery - https://expensify.slack.com/archives/C04JD99283A/p1689375389197839 |
@MariaHCD @MonilBhavsar - The Guide [email protected] says that this is still happening for her (Slack thread). Maybe you could use the new V0 Supportal tool to try reproducing in her account? Instructions on using the supportal tool are here: https://stackoverflowteams.com/c/expensify/questions/16975/16976 |
Discussed in slack. Guides are actually facing this issue #17316 |
Does that mean we should close this out @MonilBhavsar ? Or combine the issues and tackle them in one? |
I think we can bump the priority of this issue, as no one is experiencing it at the moment. As I mentioned above, guides are experiencing the other similar issue related to unread messages #17316 |
@mananjadhav @MariaHCD @MonilBhavsar @zanyrenney this issue was created 2 weeks ago. Are we close to a solution? Let's make sure we're treating this as a top priority. Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks! |
sorry what do you mean bump? do you mean punt or reduce the urgency to weekly? @MonilBhavsar |
Yes, I did and Melvin updated to daily again. I think we can close this GH as this issue is no longer faced by anyone. Thoughts @kevinksullivan @sakluger |
Closing in that case, we can reopen if we get more live cases of this! |
If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!
Action Performed:
Expected Result:
The chat that is already read shouldn't show in LHN as unread
Actual Result:
The chat is shown as unread several seconds later
Platforms:
Which of our officially supported platforms is this issue occurring on?
Version Number: v1.3.34-1
Reproducible in staging?: Y
Reproducible in production?: Y
Email or phone of affected tester (no customers): [email protected]
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:
NewDot.Bug.mov
Slack conversation: https://expensify.slack.com/archives/C036S0BVAHH/p1688135931190779
The text was updated successfully, but these errors were encountered: