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

fixed json-loader error #784

Merged
merged 4 commits into from
Jun 6, 2018
Merged

fixed json-loader error #784

merged 4 commits into from
Jun 6, 2018

Conversation

leekt
Copy link
Contributor

@leekt leekt commented Jun 6, 2018

while checking out upload-file-via-browser example,
i encountered "cannot resolve module json-loader" error
it was simply solved by updating webpack from 1.9.6-> 2.0.0
since json works on default since webpack 2.0.0 https://github.com/webpack-contrib/json-loader
react-hot & babel loader had to be specified( add '-loader')

@alanshaw
Copy link
Contributor

alanshaw commented Jun 6, 2018

Thanks @leekt216! Could you please remove the DS_Store files from your PR and also remove the json-loader from package.json dependencies since we don't need it anymore. Then we can merge!

@leekt
Copy link
Contributor Author

leekt commented Jun 6, 2018

Thanks @alanshaw i removed DS_Store and json-loader

@alanshaw alanshaw merged commit 5e7b7c4 into ipfs-inactive:master Jun 6, 2018
danieldaf pushed a commit to danieldaf/js-ipfs-api that referenced this pull request Jul 21, 2018
…ive#784)

* fixed json-loader error

* add DS_Store

* removed DS_Store

* removed json-loader
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants