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

WIP Files #8

Closed
wants to merge 1 commit into from
Closed

WIP Files #8

wants to merge 1 commit into from

Conversation

nginnever
Copy link
Contributor

Beginning the files documentation. I haven't added the tests yet but plan to do so after finishing the HTTP-API for JS-IPFS. There are still some things that need to be done to reach complete feature parity with go-ipfs but this is a good place to put the documentation of where js-ipfs is in comparison.

Starting the gzip compression and tar archiving for jsipfs get command as mentioned in the readme update here.

- Path, String format path to a file to be added to IPFS
- Object, with format `{ path: </foo/bar>, stream: <data> }`
- Array, an [array] of objects with the above format.

Copy link
Contributor

Choose a reason for hiding this comment

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

there is also the option that no 1st arg is passed, a duplex stream is returned

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ahh good, added

@daviddias
Copy link
Contributor

This is s great start! Thank you @nginnever :D


- `Recursive`. Add directory paths recursively.

`callback` must follow `function (err, object) {}` signature, where `err` is an error if the operation was not successful and `object` is an ndjson return with properties { Name (string), Hash (multihash) }
Copy link
Contributor

Choose a reason for hiding this comment

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

What is "an ndjson return"?

@daviddias
Copy link
Contributor

daviddias commented May 20, 2016

There is a lot of great activity and we are really close of having add, cat and get in both js-ipfs and js-ipfs-api with the same API, which is great.

@noffle @nginnever, let's set this as a priority for the day, as we need ipfs cat to unlock @haadcode.

What is missing

@daviddias
Copy link
Contributor

@nginnever is this PR still up to date?

@nginnever
Copy link
Contributor Author

nginnever commented Jun 28, 2016

hey @diasdavid, no this is dated and should be closed.

get spec is the only thing missing but has moved from the PR's on this list.

@nginnever nginnever closed this Jun 28, 2016
@daviddias
Copy link
Contributor

@nginnever thank you for confirming :)

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.

4 participants