Skip to content
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

Can't decode emoji CIDs #44

Closed
olizilla opened this issue Jul 21, 2022 · 3 comments Β· Fixed by #45
Closed

Can't decode emoji CIDs #44

olizilla opened this issue Jul 21, 2022 · 3 comments Β· Fixed by #45
Assignees
Labels
bug Something isn't working help wanted Extra attention is needed P1 - High

Comments

@olizilla
Copy link
Collaborator

olizilla commented Jul 21, 2022

Similar problem as #43. IPFS now supports emoji CIDs adn cid.ipfs.io does not.

example from @ribasushi
https://cid.ipfs.io/#πŸš€πŸͺβ­πŸ’»πŸ˜…β“πŸ’ŽπŸŒˆπŸŒΈπŸŒšπŸ’°πŸ’πŸŒ’πŸ˜΅πŸΆπŸ’πŸ€πŸŒŽπŸ‘ΌπŸ™ƒπŸ™…β˜ΊπŸŒšπŸ˜žπŸ€€β­πŸš€πŸ˜ƒβœˆπŸŒ•πŸ˜šπŸ»πŸ’œπŸ·βš½βœŒπŸ˜Š

The main blocker here is that to fix it the guts of the site will have to be rewritten to use the new multiformats lib

@lidel lidel added P1 - High bug Something isn't working labels Jul 25, 2022
@lidel
Copy link
Member

lidel commented Jul 25, 2022

@hacdias would you be willing (or have time) to pick up this quest? Emoji support is critical feature of our entire stack ;)

@lidel lidel added the help wanted Extra attention is needed label Jul 25, 2022
@hacdias
Copy link
Member

hacdias commented Jul 26, 2022

The website seems relatively simple so I'll be glad to take this one on. Good opportunity to check how the new multiformats lib works.

@hacdias hacdias self-assigned this Jul 26, 2022
@hacdias hacdias mentioned this issue Jul 26, 2022
6 tasks
@lidel lidel closed this as completed in #45 Aug 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working help wanted Extra attention is needed P1 - High
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants