[HOLD for payment 2025-02-04] [GR Phase 3] Step 3 - navigation, default data, logic and API call to save it

[madmax330]

Design doc section: https://docs.google.com/document/d/1Y4P7G9h0Easjcea7wyk52gKrNB9o1yTjpmJFYkrHDW4/edit?tab=t.0#bookmark=id.o63v6yk67yf5

[MrMuzyk]

I am Michał from Callstack - expert contributor group. I’d like to work on this job.

[MrMuzyk]

Implementation is ongoing. Waiting on more details regarding one of the fields and implementation of the API.

Apart from that, step 2 logic (#50898) has to be implemented so we can fully test this issue

[MrMuzyk]

In order to finish this issue the following needs to happen:

• feat: Update corpay fields logic, integrate API #52812 has to get merged (Step 1 & 2 logic). This creates bankAccountID which is needed to proceed in next steps
• Data send in Step 2 has to be saved somewhere in onyx. At the moment it's not and we cannot access the bankAccountID returned from Step 2. It should be saved inside reimbursementAccount together with all the data saved in Steps 1 & 2.
• Need a final decision on picklists in Step 3. There is a picklist called Trade volume range. It is not on designs and I haven't got a final decision if we should add it or not. If we have to add it then more directions, labels and translations are needed

[MrMuzyk]

I will be OOO from 21.12 - 06.01. Latest update for this issue:

• new picklist has been added
• testing recordings have been added
• It is still waiting on feat: Update corpay fields logic, integrate API #52812 to get merged, however it is rebased with latest changes and was kept up to date
• BE update is still needed - Data send in Step 2 has to be saved somewhere in onyx. At the moment it's not and we cannot access the bankAccountID returned from Step 2. It should be saved inside reimbursementAccount together with all the data saved in Steps 1 & 2. I believe only minor changes will be needed - reading default values when user goes back to this step

[hungvu193]

#52812 is now merge. We can continue with your draft PR when you're back @MrMuzyk

[melvin-bot]

Reviewing label has been removed, please complete the "BugZero Checklist".

[melvin-bot]

The solution for this issue has been 🚀 deployed to production 🚀 in version 9.0.89-8 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue:

• [NO QA] feat: Step 3 logic #52954

If no regressions arise, payment will be issued on 2025-02-04. 🎊

For reference, here are some details about the assignees on this issue:

• @hungvu193 requires payment through NewDot Manual Requests
• @MrMuzyk does not require payment (Contractor)