-
Notifications
You must be signed in to change notification settings - Fork 12
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
feat: user profile page #394
Conversation
@bonustrack WHen sharing the profile, what should the message be ? |
@wa0x6e It should be only the link |
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.
tACK
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.
There is issue with message flashing when opening profiles of people that didn't create their user profile at the API yet.
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.
tACK
Summary
Closes: #387
This feature introduce the "User" concept for real, with an usable User profile page, as well as editable profile details.
Changes
UiInputStampCover
component has been refactored to also support user coverHow to test
TODO
Support starknet user on offchain(will be added in another PR)