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

feat: use dweb.link as the preview url #7

Merged
merged 1 commit into from
Dec 6, 2019
Merged

Conversation

olizilla
Copy link
Member

@olizilla olizilla commented Dec 6, 2019

use base32 cid v1 flavour and use dweb.link as the preview domain.

see: ipfs-inactive/docs#405

License: MIT
Signed-off-by: Oli Evans oli@tableflip.io

use base32 cid v1 flavour and use dweb.link as the preview domain.

see: ipfs-inactive/docs#405

License: MIT
Signed-off-by: Oli Evans <oli@tableflip.io>
@cwaring
Copy link
Member

cwaring commented Dec 6, 2019

wild! that was much easier than I was expecting ❤️🙏🏻

@olizilla olizilla merged commit da65693 into master Dec 6, 2019
@olizilla olizilla deleted the use-dweb.link branch December 6, 2019 17:36
@olizilla
Copy link
Member Author

olizilla commented Dec 6, 2019

@cwaring it is built and deployed to docker. I think our websites all use the lastest verion, so the next PRs on docs / ipfs.io / blog should have dweb.link preview urls.

olizilla added a commit to ipfs-inactive/blog that referenced this pull request Dec 6, 2019
More to do, but this is the gist.

also wanted to test out the new dweb.link preview urls
see: ipfs-shipyard/ipfs-dns-deploy#7

License: MIT
Signed-off-by: Oli Evans <oli@tableflip.io>
@olizilla
Copy link
Member Author

olizilla commented Dec 6, 2019

It lives!

Screenshot 2019-12-06 at 17 54 06

ipfs-inactive/blog#337

@lidel
Copy link
Member

lidel commented Dec 7, 2019

Honorary CC: https://github.com/ipfs/ipfs/issues/337 🥇

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants