-
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
[Create Expense] Submits To Display Name Missing #56609
Comments
Triggered auto assignment to @alexpensify ( |
HI I'm Julian from Callstack and I'd like to work on this issue |
Update: I started working on this issue today. |
Update: I wanted to check if #56770 fixes this issue but unfortunately it doesn't. I am still able to reproduce it on that other branch, using a workspace that has many members on iOS. |
FYI I'm going to be OOO next week, coming back on Monday February 24th. |
Will be picked by @JKobrynski once they're back from OOO. |
Making weekly until Julian's back |
@mananjadhav @alexpensify @grgia @JKobrynski 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! |
@grgia @mananjadhav I'm back, it can be daily again! |
WOOP |
Update: I spent some time on further debugging this issue, no clear solutions yet. |
Update: I had more time today so I did a lot of testing and debugging on different platforms and accounts. It's interesting because it wouldn't reproduce for users who only have 1-2 workspaces (including Applause's Workspace 1). Moreover, now it also reproduces for me on the web, not only iOS. This is just an FYI, still looking for a fix! |
Update: The PR is ready for review! |
Original Thread
Problem:
getAccountsByID
is failing to retrieve the "Submits To" display name in workspaces with a large number of members.Repro Steps:
Submits to
but ones with large members doesn't show.Debug Findings:
ReportUtils.ts
confirms thatsubmitsToAccountDetails
is sometimes undefined.The text was updated successfully, but these errors were encountered: