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

Wrap with dir #730

Merged
merged 3 commits into from
Apr 5, 2018
Merged

Wrap with dir #730

merged 3 commits into from
Apr 5, 2018

Conversation

daviddias
Copy link
Contributor

from @wraithgar. just adding latest interface-ipfs-core

@ghost ghost assigned daviddias Apr 3, 2018
@ghost ghost added the in progress label Apr 3, 2018
@daviddias
Copy link
Contributor Author

@wraithgar seems that tests on wrap-with-directory are not passing here:

image

@wraithgar
Copy link
Contributor

That test can't pass without the code in ipfs/js-ipfs#1290

@daviddias
Copy link
Contributor Author

@wraithgar the js-ipfs-api runs its tests against go-ipfs. It doesn't use js-ipfs at all. It means that the client library is not interacting with go-ipfs properly which might mean there is some mismatch on how the http endpoint in js-ipfs is working from go-ipfs.

@wraithgar
Copy link
Contributor

Ah ok I see. The test that's failing is the new one added in interface-ipfs-core which passes the new wrapWithDirectory parameter. So a go equivalent to that js-ipfs PR needs to exist first then right?

@daviddias
Copy link
Contributor Author

This feature already exists on go-ipfs, it seems that the param for the endpoint was just not implemented in the same way.

image

See also the http-api docs https://ipfs.io/docs/api/#apiv0add

@daviddias
Copy link
Contributor Author

@wraithgar this is currently blocking me from release js-ipfs-api given that interface-ipfs-core shipped with the test before this was complete (next time we need to run all the tests and make sure it all passes).

I'm just going to tackle it quick to unblock other work.

@daviddias
Copy link
Contributor Author

it turned out that the test was the thing incorrect ipfs-inactive/interface-js-ipfs-core@a97c087

@daviddias daviddias merged commit 160860e into master Apr 5, 2018
@daviddias daviddias deleted the wrap-with-dir branch April 5, 2018 15:31
@ghost ghost removed the in progress label Apr 5, 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.

2 participants