-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
[$250] mweb - IOU - Tapping pay elsewhere button in header grey out IOU preview for few seconds #39314
Comments
👋 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:
|
Triggered auto assignment to @chiragsalian ( |
@chiragsalian 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 |
We think that this bug might be related to #vip-vsp |
I can reproduce this issue on web too. Its more easily reproducible when i set network speed to "slow 3g" in chrome developer tools. I have not verified but i suspect the issue is from the change in this PR - #33124. The optimistic data might not be set the same way as production. imo not a blocker since this is a minor bug and only affects optimistic data. removing blocker label and demoting. cc @robertjchen since I noticed you are working on this - #31345. Should the IOU preview be greyed out or not as the API processes |
Yep, not a blocker for sure- let me followup on the correct behavior. |
@chiragsalian Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
Confirmed that it should NOT be greyed out 👍 |
Job added to Upwork: https://www.upwork.com/jobs/~01d1acc8625fd29f6e |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @Ollyws ( |
Hi! I’m Pedro Guerreiro from Callstack - expert contributor group. I’d like to work on this task! |
Sure, go ahead @pac-guerreiro |
Current assignee @chiragsalian is eligible for the choreEngineerContributorManagement assigner, not assigning anyone new. |
@Ollyws, @chiragsalian Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
📣 @Ollyws 🎉 An offer has been automatically sent to your Upwork account for the Reviewer role 🎉 Thanks for contributing to the Expensify app! |
Sure go for it @pac-guerreiro. When you get the chance let us know your ETA and proposal. |
Currently I'm not able to reproduce this issue 😅 Can you check if this issue is still present? |
Issue is not reproducible on the latest build 1.4.62-10 az_recorder_20240416_193013.mp4 |
Closing this issue since its not reproducible. If it reoccurs feel free to reopen the issue. |
If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!
Version Number: 1.4.58
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/4464523
Issue reported by: Applause - Internal Team
Action Performed:
Expected Result:
In IOU report page, tapping pay elsewhere button in header must not grey out IOU preview for few seconds
Actual Result:
In IOU report page, tapping pay elsewhere button in header grey out IOU preview for few seconds
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Add any screenshot/video evidence
Bug6432142_1711753915643.dp.mp4
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: