-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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 16th May] [Wave Collect] [Xero] [Advanced] Create the main Advanced Settings page #39746
Comments
Triggered auto assignment to @Christinadobrzyn ( |
@lakchote are you working on this too? looks like there's no engineer assigned yet so just checking |
Yes, I'll be working on this. |
Updated title with the HOLD issue. |
Yes, we were on hold for #39725 to be merged. It was merged yesterday. Technically we are able to work on this now. I'm going to focus today on fixing the remaining bugs for the Xero authorization flow before working on it. |
I'm working on this |
awesome! thank you @rushatgabhane added you to the GH - we'll monitor the PR #41475 |
Deployed to production 9 hours ago. |
Payouts due—Based on our payment structure, I think this should be $500 since it relates to the roadmap? Do you think?
@rushatgabhane Do we need a regression test for this? |
nudge @rushatgabhane about a regression test for this. Maybe we can use the test steps in the PR for a regression test?
|
Payment will be handled centrally when we wrap the project, we can close this as the regression period is over. Equally, we can hand over the regression tests to Applause once the project wraps (CC: @zanyrenney). Closing it out! |
You need to build the page following the steps outlined here.
This will be worked on by SWM engineers, see Slack discussion here.
Until the Xero authorization flow (NewDot issue) is done (which entails NewDot, Web-E and IS changes), we won't be able to work on this yet.
The text was updated successfully, but these errors were encountered: