-
Notifications
You must be signed in to change notification settings - Fork 3.2k
[Due for payment 2025-03-17] [Create Expense] Submits To Display Name Missing #56609
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
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! |
Just finished with the review, it's working. Have asked for the tests. |
Triggered auto assignment to @slafortune ( |
🚨 Heads up! 🚨 @slafortune —This GitHub might require payment while I'm offline. I'm reassigning the label to avoid delays. Thanks! |
|
The solution for this issue has been 🚀 deployed to production 🚀 in version 9.1.10-6 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue: If no regressions arise, payment will be issued on 2025-03-17. 🎊 For reference, here are some details about the assignees on this issue:
|
@mananjadhav @alexpensify / @slafortune @mananjadhav The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed. Please copy/paste the BugZero Checklist from here into a new comment on this GH and complete it. If you have the K2 extension, you can simply click: [this button] |
Payment Summary
BugZero Checklist (@alexpensify)
|
Contributor: @JKobrynski no payment due - agency |
$250 approved for @mananjadhav |
Uh oh!
There was an error while loading. Please reload this page.
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.Issue Owner
Current Issue Owner: @alexpensify / @slafortuneThe text was updated successfully, but these errors were encountered: