search

Expensify / App

Welcome to New Expensify: a complete re-imagination of financial collaboration, centered around chat. Help us build the next generation of Expensify by sharing feedback and contributing to the code.
https://new.expensify.com
MIT License
3.34k stars 2.77k forks source link

[Wave Collect] [Xero] [Advanced] Create the Bill Payment Account Selector Page #39751

Closed lakchote closed 3 months ago

lakchote commented 5 months ago

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.

melvin-bot[bot] commented 5 months ago

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

miljakljajic commented 5 months ago

Commented in SWM: https://expensify.slack.com/archives/C04878MDF34/p1713189922506479?thread_ts=1712333484.848839&cid=C04878MDF34

SzymczakJ commented 5 months ago

Hey! I’m Jakub Szymczak from Software Mansion, an expert agency, and I’d like to work on this issue!

miljakljajic commented 5 months ago

Thank you @SzymczakJ :)

miljakljajic commented 4 months ago

How we doing on this @SzymczakJ

SzymczakJ commented 4 months ago

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 😃

mananjadhav commented 4 months ago

I am working on this one. (slack thread)

melvin-bot[bot] commented 4 months ago

Triggered auto assignment to @madmax330, see https://stackoverflow.com/c/expensify/questions/7972 for more details.

melvin-bot[bot] commented 4 months ago

⚠️ Looks like this issue was linked to a Deploy Blocker here

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.

pecanoro commented 4 months ago

Reassigning since @lakchote and I ended up reviewing the associated PR.

trjExpensify commented 4 months ago

Merged into the Xero branch

trjExpensify commented 4 months ago

Deployed to staging yesterday.

melvin-bot[bot] commented 3 months ago

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!

pecanoro commented 3 months ago

Do we need to pay @mananjadhav here before closing?

hungvu193 commented 3 months ago

I think No. We will be paid centrally when Xero project's done

pecanoro commented 3 months ago

Closing then!