[CFI] Show Merchant, Date, Amount Fields in the Scan flow - #100426
[CFI] Show Merchant, Date, Amount Fields in the Scan flow#100426thelullabyy wants to merge 2 commits into
Conversation
Codecov Report✅ Changes either increased or maintained existing code coverage, great job!
|
|
🚧 trjExpensify has triggered a test Expensify/App build. You can view the workflow run here. |
|
🧪🧪 Use the links below to test this adhoc build on Android, iOS, and Web. Happy testing! 🧪🧪
|
|
Took the adhoc draft for a quick spin.
Reason being, a user might click "Show more" to reveal more fields on the expense but intends to leave merchant, date, amount blank still - and we would proceed with scanning them automatically if they do leave them blank. I could see us removing the labels on all three when a value is entered in any of them though, as that's really the point in which we won't automatically read them anymore. Curious to hear what Shawn thinks about that.
2026-09-08_14-25-02.mp4
Why is that? I've cleared the three mandatory fields if this was to be created manually instead of scanned, so I should be able to proceed with creating the expense right? 2026-09-08_15-04-07.mp4CC: @shawnborton for thoughts. |
Oh interesting, I didn't realize that. I think that makes sense, otherwise it's odd to say Automatic in a case where we actually don't scan the field. |


Explanation of Change
Fixed Issues
$ #93854
PROPOSAL:
Tests
Offline tests
QA Steps
// TODO: These must be filled out, or the issue title must include "[No QA]."
PR Author Checklist
### Fixed Issuessection aboveTestssectionOffline stepssectionQA stepssectionAvatar, I verified the components usingAvatarare working as expected)StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))npm run compress-svg)Avataris modified, I verified thatAvataris working as expected in all cases)Designlabel and/or tagged@Expensify/designso the design team can review the changes.mainbranch was merged into this PR after a review, I tested again and verified the outcome was still expected according to theTeststeps.Screenshots/Videos
Android: Native
Android: mWeb Chrome
iOS: Native
iOS: mWeb Safari
MacOS: Chrome / Safari