Skip to content

Conversation

simonovic86
Copy link
Contributor

@simonovic86 simonovic86 commented Sep 2, 2020

  • fix decode
  • built IPFS components from code in order to test

@simonovic86 simonovic86 added this to the Sprint 45 milestone Sep 2, 2020
@simonovic86 simonovic86 requested review from PaulLeCam and oed September 2, 2020 15:41
@simonovic86 simonovic86 self-assigned this Sep 2, 2020
Copy link
Collaborator

@oed oed left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You know that you can push changes to the dependabot PRs right? :)

@simonovic86 simonovic86 changed the title Fix after upgrading to CID v1 CID v1 upgrade Sep 2, 2020
Comment on lines +48 to +49
"ipfs": "github:simonovic86/ipfs",
"ipfs-http-client": "github:simonovic86/ipfs-http-client"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What's going on here?

Copy link
Contributor Author

@simonovic86 simonovic86 Sep 3, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The createDagJWS method that you provided in js-did library uses, amongst other things, CID v1 and ipld-dag-cbor v0.17.0 which is not compatible with one used in the latest IPFS release. That's why I forked the js-ipfs and tested it with the latest code. Because of this incompatibility, we're currently blocked with the integration.

@simonovic86
Copy link
Contributor Author

IPFS team has released v5.0.0! closing this PR :)

@simonovic86 simonovic86 closed this Sep 4, 2020
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.

2 participants