-
Notifications
You must be signed in to change notification settings - Fork 16
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: use r/demo/profile
for gno user info
#1250
Conversation
✅ Deploy Preview for testitori ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for teritori-dapp ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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.
I think we shouldn't add a new hook and instead adapt the existing
- useNSPrimaryAlias (returns r/demo/users name)
- useNSUserInfo (returns relevant fields in r/demo/profiles)
without changing interfaces.
This way, everything we did before will be automatically supported
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.
this is much better, thanks :)
one last thing:
we should try to keep changes minimal (remove dev artifacts)
this makes the changes more atomic, make reviews easier and prevents including unwanted changes
thus can you revert the unneeded changes in
- packages/components/images/AvatarWithFrame.tsx
- packages/screens/UserPublicProfile/components/UPPIntro.tsx
pls
Signed-off-by: Norman Meier <norman@samourai.coop>
r/demo/profile
for gno user info
Signed-off-by: Norman Meier <norman@samourai.coop>
Closes #1244
Use Gno r/demo/profile for UPP page. Here are some results: