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

jsipfs block http-api and cli #81

Merged
merged 3 commits into from
Apr 2, 2016
Merged

jsipfs block http-api and cli #81

merged 3 commits into from
Apr 2, 2016

Conversation

RichardLitt
Copy link
Member

So, I stubbed out a couple of files, but I'm not sure this is what I am actually supposed to be doing here. It is hard without more examples, and I don't know how to run, or what sort of development environment I should be using to test the code as I go along.

@RichardLitt
Copy link
Member Author

Ping @diasdavid, of course. What should I be doing next?

@RichardLitt RichardLitt added help wanted kind/support A question or request for support labels Mar 10, 2016
@RichardLitt
Copy link
Member Author

Related to #50

@daviddias
Copy link
Member

@RichardLitt didn't the "Good foundations" section on #30 (comment) helped? Let's chat today to clear out any questions :)

@RichardLitt
Copy link
Member Author

Ah, that's interesting. I'll take a look through this again and see what I can do.

@daviddias
Copy link
Member

@RichardLitt do you still need some help here?

@daviddias
Copy link
Member

ping @RichardLitt

@fbaiodias fbaiodias added needs review and removed help wanted kind/support A question or request for support labels Apr 2, 2016
@fbaiodias
Copy link
Member

Just finished implementing this, ping @diasdavid


const mh = utils.isDaemonOn()
? key
: new Buffer(bs58.decode(key))
Copy link
Member

Choose a reason for hiding this comment

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

soon all these hacks will be gone (once we get core spec \o/)

@daviddias daviddias changed the title WIP Stubbed out resources and routes jsipfs block http-api and cli Apr 2, 2016
@daviddias
Copy link
Member

@xicombd sweet! Great work :D Only one comment :)

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