[HOLD for payment 2023-06-15] [HOLD for payment 2023-05-09] [Navigation Reboot] Remove the drawer navigator

[mountiny]

Tracking issue #11768

Created a new branch navigation-refactor to which we will merge all the PRs of this refactor.

Remove the drawer navigator and extract the SidebarScreen and Report screens to the ResponsiveNavigator

Bookmark in the doc

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~01039b367e157236fb
• Upwork Job ID: 1634918647147466752
• Last Price Increase: 2023-03-12

[gauravmishra-17]

Hi is there no bounty for this issue? Anyways I wanted to start contributing for native iOS, could someone point me to the starting point? For example, how I onboard on the project?

[mountiny]

@gauravmishra-17 sorry this one is not available, can you please give this doc a good read and get into our Slack channels. All instructions are there.

[adamgrzybowski]

Going to implement this

[MelvinBot]

Job added to Upwork: https://www.upwork.com/jobs/~01039b367e157236fb

[MelvinBot]

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

[adamgrzybowski]

Work will resume now once ECX is done

[mountiny]

This one will be done sooner as part of one PR with other two issues

[mountiny]

Unassigning CPlus for now, I will ask in the channel once the PR will be ready

[adamgrzybowski]

Still working on this. It's not that easy to create separate PR just for this issue so the Implementation will be pushed in a single PR including:

#15848
#15851
#15852

[mountiny]

Actively being worked on

[MelvinBot]

@mountiny, @adamgrzybowski Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

[mountiny]

Actively being worked on

[MelvinBot]

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

[mountiny]

Lets handle payments for this massive PR. It was Rajat and Aimane reviewing, although to be fair it was Rajat who spent the most time on this so I believe he should be rewarded more.

• @parasharrajat $2000
• @aimane-chnaif $1000 - 8 comments on the PR

[mountiny]

@miljakljajic Would you be bale to help us with the payments here, its described in the comment above. Thank you!

[miljakljajic]

Sorry for the delay! I will pay right away.

[miljakljajic]

Offers sent to both of you :)

[parasharrajat]

Accepted. Thanks

[miljakljajic]

Both of you have been paid, and contracts ended!

[melvin-bot]

⚠️ 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.

[melvin-bot]

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

• [Navigation Reboot] Swap the DrawerNavigator for the ResponsiveNavigator #16581

If no regressions arise, payment will be issued on 2023-06-15. 🎊

After the hold period is over and BZ checklist items are completed, please complete any of the applicable payments for this issue, and check them off once done.

• External issue reporter
• Contributor that fixed the issue
• Contributor+ that helped on the issue and/or PR

As a reminder, here are the bonuses/penalties that should be applied for any External issue:

• Merged PR within 3 business days of assignment - 50% bonus
• Merged PR more than 9 business days after assignment - 50% penalty