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

Add Arweave Codec #225

Merged
merged 1 commit into from
Aug 26, 2021
Merged

Add Arweave Codec #225

merged 1 commit into from
Aug 26, 2021

Conversation

rekpero
Copy link
Contributor

@rekpero rekpero commented Aug 25, 2021

This PR adds an Arweave codec to ultimately add ENS support for Arweave Hashes.
Arweave is a decentralized CDN network that permanently stores any file on its network, find out more here.

@rekpero
Copy link
Contributor Author

rekpero commented Aug 26, 2021

@vmx @Stebalien @rvagg Sorry for directly tagging you guys here, but would it be possible for anyone of you guys to take a look at this PR? We are trying to add ENS content hash support for Arweave Protocol so that I can integrate ENS into ArGoApp, and having a codec is a requirement.

@rvagg
Copy link
Member

rvagg commented Aug 26, 2021

No problems @rekpero, I saw this but had to go fishing for someone who knows more about namespace entries than I. No objections have emerged and the skynet-ns entry suggests this is reasonable.

@rvagg rvagg merged commit 9493726 into multiformats:master Aug 26, 2021
@vmx
Copy link
Member

vmx commented Aug 27, 2021

It was the same for me. I've also no objections, but a also a lack of knowledge in that space. No worries about tagging me.

@rekpero
Copy link
Contributor Author

rekpero commented Aug 27, 2021

Thanks, @rvagg @vmx for merging this PR.

Just wanted to know when are you guys gonna release the new version of the lib so that we can test out our integration on the ENS side? Would be fine if you can give us a rough timeline for your release

@Abhimanyu121
Copy link

Hey, can I get a tentative release schedule for this PR?

@rvagg
Copy link
Member

rvagg commented Aug 27, 2021

which lib in particular? there's no library in this repo but there's a number of dependencies that pull from here, you may need to submit a PR to whatever library you're depending on

@rekpero
Copy link
Contributor Author

rekpero commented Aug 27, 2021

Thanks, @rvagg for the reply, I guess I just need to create a PR on js-multicodec with the updated table

@rekpero
Copy link
Contributor Author

rekpero commented Aug 27, 2021

@rvagg check this out as well multiformats/js-multicodec#81

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.

None yet

4 participants