Skip to content

Clean up the getOutstandingChildRequest logic to match holistically what is in backend #39953

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

Open
mountiny opened this issue Apr 9, 2024 · 32 comments
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Internal Requires API changes or must be handled by Expensify staff Monthly KSv2

Comments

@mountiny
Copy link
Contributor

mountiny commented Apr 9, 2024

The getOutstandingChildRequest logic is used to determine if the chat should have GBR in relation to money request. There is many cases this can work so we should make sure we correctly handle this optimistically.

I will try to clean this up to match the backend logic as much as possible to avoid any flashing of the GBR

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~01f7e94d9a1d232c0e
  • Upwork Job ID: 1777725361246265344
  • Last Price Increase: 2024-04-09
Issue OwnerCurrent Issue Owner: @mountiny
@mountiny mountiny added Daily KSv2 Internal Requires API changes or must be handled by Expensify staff Bug Something is broken. Auto assigns a BugZero manager. labels Apr 9, 2024
@mountiny mountiny self-assigned this Apr 9, 2024
Copy link

melvin-bot bot commented Apr 9, 2024

Job added to Upwork: https://www.upwork.com/jobs/~01f7e94d9a1d232c0e

Copy link

melvin-bot bot commented Apr 9, 2024

Triggered auto assignment to Contributor Plus for review of internal employee PR - @parasharrajat (Internal)

Copy link

melvin-bot bot commented Apr 9, 2024

Triggered auto assignment to @stephanieelliott (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

@mountiny mountiny added Weekly KSv2 and removed Daily KSv2 labels Apr 9, 2024
@mountiny
Copy link
Contributor Author

mountiny commented Apr 9, 2024

Making ths weekly now as I am not sure I will get to this tomorrow and then I am ooo

@stephanieelliott stephanieelliott removed the Bug Something is broken. Auto assigns a BugZero manager. label Apr 23, 2024
@melvin-bot melvin-bot bot added the Overdue label Apr 23, 2024
@stephanieelliott stephanieelliott added Bug Something is broken. Auto assigns a BugZero manager. and removed Overdue labels Apr 23, 2024
@melvin-bot melvin-bot bot added the Overdue label Apr 23, 2024
Copy link

melvin-bot bot commented Apr 23, 2024

Triggered auto assignment to @laurenreidexpensify (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.

@melvin-bot melvin-bot bot added Daily KSv2 and removed Overdue Weekly KSv2 labels Apr 23, 2024
@stephanieelliott
Copy link
Contributor

Reapplying the Bug label to get another BZ member on this while I am OOO til May 2. Thanks @laurenreidexpensify I'll grab this back from you when I return!

@mountiny
Copy link
Contributor Author

Still havent got to this as its clean up

@mountiny mountiny added Weekly KSv2 and removed Daily KSv2 labels Apr 23, 2024
Copy link

melvin-bot bot commented Apr 23, 2024

@parasharrajat @stephanieelliott @mountiny @laurenreidexpensify 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!

@trjExpensify
Copy link
Contributor

Gotcha, so you're saying right now the GBR is not applied if an admin submits a report to themselves for example?

@mountiny
Copy link
Contributor Author

Screen.Recording.2024-08-15.at.16.46.45.mov

Yeah you saw the delay in there

Screen.Recording.2024-08-15.at.16.54.45.mov

When offline no GBR

@trjExpensify
Copy link
Contributor

Ah right okay, so online there's a delay and offline it's not there at all.

I guess it's not that important, but it is incorrect.

@melvin-bot melvin-bot bot added the Overdue label Sep 16, 2024
@mountiny
Copy link
Contributor Author

Still in a backlog

@mountiny
Copy link
Contributor Author

low

@mountiny
Copy link
Contributor Author

same

@melvin-bot melvin-bot bot removed the Overdue label Nov 21, 2024
@melvin-bot melvin-bot bot added the Overdue label Dec 23, 2024
@parasharrajat
Copy link
Member

@mountiny Any plan for this yet?

@mountiny
Copy link
Contributor Author

mountiny commented Jan 6, 2025

i will have to circle back to this one, prioritizing other issues now

@melvin-bot melvin-bot bot removed the Overdue label Jan 6, 2025
@melvin-bot melvin-bot bot added the Overdue label Feb 7, 2025
@mountiny
Copy link
Contributor Author

mountiny commented Feb 7, 2025

low

@melvin-bot melvin-bot bot removed the Overdue label Feb 7, 2025
@melvin-bot melvin-bot bot added the Overdue label Mar 11, 2025
@mountiny
Copy link
Contributor Author

low

@melvin-bot melvin-bot bot removed the Overdue label Mar 11, 2025
@melvin-bot melvin-bot bot added the Overdue label Apr 14, 2025
@mountiny
Copy link
Contributor Author

Low

@melvin-bot melvin-bot bot removed the Overdue label Apr 14, 2025
@melvin-bot melvin-bot bot added the Overdue label May 16, 2025
@mountiny
Copy link
Contributor Author

low

@melvin-bot melvin-bot bot removed the Overdue label May 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something is broken. Auto assigns a BugZero manager. Internal Requires API changes or must be handled by Expensify staff Monthly KSv2
Projects
Status: No status
Development

No branches or pull requests

5 participants