Skip to content
This repository has been archived by the owner on Jan 26, 2023. It is now read-only.

Webpack + React Prototype #31

Open
wants to merge 11 commits into
base: stage
Choose a base branch
from
Open

Webpack + React Prototype #31

wants to merge 11 commits into from

Conversation

kamescg
Copy link
Contributor

@kamescg kamescg commented Jan 1, 2018

Webpack helps generate the perfect* bundle

*What's ever really perfect :)

The React family has been added (Redux, Router, Sagas) to help scale the interface.

As mentioned in a recent Tweet (https://twitter.com/GetGitcoin/status/947533717689450497) the future is bright for the Gitcoin ecosystem. As Gitcoin features become interconnected, hopefully predictable interface state management can help designers and developers more quickly and easily code new features.

Common async management (generators, await and promises) are implemented within the application bundle. Helping designers and developers more easily reason about Javascript's notorious single threaded callback style.

@owocki
Copy link
Contributor

owocki commented Jan 2, 2018

i get an error when i try to load this into chrome.. http://bits.owocki.com/2O1p2I033V0p/Screen%20Shot%202018-01-02%20at%209.14.31%20AM.png

i tried running these commands from the root chrome_ext folder

npm run build || yarn build <---- Bundles Chrome Extension
npm run start || yarn start <---- React Hot Loading for Popup Development (localhost: 3000)

but i got some npm errors. i suppose i need to npm install it first?

@kamescg
Copy link
Contributor Author

kamescg commented Jan 2, 2018

Sorry should have clarified :)

npm install
npm run build

Upload the build folder to Chrome.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
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.

None yet

2 participants