-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
Comments
Job added to Upwork: https://www.upwork.com/jobs/~01f7e94d9a1d232c0e |
Triggered auto assignment to Contributor Plus for review of internal employee PR - @parasharrajat ( |
Triggered auto assignment to @stephanieelliott ( |
Making ths weekly now as I am not sure I will get to this tomorrow and then I am ooo |
Triggered auto assignment to @laurenreidexpensify ( |
Reapplying the |
Still havent got to this as its clean up |
@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! |
Gotcha, so you're saying right now the GBR is not applied if an admin submits a report to themselves for example? |
Screen.Recording.2024-08-15.at.16.46.45.movYeah you saw the delay in there Screen.Recording.2024-08-15.at.16.54.45.movWhen offline no GBR |
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. |
Still in a backlog |
low |
same |
@mountiny Any plan for this yet? |
i will have to circle back to this one, prioritizing other issues now |
low |
low |
Low |
low |
Uh oh!
There was an error while loading. Please reload this page.
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
Issue Owner
Current Issue Owner: @mountinyThe text was updated successfully, but these errors were encountered: