Update create expense confirmation page to more clearly indicate automation#78859
Update create expense confirmation page to more clearly indicate automation#78859ShridharGoel wants to merge 62 commits intoExpensify:mainfrom
Conversation
|
Hey, I noticed you changed If you want to automatically generate translations for other locales, an Expensify employee will have to:
Alternatively, if you are an external contributor, you can run the translation script locally with your own OpenAI API key. To learn more, try running: npx ts-node ./scripts/generateTranslations.ts --helpTypically, you'd want to translate only what you changed by running |
|
@shawnborton Can you share this receipt so that I can use it for testing before finalising the changes ? |
|
I triggered another ad-hoc. @Expensify/design do you think any of the remaining requests are blocking changes? If not, I'd love to get a final review from @Pujan92 and merge this and then handle any other requests in a follow-up for polish. This change has been in the works for a really long time so I'd love to try and get it out there, even if it's only 95% awesome. |
|
🚧 @puneetlath 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! 🧪🧪
|
|
Gap still happening for me in the latest adhoc. Here's my test receipt: |
|
Was able to repro with page zoomed out to 50%. Should be fixed now. |
|
🚧 @shawnborton 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! 🧪🧪
|
|
Blank space is fixed for me now. Thanks 😄 |
|
Same, send it! |
|
@puneetlath feel free to give it a final review now |
src/components/MoneyRequestConfirmationListFooter/getImageCompactModeStyle/index.native.ts
Show resolved
Hide resolved
This comment was marked as outdated.
This comment was marked as outdated.
|
@Pujan92 Can you have a look now? |



Explanation of Change
Update create expense confirmation page to more clearly indicate automation
This is continued from #72002 and #75196
Fixed Issues
$ #71601
PROPOSAL:
Tests
Offline tests
QA Steps
Same.
PR Author Checklist
### Fixed Issuessection aboveTestssectionOffline stepssectionQA stepssectioncanBeMissingparam foruseOnyxtoggleReportand notonIconClick)src/languages/*files and using the translation methodSTYLE.md) were followedAvatar, 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.ScrollViewcomponent to make it scrollable when more elements are added to the page.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
Screen.Recording.2026-01-20.at.11.38.30.PM.mov
Android: mWeb Chrome
Screen.Recording.2026-01-20.at.10.59.03.PM.mov
iOS: Native
Screen.Recording.2026-01-20.at.11.13.23.PM.mov
iOS: mWeb Safari
Screen.Recording.2026-01-20.at.10.55.36.PM.mov
MacOS: Chrome / Safari
Screen.Recording.2026-01-20.at.10.52.57.PM.mov