[Awaiting Payment 16th May] [Wave Collect] [Xero] Handle errors in the connections

[lakchote]

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.

[melvin-bot[bot]]

Triggered auto assignment to @sonialiap (NewFeature), see https://stackoverflowteams.com/c/expensify/questions/14418#:~:text=BugZero%20process%20steps%20for%20feature%20requests for more details.

[sonialiap]

On hold for https://github.com/Expensify/App/issues/39725

[lakchote]

Double-checking with internal engineers, some might be interested to work on this.

[SzymczakJ]

I can take care of that, since I'm already in context because I was doing Xero authorization flow. Could you assign me @mountiny?

[hayata-suenaga]

@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?

[mountiny]

I think this is purely the connection errors

[lakchote]

I think this is purely the connection errors

Yes @hayata-suenaga, as @mountiny said it's purely showing the connection errors and the RBR.

[sonialiap]

On hold for https://github.com/Expensify/App/issues/39725

[lakchote]

Not on hold anymore, cc @SzymczakJ

[lakchote]

Assigning @mananjadhav as he'll be working on this.

[mananjadhav]

@lakchote Added the PR to fix the disconnect Xero.

[trjExpensify]

Deployed to prod 9 hours ago.

[mananjadhav]

Payout for this will be handled after all Xero issues are done. We can close this after the regression period.

[trjExpensify]

Yep, agreed. Regression period is over, closing.