decentdao / decent-interface

Govern at startup speed
https://app.decentdao.org
MIT License
14 stars 7 forks source link

`2515` Update typing and simplify DAO loading logic #2516

Closed Da-Colon closed 6 days ago

Da-Colon commented 1 week ago

Closes #2515

During the screenshare earlier, QA'ing the work on stores, Noticed that there was an extra address property while loading DAOInfo, along with some other weird typing. Continuing from there noticed the similar logic and disconnect between loading the initial DAO info and the Organization page.

This improves I believe the whole setup, deletes a bunch of code and looks like loading is quite smoother(er?)

Check out Myosin's Organization page to stress test the loading.

/hierarchy?dao=base:0x0c14aA2d34C0E13Bbe441963e4de2EeAef4abeD2

netlify[bot] commented 1 week ago

Deploy Preview for decent-interface-dev ready!

Name Link
Latest commit cb21c2eb41f36895461f7bdb41e9301e2c0073d8
Latest deploy log https://app.netlify.com/sites/decent-interface-dev/deploys/673c9d86b1ef16000823a8b8
Deploy Preview https://deploy-preview-2516.app.dev.decentdao.org
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.