Skip to content
This repository has been archived by the owner on Mar 10, 2020. It is now read-only.

fix: updates ipld-dag-pb dep to version without .cid or .multihash properties #25

Merged
merged 1 commit into from
Nov 12, 2018

Conversation

achingbrain
Copy link
Collaborator

Follows on from ipld/js-ipld-dag-pb#99 and updates this module to not rely on DAGNodes having knowledge of their CIDs.

Bonus: also fixes #24 by removing use of js-ipfs from this module breaking another circular dependency from the project.

License: MIT
Signed-off-by: achingbrain alex@achingbrain.net

@ghost ghost assigned achingbrain Nov 8, 2018
@ghost ghost added the in progress label Nov 8, 2018
@achingbrain
Copy link
Collaborator Author

N.b. this PR uses a github url for the ipld-dag-pb dependency temporarily and will need a new release of js-ipld that dependes on ipld-dab-pb with ipld/js-ipld-dag-pb#99 merged

@achingbrain achingbrain force-pushed the update-dag-pb-to-not-have-cid-property branch 5 times, most recently from e2919db to 418ddfe Compare November 8, 2018 09:56
@achingbrain achingbrain changed the title fix: updates ipld-dag-pb dep to version without .cid properties fix: updates ipld-dag-pb dep to version without .cid or .multihash properties Nov 9, 2018
Follows on from ipld/js-ipld-dag-pb#99 and updates this module to not
rely on DAGNodes having knowledge of their CIDs.

Bonus: also fixes #24 by removing use of js-ipfs from this module
breaking another circular dependency from the project.

License: MIT
Signed-off-by: achingbrain <alex@achingbrain.net>
@achingbrain achingbrain force-pushed the update-dag-pb-to-not-have-cid-property branch from 418ddfe to fa9029d Compare November 12, 2018 09:53
@achingbrain achingbrain merged commit ea8ba91 into master Nov 12, 2018
@achingbrain achingbrain deleted the update-dag-pb-to-not-have-cid-property branch November 12, 2018 10:03
@ghost ghost removed the in progress label Nov 12, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove dependency on IPFS
1 participant