Add dashboard summary #2630
Add dashboard summary #2630
Conversation
|
PR deployed in Google Cloud |
|
PR deployed in Google Cloud |
kattylucy
force-pushed
the
2585_assets_page
branch
from
32fe42a to
e830027
Compare
kattylucy
force-pushed
the
2582_landing_dashboard_page
branch
from
5a37f28 to
a535ffe
Compare
kattylucy
force-pushed
the
2585_assets_page
branch
from
e830027 to
f70c459
Compare
kattylucy
force-pushed
the
2582_landing_dashboard_page
branch
from
4ae980e to
97abeb9
Compare
| useEffect(() => { | ||
| if (selectedPools.length === 0 && pools.length > 0) { | ||
| setSelectedPools(pools.map((pool) => pool.id)) | ||
| } | ||
| }, [pools.length, selectedPools.length, setSelectedPools, pools]) |
There was a problem hiding this comment.
I would rather see this in the selected pool context, I think it could be a valuable option to default select all pools.
https://github.com/centrifuge/apps/pull/2609/files#diff-fa36ec06969a8437b730d7d7bbbdd6736b00b791e6dbdd9eb8892ed5054cd861R15
I tried it like this, but let me know if you have a better idea.
There was a problem hiding this comment.
I think you would merge investors first and I will grab your change! Thank you
| ;<Select | ||
| options={aumOptions} | ||
| onChange={(e) => setSelectedAumOption(aumOptions.find((option) => option.value === e.target.value) ?? aumOptions[0])} | ||
| value={selectedAumOption.value} | ||
| variant="secondary" | ||
| hideBorder | ||
| small | ||
| /> |
There was a problem hiding this comment.
forgot to delete maybe?
There was a problem hiding this comment.
I have no idea what this is but yeah haha
kattylucy
force-pushed
the
2582_landing_dashboard_page
branch
from
97abeb9 to
79ea199
Compare
kattylucy
force-pushed
the
2582_landing_dashboard_page
branch
from
79ea199 to
0052e55
Compare
kattylucy
force-pushed
the
2582_landing_dashboard_page
branch
from
4899f70 to
cc2fe1f
Compare
Add new landing page for our dashboard page - it does not include the CTA drawers.
#2582