marthendalnunes
commented
1 year ago Looking good! Before a more in-depth code review I'd like to point a few UX improvements that can be made:
- Handle unsupported chains. In the image below I've connected to Celo, which is unsupported by Aave, but the page still shows the dashboard with assets. I'd recommed to check if the chain is supported and if not just show a message saying that it's an unsupported chain rather than rendering the dashboard.
- The supply/borrow select is not needed unless you're in a small screen, so you can conditionally hide it with tailwind
- (Optional) On the supply table you could hide assets with 0 balance as default and add a checkbox to toggle it (like in Aave's UI)
vercel[bot]
@0xChin is attempting to deploy a commit to the District Labs Team on Vercel.
A member of the Team first needs to authorize it.