Skip to content
This repository has been archived by the owner on Feb 12, 2024. It is now read-only.

feat: add partial implementation for ipfs.resolve #1455

Merged
merged 10 commits into from
Aug 22, 2018
Merged

Conversation

alanshaw
Copy link
Member

@alanshaw alanshaw commented Jul 19, 2018

Adds a partial implementation for ipfs.resolve - we can resolve:

/ipfs/QmRootHash/path/to/file to /ipfs/QmFileHash

Resolving IPNS paths will callback with a not implemented error.

Allows the following tests from ipfs-inactive/interface-js-ipfs-core#332 to pass:

screen shot 2018-07-19 at 22 35 36

depends on ipfs-inactive/interface-js-ipfs-core#332, ipfs-inactive/js-ipfs-http-client#794, ipld/js-ipld-dag-pb#78 and ipfs/js-ipfsd-ctl#285

@ghost ghost assigned alanshaw Jul 19, 2018
@ghost ghost added the status/in-progress In progress label Jul 19, 2018
@alanshaw alanshaw changed the title [WIP] feat: add partial implementation for ipfs.resolve feat: add partial implementation for ipfs.resolve Jul 23, 2018
@alanshaw alanshaw requested a review from daviddias July 23, 2018 12:38
@alanshaw alanshaw force-pushed the feat/resolve branch 2 times, most recently from 7f3c599 to 7273bff Compare August 2, 2018 18:01
Copy link
Member

@daviddias daviddias left a comment

Choose a reason for hiding this comment

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

Code LGTM. Please make sure that CI is green before merging

@alanshaw alanshaw mentioned this pull request Aug 9, 2018
22 tasks
@alanshaw alanshaw force-pushed the feat/resolve branch 2 times, most recently from 2d43474 to a023218 Compare August 15, 2018 15:24
License: MIT
Signed-off-by: Alan Shaw <alan@tableflip.io>
License: MIT
Signed-off-by: Alan Shaw <alan@tableflip.io>
License: MIT
Signed-off-by: Alan Shaw <alan@tableflip.io>
License: MIT
Signed-off-by: Alan Shaw <alan@tableflip.io>
License: MIT
Signed-off-by: Alan Shaw <alan@tableflip.io>
License: MIT
Signed-off-by: Alan Shaw <alan@tableflip.io>
License: MIT
Signed-off-by: Alan Shaw <alan@tableflip.io>
License: MIT
Signed-off-by: Alan Shaw <alan@tableflip.io>
License: MIT
Signed-off-by: Alan Shaw <alan@tableflip.io>
@daviddias daviddias merged commit 4df45d1 into master Aug 22, 2018
@daviddias daviddias deleted the feat/resolve branch August 22, 2018 07:05
@ghost ghost removed the status/in-progress In progress label Aug 22, 2018
@alanshaw
Copy link
Member Author

ah, I was pausing on merging these 0.32 PRs until I was able to merge #1523 and do another 0.31 patch release...

🙄 😆

@daviddias
Copy link
Member

Oh, sorry about that. I didn't see a breaking change note :(

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