-
Notifications
You must be signed in to change notification settings - Fork 103
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
Include tags from kind 0 in NDKUserProfile #155
Comments
Curious @verbiricha, would you think it'd be better to include |
I think is best to include the tags, then an app can do replacement of emoji shortcodes into tiny images and apps that don't support that will show shortcodes.
The tags on kind 0 could be also `t` for tagging a profile, or `g` for specifying your location. Don't know if any client does this but they can so I think including the raw tags is better.
…-------- Mensaje original --------
El 15 dic 2023 17:39, JeffG escribió:
Curious ***@***.***(https://github.com/verbiricha), would you think it'd be better to include tags as an object on NDKUserProfile object or do you think NDK should just go ahead and replace the emoji content in the name and about fields directly?
—
Reply to this email directly, [view it on GitHub](#155 (comment)), or [unsubscribe](https://github.com/notifications/unsubscribe-auth/AWAGFU73O35ZBRCKHZBF2KDYJR4MTAVCNFSM6AAAAABAWKFZU6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNJYGE3DENZZHA).
You are receiving this because you were mentioned.Message ID: ***@***.***>
|
@pablof7z thoughts on this one? We'd need to add tags for user profiles into all the cache adapters as well as change the profile to process tags. |
@pablof7z bump – curious your thoughts on this. |
Would be nice to include tags from the profile event in
NDKUserProfile
. Certain profiles use custom emojis for username/bio as per NIP-30.The text was updated successfully, but these errors were encountered: