-
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
[HOLD E#326093] [$1000] Chat inconsistency between User A and User B after a Request money #24042
Comments
Triggered auto assignment to @johncschuster ( |
Bug0 Triage Checklist (Main S/O)
|
@johncschuster Eep! 4 days overdue now. Issues have feelings too... |
I haven't gotten to testing this behavior. I'll work on it this afternoon. |
Job added to Upwork: https://www.upwork.com/jobs/~0106ee6101f89ce228 |
Current assignee @johncschuster is eligible for the External assigner, not assigning anyone new. |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @eVoloshchak ( |
Commenting to get Melvin to chill out with the Overdue label. |
Hey, I'm Piotr from Callstack - expert contributor group - I would like to help resolve this issue |
📣 @Piotrfj! 📣
|
@johncschuster, @eVoloshchak Eep! 4 days overdue now. Issues have feelings too... |
Update: I was able to reproduce the issue. Seems like it could be backend issue, I've checked in websocket and there were no events to add new report. |
@johncschuster @eVoloshchak this issue is now 3 weeks old. There is one more week left before this issue breaks WAQ and will need to go internal. What needs to happen to get a PR in review this week? Please create a thread in #expensify-open-source to discuss. Thanks! |
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸 |
@johncschuster, @eVoloshchak 6 days overdue. This is scarier than being forced to listen to Vogon poetry! |
@johncschuster, @eVoloshchak 10 days overdue. Is anyone even seeing these? Hello? |
@eVoloshchak do you agree with @Piotrfj that this could be a backend issue? If so, I'll make this internal. |
I can confirm I was able to reproduce this |
Nice one, @flodnv! |
No update here |
@flodnv, @johncschuster, @eVoloshchak Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
No updates, still focusing on higher priority strategic projects. |
Thanks for the update, @flodnv! Should we downgrade the label on this one? |
I did that but you undid that last week 😄 |
Still no update for now. |
Sorry for the delay. Turns out user B will never receive the pusher update, does not matter how many message there are in the IOU thread, the backend was never sending it, so I fixed that 🙃 PR is in review... |
I will put this on hold for internal issue https://github.com/Expensify/Expensify/issues/326093 |
This issue has not been updated in over 15 days. @flodnv, @johncschuster, @eVoloshchak eroding to Monthly issue. P.S. Is everyone reading this sure this is really a near-term priority? Be brave: if you disagree, go ahead and close it out. If someone disagrees, they'll reopen it, and if they don't: one less thing to do! |
@flodnv, @johncschuster, @eVoloshchak, this Monthly task hasn't been acted upon in 6 weeks; closing. If you disagree, feel encouraged to reopen it -- but pick your least important issue to close instead. |
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:
both users should be able to see chat messages the moment they are sent
Actual Result:
both users cannot see the chat messages at the time they are sent, one of the users must exit the chat to update
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:
Which of our officially supported platforms is this issue occurring on?
Version Number: 1.3.49-0
Reproducible in staging?: y
Reproducible in production?: y
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
Notes/Photos/Videos: Any additional supporting documentation
WhatsApp.Video.2023-08-01.at.22.24.23.mp4
Compartir.pantalla.-.2023-08-01.22_03_22.mp4
Recording.1427.mp4
Expensify/Expensify Issue URL:
Issue reported by: @eusalazar
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1690939247360729
View all open jobs on GitHub
Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @flodnvIssue Owner
Current Issue Owner: @flodnvThe text was updated successfully, but these errors were encountered: