Company Cards: Add statement close date to new direct card feeds (using plaid)#65597
Conversation
|
@eVoloshchak Please copy/paste the Reviewer Checklist from here into a new comment on this PR and complete it. If you have the K2 extension, you can simply click: [this button] |
|
@eVoloshchak Please ignore, @ikevin127 will review |
JS00001
left a comment
There was a problem hiding this comment.
Looking good, one question: https://expensify.slack.com/archives/C0920G9GJ0N/p1751976750758379?thread_ts=1751923588.775919&cid=C0920G9GJ0N
luacmartins
left a comment
There was a problem hiding this comment.
Additionally, can we fill out the OP with tests steps please
|
|
||
| const [cardFeeds] = useCardFeeds(policyID); | ||
|
|
||
| const isPlaid = isBetaEnabled(CONST.BETAS.PLAID_COMPANY_CARDS) && !!addNewCard?.data?.publicToken; |
There was a problem hiding this comment.
We released this beta to everyone, so no need to check for it here
There was a problem hiding this comment.
I copied this logic from src/pages/workspace/companyCards/BankConnection/index.tsx I think it's okay to keep it for consistency. Ideally I think there should be a PR that removes that beta.
There was a problem hiding this comment.
I think there will be a separate PR to remove this logic. We could also remove this instance in this PR, but NAB.
Reviewer Checklist
Screenshots/VideosAndroid: HybridApp ✅android-hybrid.mp4Android: mWeb Chrome ☑️iOS: HybridApp ☑️iOS: mWeb Safari ✅ios-mweb.mp4MacOS: Chrome / Safari ✅web.movMacOS: Desktop ✅desktop.mov |
|
@s77rt for this step The doc says this:
Let's make sure that we're defaulting to the last day of the month so we don't block users on this flow |
|
I missed that, thanks for the catch. Just added |
|
@ikevin127 When testing this you may get an error in the select country page, this is a bug in main, you can just close the error and move forward. (reported here #65636 (comment)) |
ikevin127
left a comment
There was a problem hiding this comment.
LGTM ![]()
Let's make sure that we're defaulting to the last day of the month so we don't block users on this flow
This was fixed:
web-updated.mov
@ikevin127 When testing this you may get an error in the select country page, this is a bug in main, you can just close the error and move forward.
I reproduced it too, seems like a deploy blocker, commented on the offending PR as well, hopefully they fix it asap ✅
|
Looks good, lets let @luacmartins review & merge |
|
✋ 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.1.80-0 🚀
|
|
🚀 Deployed to production by https://github.com/luacmartins in version: 9.1.80-8 🚀
|
Explanation of Change
Fixed Issues
$ #64484
PROPOSAL:
Tests
Offline tests
n/a
QA Steps
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))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
Android: mWeb Chrome
iOS: Native
Screen.Recording.2025-07-09.at.4.52.03.PM.mov
iOS: mWeb Safari
MacOS: Chrome / Safari
Screen.Recording.2025-07-09.at.4.46.53.PM.mov
MacOS: Desktop