Add 'Exported to' to custom columns#77946
Conversation
Codecov Report❌ Looks like you've decreased code coverage for some files. Please write tests to increase, or at least maintain, the existing level of code coverage. See our documentation here for how to interpret this table.
|
|
waiting on some more icons for this |
Reviewer Checklist
Screenshots/VideosAndroid: HybridAppAndroid: mWeb ChromeiOS: HybridAppiOS: mWeb SafariMacOS: Chrome / SafariScreen.Recording.2025-12-18.at.18.32.30.mov |
This reverts commit 50f72db.
|
✋ This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release. |
|
🚀 Deployed to staging by https://github.com/luacmartins in version: 9.2.85-0 🚀
|
|
🚀 Deployed to production by https://github.com/marcaaron in version: 9.2.85-7 🚀
|
| if (isExportedToIntegrationAction(action)) { | ||
| const label = getOriginalMessage(action)?.label; |
There was a problem hiding this comment.
Coming from the #78226 checklist: when an export template is created on OldDot or when exporting “All Data,” we should verify it using the action type EXPORT_TEMPLATE and mark it as isExportedToCsv.
Explanation of Change
Adds support for 'exported to' in the expense reports page & expenses page view, via custom columns
Fixed Issues
$ #76865
Tests
The 'exported to' column should not be visible by default
Open the transaction page 'Columns' RHP
Toggle 'exported to' on
Ensure that the 'exported to' column is visible and accurate
Open the reports page 'Columns' RHP
Toggle 'exported to' on
Ensure that the 'exported to' column is visible and accurate
Offline tests
N/A
QA Steps
Same as tests
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./** comment above it */thisproperly so there are no scoping issues (i.e. foronClick={this.submit}the methodthis.submitshould be bound tothisin the constructor)thisare necessary to be bound (i.e. avoidthis.submit = this.submit.bind(this);ifthis.submitis never passed to a component event handler likeonClick)Screenshots/Videos
Screen.Recording.2025-12-17.at.5.45.22.PM.mov