[HOLD App PR #39094][Wave Collect][QBO] Use WORKSPACE_ACCOUNTING's getRoute when specifying the callback path for the QBO auth flow

[hayata-suenaga]

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~010f098ac205edac04
• Upwork Job ID: 1779907490337906688
• Last Price Increase: 2024-04-15

[melvin-bot]

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

[alexpensify]

Noted to follow the GHS listed in the title.

[trjExpensify]

Making sure this is captured in release 1.

[alexpensify]

Weekly Update:

#38283 has been deployed to staging
#39094 is in a draft PR

[hayata-suenaga]

Weekly Update:
#38283 has been deployed to staging

Thank you very much for the update. A correction on the status of #38283 PR: It hasn't been merged yet 🙇

[alexpensify]

Weekly Update: On hold and waiting on the other GHs.

[trjExpensify]

The (new) PR for implementing the accounting page is done, so we're down to waiting on #39094 for authentication which is deep in review. @hayata-suenaga can you confirm if this is internal or external? The OP is a little... light. 😅 Thanks!

[melvin-bot]

Job added to Upwork: https://www.upwork.com/jobs/~010f098ac205edac04

[melvin-bot]

Triggered auto assignment to Contributor Plus for review of internal employee PR - @getusha (Internal)

[hayata-suenaga]

Hello Tom thank you for keeping an eye on this one 😄 This issue is rather a memo I'm using to make sure that a certain code is used in the implementation. We might not have to do this depending on the outcome of the linked PR, but even if we have to do this, this is just one line change 🙇

[trjExpensify]

Sounds good!

[trjExpensify]

The PR for #39094 has been deployed to staging. Can we make an assessment on this one now, so we either close it or take it off hold? Thanks!

[hayata-suenaga]

The PR uses the environment URL

onPress={() => Link.openLink(getQuickBooksOnlineSetupLink(policyID), environmentURL, false)}

we don't need to go ahead with this issue. Closing this now