-
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
[HOLD for Payment 2024-09-6][$250] iOS - Scan - After taking picture, image is not displayed on confirmation screen #48118
Comments
Triggered auto assignment to @neil-marcellini ( |
👋 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:
|
Job added to Upwork: https://www.upwork.com/jobs/~01cc5d61efd7cbd67d |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @situchan ( |
I'm suspicious of this PR On paste file add file extension if not found, so I'm going to put up a revert, create an adhoc build, and see if the issue is solved. I'll also work on this locally. |
That PR can't be the culprit as it's only related to pasting image in the composer |
Removing the regular deployblocker label since this is an iOS only bug |
I ran into a bunch of problems trying to get App running on my iPhone, so I'll have to check back tomorrow. |
Maybe its the RN upgrade. Can you please check @MrRefactor @WoLewicki? |
I finally got the App running on my iPhone and I'm able to reproduce. I'll try reverting the RN upgrade now. |
📣 @ishpaul777 🎉 An offer has been automatically sent to your Upwork account for the Contributor role 🎉 Thanks for contributing to the Expensify app! Offer link |
I found the root cause and had @ishpaul777 author and test the fix. We didn't use another C+. Assigning a BZ member to handle payment. |
Triggered auto assignment to @adelekennedy ( |
Verified fixed here |
not overdue, [HOLD for Payment 2024-09-6] |
@neil-marcellini, @adelekennedy, @ishpaul777 Huh... This is 4 days overdue. Who can take care of this? |
Updated title |
not overdue Paydate is tomorrow! |
Payouts due:
Upwork job is here. |
Closing! |
Thank you! |
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: .0.25-8
Reproducible in staging?: Y
Reproducible in production?: N
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
Action Performed:
Expected Result:
The taken picture should be displayed on the confirmation screen
Actual Result:
The taken picture is not displayed on the confirmation screen, an empty space is displayed instead
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Add any screenshot/video evidence
Bug6584474_1724784788375.Rpreplay_Final1724783767.mp4
View all open jobs on GitHub
Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @adelekennedyThe text was updated successfully, but these errors were encountered: