-
Notifications
You must be signed in to change notification settings - Fork 3.2k
Attendees - Attendees field is absent on invoice confirmation page but present in invoice thread #59602
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 @zanyrenney ( |
Triggered auto assignment to @MonilBhavsar ( |
💬 A slack conversation has been started in #expensify-open-source |
👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open `StagingDeployCash` deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:
|
@zanyrenney FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors. |
Found the bug. Commented on the PR https://github.com/Expensify/App/pull/58120/files#r2027515540 |
Discussed here https://expensify.slack.com/archives/C01GTK53T8Q/p1743699449593819 |
Thanks @MonilBhavsar |
We're CP'ing a PR to hide the attendee selector, I'm going to remove the blocker label and keep this open as an issue to fix later. |
Thanks! Will make it daily then |
@allroundexperts, @MonilBhavsar, @zanyrenney Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
@allroundexperts Huh... This is 4 days overdue. Who can take care of this? |
This seems like a quality issue where the UI is not matching |
This is being handled here. Let's keep an eye on the PR and see when it gets merged. Currently it's waiting on backend changes from @francoisl |
@allroundexperts @MonilBhavsar @zanyrenney 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! |
@allroundexperts Huh... This is 4 days overdue. Who can take care of this? |
PR undergoing pre-merge testing by QA |
Uh oh!
There was an error while loading. Please reload this page.
If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!
Version Number: 9.1.23-1
Reproducible in staging?: Yes
Reproducible in production?: No
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: Yes, reproducible on both
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Email or phone of affected tester (no customers): [email protected]
Issue reported by: Applause Internal Team
Device used: Mac 15.3 / Chrome
App Component: Other
Action Performed:
Precondition:
Expected Result:
Attendees field should be present or absent on both invoice confirmation page and invoice thread.
Actual Result:
Attendees field is absent on invoice confirmation page but present in invoice thread.
Workaround:
Unknown
Platforms:
Screenshots/Videos
bug.mp4
View all open jobs on GitHub
Issue Owner
Current Issue Owner: @allroundexpertsThe text was updated successfully, but these errors were encountered: