Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merge webui publish fix + bump ipfs-api version #75

Merged
merged 5 commits into from
Aug 19, 2015
Merged

Merge webui publish fix + bump ipfs-api version #75

merged 5 commits into from
Aug 19, 2015

Conversation

daviddias
Copy link
Member

No description provided.

@jbenet jbenet added the status/in-progress In progress label Aug 13, 2015
@daviddias
Copy link
Member Author

@krl can you merge this and push the new webui to go-ipfs?

This also fixes the upload bug on electron-app console mode

//cc @jbenet @whyrusleeping

@whyrusleeping
Copy link
Member

👍 🎆 😄 👍 wooo!

"dependencies": {
"bootstrap": "3.3.4",
"d3": "3.5.5",
"debug": "2.1.3",
"font-awesome": "4.3.0",
"ipfs-api": "1.1.4",
"ipfs-api": "2.1.5",
Copy link
Member

Choose a reason for hiding this comment

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

might want to bump to 2.1.6 if ipfs-inactive/js-ipfs-http-client#45 merges first

@jbenet
Copy link
Member

jbenet commented Aug 14, 2015

LGTM. @diasdavid i'm good to ship this.

the tests... arent testing.

@daviddias
Copy link
Member Author

@jbenet done. Tests fail because the repo doesn't have any tests :(

jbenet added a commit that referenced this pull request Aug 19, 2015
Merge webui publish fix + bump ipfs-api version
@jbenet jbenet merged commit 243b076 into master Aug 19, 2015
@jbenet jbenet removed the status/in-progress In progress label Aug 19, 2015
@jbenet jbenet deleted the build-fix branch August 19, 2015 03:28
@daviddias daviddias mentioned this pull request Aug 26, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants