-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add username edit feature #59
Add username edit feature #59
Conversation
@MattPereira is attempting to deploy a commit to the Deez Stealth Team on Vercel. A member of the Team first needs to authorize it. |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@MattPereira all lgtm, but refetch doesn't work. I have to refresh the page to see new name. And looks like there is some bug with "Loading" state. I guess it should be Loading once I click on submit icon and while tx is mining. But instead I've seen it only once when I cancelled one of my transactions: Loading... just stuck there after.
Also I think we should consult Stan about icons
@MattPereira i checked it all via deploy for this PR: https://updev-pj96crwwd-deez-stealth.vercel.app/profile/0x7885f82e19e5950129cB78356C56DD571E792508 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
great job @MattPereira !
Fixes #60
Summary
ProfileDetails
section of profile page into its own componentgetData
andsetData
functions of UP contractonClick
that triggers edit modesetData
transaction to UP contract on mumbaiSee screenshots of changes