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

feat(files.add): conform with latest interface-ipfs-core spec #345

Merged
merged 2 commits into from
Aug 12, 2016

Conversation

daviddias
Copy link
Contributor

@daviddias daviddias commented Aug 12, 2016

CI will be happy after ipfs-inactive/interface-js-ipfs-core#56 is merged and released

@dignifiedquire
Copy link
Contributor

  • sad CI is sad, code lgtm
  • CI doesn't test node@6 could you add that to travis please?

const mh = bs58.encode(res[0].node.multihash()).toString()
expect(mh).to.equal('Qma4hjFTnCasJ8PVp3mZbZK5g2vGDT4LByLJ7m8ciyRFZP')
expect(res[0].path).to.equal(mh)
expect(res[0].node.links).to.have.length(0)
Copy link
Contributor

Choose a reason for hiding this comment

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

No longer checking # links?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Because we don't return the full dag node anymore

@daviddias daviddias merged commit 3fc7042 into master Aug 12, 2016
@daviddias daviddias deleted the feat/update-files-add branch August 12, 2016 23:52
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.

3 participants