GoodDollar / GoodDAPP

GoodDollar.org Wallet is the simplest access point to Claim your daily G$. It Is based on web3 and React native web.
good-dapp.vercel.app
MIT License
100 stars 53 forks source link

[Feature] Profile Improvements #4162

Closed patpedrosa closed 7 months ago

patpedrosa commented 8 months ago

We need to do some small changes to the profile to improve UX.

Figma for all changes above: https://www.figma.com/file/ihw1PxBvLxacTHnN2aj4lC/3.-Product-%F0%9F%92%B5?type=design&node-id=9799-42332&mode=design&t=uIGGsZjdGb7TN3HW-0

vldkhh commented 8 months ago

@L03TJ3 @patpedrosa Do we want to keep this "note" on the "My Profile" page and on the "Edit profile" page? According to the design the note should be displayed on the Edit profile page

Snag_171ef05a.png
vldkhh commented 8 months ago

NOTE: Also, to see the login method you should logout, do a hard reset and clear the cache, then login (do it 2-4 times to see that)

L03TJ3 commented 8 months ago

@patpedrosa @decentralauren I forgot to share this..

Basically we never really stored anywhere someone's used login-method. it was just saved as 'torus' (which says nothing to the user). So in order for the login-method to show I needed to add a new flow during login/sign-up where this get stored/saved.

Meaning, that login-method would not show anything for an already logged-in user up until they re-login (I will look into vlad's reported bug)

How to handle this? Should we just hide the whole block when we don't know the login-method (yet) ?

patpedrosa commented 8 months ago

@L03TJ3 Yes, that's a good solution. Better than showing blank or torus.

sirpy commented 8 months ago

we are saving the login method in our user database via the backend (goodserver)

L03TJ3 commented 8 months ago

reading from database is made a separate issue here: https://github.com/GoodDollar/GoodDAPP/issues/4169 Rejected points from UAT:

L03TJ3 commented 8 months ago

@vldkhh this can be retested on dev

  1. it should now show a verify button for the phone/email fields
  2. save button (next to avatar message) only shown when full-name is editted
vldkhh commented 8 months ago

@L03TJ3 for me it works as expected, good job! @decentralauren you can try on https://dev.gooddollar.org/

decentralauren commented 8 months ago

@L03TJ3 @vldkhh I keep getting stuck at the OTP (it keeps timing out and not sending). Happy to test once pushed to prod if it works well for everyone else!

Yay :)

decentralauren commented 8 months ago

OK I found 3 issues:

(I have slacked a video describing these items to @vldkhh and @L03TJ3 as it has PII)

L03TJ3 commented 7 months ago

@decentralauren above points got discussed or fixed and is already released. so is this feature review done and we can move this ticket?

decentralauren commented 7 months ago

@L03TJ3 Yes! Please close this ticket and thanks for following up.