-
Notifications
You must be signed in to change notification settings - Fork 3k
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
[Wave Collect] [Xero] [Advanced] Create the Bill Payment Account Selector Page #39751
Comments
Triggered auto assignment to @miljakljajic ( |
Hey! I’m Jakub Szymczak from Software Mansion, an expert agency, and I’d like to work on this issue! |
Thank you @SzymczakJ :) |
How we doing on this @SzymczakJ |
I'm implementing Advanced settings page along with this Bill Payment Account Selector Page. I'll be OOO next week but I will try to put out a draft PR for this today 😃 |
I am working on this one. (slack thread) |
Triggered auto assignment to @madmax330, see https://stackoverflow.com/c/expensify/questions/7972 for more details. |
If you are the assigned CME please investigate whether the linked PR caused a regression and leave a comment with the results. If a regression has occurred and you are the assigned CM follow the instructions here. If this regression could have been avoided please consider also proposing a recommendation to the PR checklist so that we can avoid it in the future. |
Reassigning since @lakchote and I ended up reviewing the associated PR. |
Merged into the Xero branch |
Deployed to staging yesterday. |
This issue has not been updated in over 15 days. @pecanoro, @lakchote, @miljakljajic, @SzymczakJ eroding to Monthly issue. P.S. Is everyone reading this sure this is really a near-term priority? Be brave: if you disagree, go ahead and close it out. If someone disagrees, they'll reopen it, and if they don't: one less thing to do! |
Do we need to pay @mananjadhav here before closing? |
I think No. We will be paid centrally when Xero project's done |
Closing then! |
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: