Repository navigation
[Pay 4/29][$250] [Track Tax] Error on Amount and Merchant field after changing tax rate when scanning is ongoing #39699
Description
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.60-8
Reproducible in staging?: Y
Reproducible in production?: N
Logs: https://stackoverflow.com/c/expensify/questions/4856
Issue reported by: Applause-Internal team
Action Performed:
Precondition:
- User is an employee of Collect workspace.
- The Collect workspace has several tax rates.
- Go to staging.new.expensify.com
- Go to workspace chat.
- Create a scan request and include a tax rate.
- When scanning is in progress, go to transaction thread.
- Click Tax rate and change to a different tax rate.
Expected Result:
No error on Amount and Merchant field will show up after changing the tax rate.
Actual Result:
Error on Amount and Merchant field shows up briefly after changing the tax rate when scnaning is ongoing.
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
- Android: Native
- Android: mWeb Chrome
- iOS: Native
- iOS: mWeb Safari
- MacOS: Chrome / Safari
- MacOS: Desktop
Screenshots/Videos
Bug6438765_1712290267104.20240405_120557.mp4
Upwork Automation - Do Not Edit
- Upwork Job URL: https://www.upwork.com/jobs/~011d90b50207f85d2f
- Upwork Job ID: 1777350902764363776
- Last Price Increase: 2024-04-08
- Automatic offers:
- cubuspl42 | Reviewer | 0
- tienifr | Contributor | 0
Activity
melvin-bot commentedon Apr 5, 2024
Triggered auto assignment to @lakchote (
DeployBlockerCash), see https://stackoverflowteams.com/c/expensify/questions/9980/ for more details.melvin-bot commentedon Apr 5, 2024
Triggered auto assignment to @adelekennedy (
Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.github-actions commentedon Apr 5, 2024
👋 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:
izarutskaya commentedon Apr 5, 2024
@adelekennedy 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.
izarutskaya commentedon Apr 5, 2024
We think this issue might be related to the #collect project.
55 remaining items
mallenexpensify commentedon May 1, 2024
@adelekennedy , thanks for double checking. This was an edge case (and/or a mistake on my part), I've paid @cubuspl42 for this outside of this issue. I checked to box above,
@lakchote , @adelekennedy @tienifr do we want a regression test for this?
melvin-bot commentedon May 3, 2024
@lakchote @adelekennedy @tienifr this issue is now 4 weeks old, please consider:
Thanks!
lakchote commentedon May 6, 2024
I think a regression test can't hurt to prevent this from happening in the future and ensuring a smooth user experience.
adelekennedy commentedon May 6, 2024
Then to close this out @tienifr will you suggest a regression test and I can create an issue for it?
mvtglobally commentedon May 6, 2024
Issue not reproducible during KI retests. (First week)
adelekennedy commentedon May 8, 2024
bump @tienifr for the test steps!