MetaMask / MetaMask/metamask-mobile

Parse Navigators from MainNavigator

Open
#12,557 0 comments 0 reactions 1 assignee Claimed by @frankvonhoven View on GitHub
team-mobile-platform
Dominant language
TypeScript
Stars
3k
Forks
1.7k
Avg merge
1d 14h
Merged PRs (30d)
669

Description

#### What is this about?

This is to track the work around separating Navigators out of the single file MainNavigator. This would establish a pattern and reduce the file size and separate concerns into manageable Folders for navigation.

## Potential Folder Structure Solution:
- Nav
- App
- Main
- [MainNavigator.js](./MainNavigator.js)
- Wallet
- [WalletModalFlow.js](./Wallet/WalletModalFlow.js)
- [WalletTabStackFlow.js](./Wallet/WalletTabStackFlow.js)
- [WalletTabModalFlow.js](./Wallet/WalletTabModalFlow.js)
- Asset
- [AssetStackFlow.js](./Asset/AssetStackFlow.js)
- [AssetModalFlow.js](./Asset/AssetModalFlow.js)
- Transactions
- [TransactionsHome.js](./Transactions/TransactionsHome.js)
- Browser
- [BrowserFlow.js](./Browser/BrowserFlow.js)
- Snaps
- [SnapsSettingsStack.js](./Snaps/SnapsSettingsStack.js)
- Notifications
- [NotificationsOptInStack.js](./Notifications/NotificationsOptInStack.js)
- [NotificationsModeView.js](./Notifications/NotificationsModeView.js)
- Settings
- [SettingsFlow.js](./Settings/SettingsFlow.js)
- Home
- [HomeTabs.js](./Home/HomeTabs.js)
- Webview
- [Webview.js](./Webview/Webview.js)
- Send
- [SendView.js](./Send/SendView.js)
- [SendFlowView.js](./Send/SendFlowView.js)
- NFT
- [NftDetailsModeView.js](./Nft/NftDetailsModeView.js)
- [NftDetailsFullImageModeView.js](./Nft/NftDetailsFullImageModeView.js)
- Bookmark
- [AddBookmarkView.js](./Bookmark/AddBookmarkView.js)
- Offline
- [OfflineModeView.js](./Offline/OfflineModeView.js)
- Payment
- [PaymentRequestView.js](./Payment/PaymentRequestView.js)
- Swaps
- [Swaps.js](./Swaps/Swaps.js)
- Password
- [SetPasswordFlow.js](./Password/SetPasswordFlow.js)

### Scenario

_No response_

### Design

_No response_

### Technical Details

_No response_

### Threat Modeling Framework

_No response_

### Acceptance Criteria

_No response_

### Stakeholder review needed before the work gets merged

- [ ] Engineering (needed in most cases)
- [ ] Design
- [ ] Product
- [ ] QA (automation tests are required to pass before merging PRs but not all changes are covered by automation tests - please review if QA is needed beyond automation tests)
- [ ] Security
- [ ] Legal
- [ ] Marketing
- [ ] Management (please specify)
- [ ] Other (please specify)

### References

[Previous exploration](https://github.com/MetaMask/mobile-planning/issues/1966)

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.