mstable / mStable-apps

All mStable apps
GNU Lesser General Public License v3.0
14 stars 24 forks source link

208 notify users with legacy earn add withdrawal process #209

Closed toniocodo closed 2 years ago

github-actions[bot] commented 2 years ago

Visit the preview URL for this PR (updated for commit d7f1305):

https://apps-staking--pr209-208-notify-users-wit-3pi7cn5d.web.app

(expires Sat, 15 Oct 2022 14:15:54 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

github-actions[bot] commented 2 years ago

:rocket: Deploy protocol app on https://apps-protocol-af0f6--pr209-208-notify-users-wit-2m5pd2q5.web.app

github-actions[bot] commented 2 years ago

:rocket: Deploy governance app on https://apps-staking--pr209-208-notify-users-wit-3pi7cn5d.web.app

dimsome commented 2 years ago

Screen Shot 2022-09-13 at 23 10 19

For me, it shows like this?

toniocodo commented 2 years ago

Screen Shot 2022-09-13 at 23 10 19

For me, it shows like this?

I guess you test with impersonator? If yes, you just need a page refresh to update the context after the impersonator login/user update.

toniocodo commented 2 years ago

@dimsome I updated the code so you won't have to refresh with impersonator, it will update accordingly

toniocodo commented 2 years ago

what do you think of keeping the label short so it does not wrap? image

dimsome commented 2 years ago

Legacy Contracts ?

Does wrap as well, but might give it more space to squeeze it in?

dimsome commented 2 years ago

Work mostly fine, a few points:

Also could we use the primary button? or is there is reason we shouldn't? Screen Shot 2022-09-15 at 15 57 23