-
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] Handle errors in the connections #39726
Comments
Triggered auto assignment to @sonialiap ( |
On hold for #39725 |
Double-checking with internal engineers, some might be interested to work on this. |
I can take care of that, since I'm already in context because I was doing Xero authorization flow. Could you assign me @mountiny? |
@lakchote thank you for handling this. I have one question: Is this about the error handling when connecting to Xero or error handling for invalid configuration combinations? or is it about error handling for sync errors? |
I think this is purely the connection errors |
Yes @hayata-suenaga, as @mountiny said it's purely showing the connection errors and the RBR. |
On hold for #39725 |
Not on hold anymore, cc @SzymczakJ |
Assigning @mananjadhav as he'll be working on this. |
@lakchote Added the PR to fix the disconnect Xero. |
Deployed to prod 9 hours ago. |
Payout for this will be handled after all Xero issues are done. We can close this after the regression period. |
Yep, agreed. Regression period is over, closing. |
We need to handle Xero connection errors. More details 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: