Added visualisations for electricity emissions

Use npm run build to bundle the react components and then open localhost:3080. #14 (closed)

changed the description

A lot of unrequired files have also been deleted.

Thanks for the MR! Could you also post some screenshots here?

We are still in the process of adding proper titles to the visualisations. Will push that in the next MR.

changed the description

Can we merge this ? :)

I am getting the following error after installing many packages that didn't get installed on npm install

npm run build

> carbonfootprint@0.0.0 build /Users/Chirag/git-clients/CarbonFootprint-API
> webpack

Hash: b33bfe389cfcfab3d54a
Version: webpack 3.0.0
Time: 11181ms
    Asset     Size  Chunks                    Chunk Names
bundle.js  2.39 MB       0  [emitted]  [big]  main
 [208] (webpack)/buildin/module.js 517 bytes {0} [built]
 [468] ./client/src/routes.js 606 bytes {0} [built]
 [584] ./client/src/index.js 2.82 kB {0} [built]
 [625] (webpack)/buildin/global.js 509 bytes {0} [built]
    + 901 hidden modules

ERROR in ./node_modules/history/node_modules/query-string/index.js
Module not found: Error: Can't resolve 'strict-uri-encode' in '/Users/Chirag/git-clients/CarbonFootprint-API/node_modules/history/node_modules/query-string'
 @ ./node_modules/history/node_modules/query-string/index.js 2:22-50
 @ ./node_modules/history/lib/useQueries.js
 @ ./node_modules/react-router/lib/Router.js
 @ ./node_modules/react-router/lib/index.js
 @ ./client/src/routes.js
npm ERR! code ELIFECYCLE
npm ERR! errno 2
npm ERR! carbonfootprint@0.0.0 build: `webpack`
npm ERR! Exit status 2
npm ERR!
npm ERR! Failed at the carbonfootprint@0.0.0 build script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR! A complete log of this run can be found in:
npm ERR!     /Users/Chirag/.npm/_logs/2017-07-10T23_56_35_389Z-debug.log

@heychirag can you try deleting the node_modules folder and run npm install again. Try yarn install if the error still persists.

This looks amazing! Just indicate somewhere that the emissions are for electricity.

@heychirag , did you manage to solve the installation/building issues that you described above?

@bruno-wp Yes. yarn install did the trick.

@heychirag We have implemented the chart title in the next MR.

@r0hitgupta could you update the readme? Currently it says either npm or yarn could be used, but it seems (based on the issue faced by @heychirag) that npm does not really work. Maybe our readme should just say that yarn must be used.

@bruno-wp Will add that to the readme and update the mr.

added 1 commit

• 33f7ff0c - updated README.md for installation using yarn.

Compare with previous version

mentioned in merge request !15 (merged)

mentioned in issue #19 (closed)

merged