-
Notifications
You must be signed in to change notification settings - Fork 11
Tech Debt
Jorge Galat edited this page Nov 22, 2022
·
12 revisions
- common context or service with easier access to vrp/frp by asset -> this data is shown in many part of the app so no component should be making this calc. Should be centralized in a context or custom hook
- too many nullable types makes it confusing and forces the use of '!'. When the info is there, try stop propagating the nullable
- fix "switch-network" and checks data when user is connected to an unsupported network
- use
localization
from MUI to manage language - Change Tx status, use AAVE as example
Possible custom hooks:
-
useTranslations()
-
useAccountData()
-
useMaturityData()
-
useFloatingData()
-
Introduce react-query for async state management. Replace all async calls (and corresponding effects) with it.
-
Utility hooks for common behaviors.