-
Notifications
You must be signed in to change notification settings - Fork 2.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
add settlement frequency link complete #8050
Conversation
CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅ |
@@ -32,6 +32,13 @@ const WorkspaceCardVBAWithECardView = props => ( | |||
shouldShowRightIcon: true, | |||
iconRight: Expensicons.NewWindow, | |||
}, | |||
{ | |||
title: props.translate('workspace.common.settlementFrequency'), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This needs to be behind the monthlySettlement
beta
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Have been researching and found an SO but all the info is about how to implement Beta in Web-Expensify
You know how i can do same in newdot?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure myself, I don't really work in newDot. But if you search the codebase you should be able to find some examples. Alternatively you can ask in #eng-chat
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
check Permissions.canUseWallet(props.betas)
and you can do something similar
updated and waiting for another review |
hmm I think you hid it behind the wrong beta. |
iconRight: Expensicons.NewWindow, | ||
}, | ||
]; | ||
if (Permissions.canUseWallet(props.betas)) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This isn't the right beta. It should be monthlySettlements
but the beta isn't on newDot. So you have to add that in
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
updated and awaiting another review
updated and awaiting another review |
* @param {Array<String>} betas | ||
* @returns {Boolean} | ||
*/ | ||
function canUseMonthlySettlements(betas) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You need to export this
updated and awaiting another review |
Gentle bump @madmax330 |
✋ 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 @Justicea83 in version: 1.1.46-0 🚀
|
Details
This PR needs this https://github.com/Expensify/Web-Expensify/pull/33250 to be merged before it works.
Fixed Issues
$https://github.com/Expensify/Expensify/issues/199498
Tests
For example:
--->
PR Review Checklist
Contributor (PR Author) Checklist
main
### Fixed Issues
section abovesrc/languages/*
files (if applicable)Styling.md
) for all style edits I madeSTYLE.md
)Avatar
, I verified the components usingAvatar
are working as expected)main
branch)PR Reviewer Checklist
main
### Fixed Issues
section abovesrc/languages/*
files (if applicable)STYLE.md
) were followedAvatar
, I verified the components usingAvatar
are working as expected)main
branch)QA Steps
Screenshots
Web
Mobile Web
Desktop
iOS
Android