-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
[Awaiting Payment May 16th] [Wave Collect] [Xero] Put Xero display logic under a specific beta #41394
Comments
ProposalPlease re-state the problem that we are trying to solve in this issue.Put Xero in beta. What is the root cause of that problem?New change. What changes do you think we should make in order to solve the problem?Add the below logic in PolicyAccountingPage:
This will ensure that Xero will not show only if it's not already connected. In Permissions.ts:
|
@ShridharGoel since I've assigned myself already, no need to submit proposals. Thank you. |
Deployed to prod 9 hours ago. Assigning Rushat for the PR review. |
Payment summary as follows: $250 to @rushatgabhane for the C+ review of the PR |
Payment will be handled centrally when we wrap the project, we can close this as the regression period is over. |
We need to put Xero under a specific beta since QBO will be released sooner and they share the same accounting menu.
The text was updated successfully, but these errors were encountered: