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

feat: DAG API basics (get, put, rm) #112

Merged
merged 5 commits into from
Feb 1, 2017
Merged

Conversation

daviddias
Copy link
Contributor

@daviddias daviddias commented Jan 31, 2017

}
})

describe('.put', () => {
Copy link
Contributor

Choose a reason for hiding this comment

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

You should also get these nodes to check that they have been actually put in the correct way

Copy link
Contributor Author

Choose a reason for hiding this comment

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

that is what happens in the get tests, this way put is tested in isolation and in conjunction with get

Copy link
Contributor

Choose a reason for hiding this comment

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

But not for all the formats as far as I can see.

@daviddias
Copy link
Contributor Author

I'm divided in waiting for feedback on DAG API things and moving forward, I've been kind of waiting for more throughout feedback since last quarter and now I've so many 'little' things in my plate that it is impossible to achieve something during two weeks if I wait for a lot of RTT

I'll just move fast for the dag.get dag.put and dag.resolve API in js-ipfs land and bring up what I need to happen in go-ipfs land to make it go through js-ipfs-api

@whyrusleeping @nicola

@daviddias daviddias merged commit c8774cd into master Feb 1, 2017
@daviddias daviddias deleted the feat/dag-api-basics branch February 1, 2017 10:34
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.

2 participants