-
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
HIGH: [Public] Thread messages are missing from public room #38159
Comments
Job added to Upwork: https://www.upwork.com/jobs/~01fe380d1a88aa6cc2 |
Triggered auto assignment to Contributor Plus for review of internal employee PR - @abdulrahuman5196 ( |
Triggered auto assignment to @anmurali ( |
Seems like David's OpenReport call is not even getting the childReportID property and other thread related properties for the report actions which should be added here https://github.com/Expensify/Auth/blob/bfd557749a4f372ad331f74defb7df026fe1037d/auth/lib/Report.cpp#L6984-L7031 |
My plate is quite full right now focusing on performance and Critical wave collect tasks so sending this to the pool if anyone has spare cycles to dig into it. Given the reproduction steps will be quite tough locally it might take some time. |
Still haven't picked up |
@quinthar do you want to do an open call for volunteers for this bug? |
Can look into this |
Thanks @jasperhuangg ! |
@jasperhuangg is working on this now. |
I'm going OOO for the rest of this week, so I'm looking to get started on this next week |
@jasperhuangg I know you just got back today but can you add an update either here or in Slack so we can keep the issue up to date? |
is this still happening for anyone? This is working fine for me. I'm going to just close this; reopen if it's happening to you. |
Tried to reproduce it as well with the same room and I'm able to load all the threads. Feel free to reopen if we see it crop up again and provide the reportID of the parent report and the thread, thanks! |
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 reproduciton
Reproducible in production?: needs reproduciton
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers): @quinthar https://staging.new.expensify.com/r/868417741568295 and https://staging.new.expensify.com/r/6776919265133947
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @quinthar
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1710204955795119
Action Performed:
Expected Result:
User should be able to view all the thread messages
Actual Result:
Thread messages are disappaeared
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: