-
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
Message didn’t show up in #social as unread in LHN, but I got the email notification. ie. LHN is not updating from posts in #social til it's clicked on #34273
Comments
Triggered auto assignment to @MitchExpensify ( |
@MitchExpensify if this never happens to you, will you unassign and put me on as BZ? Thx |
Still unable to reproduce reliably https://expensify.slack.com/archives/C049HHMV9SM/p1705174729264039?thread_ts=1704231385.914329&cid=C049HHMV9SM |
Switching to Weekly while we figure out how to reproduce |
If I had to guess, the repro step involves putting your computer to sleep or closing the lid. |
Tried closing the lid and sleeping in a variety of ways to no success.. I think I can only close this issue at this point and we can reopen if we are able to reliable reproduce |
Gonna take over for ya Mitch since it keep happening to me, especially with #social 2024-01-16_17-17-53.mp4 |
It's not just #social, just happened with chat with Scott too, got a desktop notification 20 mins ago, noticed it didn't show in LHN, loaded once I clicked it |
Also being discussed here |
Just adding in here that I experience it too - persistently always with #social, but inconsistently with other chats as well. It was worst when we were having issues during harvesting (#fireroom-2024-01-10-site-slowness). And it's definitely worse for chats that have a lot of content. 2024-01-17_16-49-34.mp4 |
Job added to Upwork: https://www.upwork.com/jobs/~01059dcd731dd4aa05 |
There are a lot of potential things to look into. Generally speaking, it is very hard (maybe impossible?) to confirm the delivery of an update from the server to the client in realtime (or after the fact). If you didn't see the LHN update but got an email then the basic problem would logically be one of these:
B has another layer of complexity because we have "reliable updates" now which means that if you miss an update we should be sending you those updates at some point. I think we don't really do that until you reconnect from being offline or take some other kind of action in the app. So it could be that you just "missed an update" and the app is working how it's supposed to work when that happens. |
Thanks for the deets and especially the link. I commented there, asking for an update once I can test via staging. Also also ran into another issue of #social not updating
|
Def doesn't seem to be getting better, I've updated NewDot 4 times today and am continually testing, both Flo and Scott have run into issues today too. https://expensify.slack.com/archives/C049HHMV9SM/p1706569754923139?thread_ts=1706551202.735269&cid=C049HHMV9SM Checking in #ngineering https://expensify.slack.com/archives/C03SSAQ7P/p1706569735098989 |
Maybe this is related to what is being fixed here: https://github.com/Expensify/Web-Expensify/pull/40434
|
Another (potentially) interesting bit. I closed/quit NewDot last night. When I opened it this morn, it showed an old chat in LHN or #social. So, even with a clean/fresh instance, it's still not loading and showing correctly. Thanks for the comment @aldo-expensify , I'm going to follow that PR. |
From my testing yesterday, even if |
Is this specific to Pusher? |
I think the bug makes you miss the message itself and the latest visible message in the LHN (both updates come from the pusher when someone sends a message).
Are you referring to "messages"? are you saying that they sent a message in social that failed to be posted? |
This is the other bug (that I don't think is directly related but ¯_(ツ)_/¯ ) |
I can confirm that messages in Log of the command running out of memory: https://www.expensify.com/_devportal/tools/logSearch/#query=request_id:(%2284dc4505deef2dab-YVR%22)+AND+timestamp:[2024-01-30T18:54:56.360Z+TO+2024-01-30T20:54:56.360Z]&index=logs_expensify-026534 This does allow the message to be posted (saved in the database), but the pusher events are not sent so no one gets notified/updated with the new message |
that looks different, but can't be certain until investigated :P |
ooooh, and the PR is fixing the |
Apparently it is not fixing it completely. In my local testing of the PR, the command doesn't crash anymore, but I still don't get the pusher events. I haven't investigated why the pusher events are still getting lost. |
@akinwale @mallenexpensify this issue is now 3 weeks old. There is one more week left before this issue breaks WAQ. What needs to happen to get a PR in review this week? Please create a thread in #expensify-open-source to discuss. Thanks! |
Tracked another ~dozen updates in #social and it appears that I'm receiving the desktop notification for all chats and that LHN is updating for each comment. |
@akinwale, @mallenexpensify Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
Closing, issue has been fixed, |
If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!
Version Number:
Reproducible in staging?: needs reproduction
Reproducible in production?: needs reproduction
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @JmillsExpensify
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1704231385914329
Action Performed: (Updated by @mallenexpensify on Jan 17)
Precondition: must have access to #social channel in NewDot on the
Expensify Chat
workspace (hence why this needs to be internal, Matt A hasn't been able to reproduce in other rooms)Expected Result:
New chats to show in LHN for the #social room
Actual Result:
New chat don't show in LHN until the chat is clicked on, then chats load in the main #social report and LHN updates.
Workaround:
unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Add any screenshot/video evidence
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: