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

docs: fix js dependencies in webpack example #1027

Merged
merged 7 commits into from
Oct 22, 2017

Conversation

dfguo
Copy link
Contributor

@dfguo dfguo commented Sep 22, 2017

example won't run unless these modules are installed.

@@ -11,13 +11,17 @@
"babel-core": "^6.24.1",
"babel-loader": "^7.0.0",
"babel-preset-react": "^6.24.1",
"babel-preset-stage-0": "^6.24.1",
Copy link
Member

Choose a reason for hiding this comment

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

Wha does the babel-preset-stage-0 do and why do we need it here?

Copy link
Contributor Author

@dfguo dfguo Sep 23, 2017

Choose a reason for hiding this comment

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

I'm not sure why it was included in the .babelrc https://github.com/ipfs/js-ipfs/blob/master/examples/browser-webpack/.babelrc

We can perhaps take that away if that wasn't needed too.

"react-hot-loader": "^1.3.1",
"webpack": "^2.5.1",
"webpack-dev-server": "^2.4.5"
},
"dependencies": {
"ipfs": "^0.26.0",
Copy link
Member

Choose a reason for hiding this comment

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

We still want to use IPFS from this repo - it needs to be something like:

"dependencies": {
   "ipfs": "../../../src/core",
   ....
}

Copy link
Contributor

Choose a reason for hiding this comment

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

This one should be correct

"dependencies": {
   "ipfs": "file:../../",
}

Copy link
Member

Choose a reason for hiding this comment

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

Oh, nice! Much better

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.

Needs remaining comments to be addressed.

@@ -2,8 +2,8 @@

const React = require('react')
const Buffer = require('safe-buffer').Buffer
const IPFS = require('../../../../src/core') // replace this by line below
// const IPFS = require('ipfs')
// const IPFS = require('../../../../src/core') // replace this by line below
Copy link
Member

Choose a reason for hiding this comment

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

You can remove this line now.

@daviddias daviddias added the status/ready Ready to be worked label Oct 13, 2017
@daviddias
Copy link
Member

@dryajov can you investigate and test if these changes are necessary? I've already run the examples just fine.

@dryajov
Copy link
Member

dryajov commented Oct 18, 2017

looking into it!

daviddias
daviddias previously approved these changes Oct 22, 2017
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.

applied my CR :)

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