-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
[Create Money Request] Update iouReport optimistically #49717
Comments
Triggered auto assignment to @mallenexpensify ( |
Checking to see if this is same/similar to what Cole reported |
@mallenexpensify, @grgia, @eh2077 Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
1 similar comment
@mallenexpensify, @grgia, @eh2077 Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
Throwing on #wave-collect cuz the other avatar-related issue was on the same. And.. the bug affects collect users. |
@mallenexpensify @grgia @eh2077 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! |
not overdue, this is polish |
@grgia is working on. Gonna drop in Quality project for now |
I haven't jumped back into this as I was working on some higher priority quality issues, but still planning to look into a solution here |
I'm back from OOO on Nov 14th, not assigning another BZ because it's working being worked on internally and payment won't be due til after I'm back. If one is needed please add or post in #contributor-plus to ask for one to be added, thx. |
Yeah I haven't had a chance to take this, I think I may unassign for now if we still think we should fix this |
@grgia , this needs to be Internal, right? It looks like it's |
Context
Follow up to #49172
Reported here: #49172 (comment)
Steps Taken
MULTI TRANSACTIONS FROM DIFFERENT USERS (IOU) AVATAR CASE
Expected Result:
Report preview avatar is both avatars, user A and B
Actual Result:
Report preview avatar is still only user A until refresh
Video
Screen.Recording.2024-09-25.at.10.29.44.PM.mov
Screen.Recording.2024-09-25.at.9.48.46.PM.mov
Solution
I think this is probably an issue with updating the iouReport data optimistically when the IOU is made.
https://github.com/Expensify/App/blob/main/src/libs/actions/IOU.ts#L1998
The text was updated successfully, but these errors were encountered: