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

Add Dev tools #70

Merged
merged 3 commits into from
Dec 4, 2015
Merged

Add Dev tools #70

merged 3 commits into from
Dec 4, 2015

Conversation

geowarin
Copy link
Contributor

@geowarin geowarin commented Dec 3, 2015

This pull request add dev-tools to the basic example.
I hope this will help to fix #61.

@kimjoar kimjoar mentioned this pull request Dec 4, 2015
@kimjoar
Copy link
Collaborator

kimjoar commented Dec 4, 2015

@geowarin I created another PR based on this one that attempts to solve #61. Check it out here: #73

@@ -21,7 +21,7 @@ module.exports = {
// This will make the redux-simpler-router module resolve to the
// latest src instead of using it from npm. Remove this if running
// outside of the source.
var src = path.join(__dirname, '..', '..', 'src')
var src = path.join(__dirname, '..', '..', 'lib')
Copy link
Member

Choose a reason for hiding this comment

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

Please remove this change. We want to pull the latest source, not the latest built source. We are already running it through webpack and it'll build it again.

@jlongster
Copy link
Member

Looks great, thanks, I needed to do this soon but this helps a lot. @kjbekkelund I may go ahead and merge this so we can keep discussing solutions.

@geowarin
Copy link
Contributor Author

geowarin commented Dec 4, 2015

Hi @jlongster, I amended my branch according to your remarks.
@kjbekkelund, sorry I rebased and push forced so you might need to repair your branch 😖

@kimjoar
Copy link
Collaborator

kimjoar commented Dec 4, 2015

@jlongster Absolutely — this should be merged! :)

@geowarin No prob, my branch was just to get a discussion going on how to solve the devtools problem :)

@jlongster
Copy link
Member

Ok I'll go ahead and merge it and look into it today.

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.

0.0.10 breaks devtools
3 participants