Description
What is this about?
This ticket defines the requirements and intended outcomes for an audit of the native mobile Portfolio View. This audit should help identify improvements or PoC. Each of these improvements should have an effort and impact associated.
The following areas of the Portfolio View should be exercised while using preferred performance tooling and a wallet with a predefined set of assets &NFTs to identify problems:
- PV switch to popular networks
- PV switch to current network
- PV add custom network and switch
- PV switch to Tokens & NFTS views
Areas of code to look at:
- Views & Components
- Navigation
- Engine
- Controllers
Scenario
No response
Design
These are the areas that we consider "portfolio view":
Technical Details
UI components
- Global component that displays the tokenList and NFT tab: app/components/Views/Wallet/index.tsx
- Asset list is here app/components/UI/Tokens/index.tsx
- Nft list is here: app/components/UI/CollectibleContracts/index.js
- NetworkFilterDropdown and token import/filter: app/components/UI/Tokens/TokenListControlBar/TokenListControlBar.tsx
- Aggregated balance and percentage: app/components/UI/Tokens/TokenList/PortfolioBalance/index.tsx
Polling logic
Polling is defined here: app/components/hooks/AssetPolling/AssetPollingProvider.tsx where it triggers polling by calling the related controllers.
Polling is initially triggered by the tokenList component.
UI polling will trigger controller polling on the:
CurrencyRateController
TokenRatesController
TokenDetectionController
AccountTrackerController
TokenListController
TokenBalancesController
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
No response