-
Notifications
You must be signed in to change notification settings - Fork 396
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Redux with stub data (fixes #8) #75
Conversation
const store = createStore(combineReducers({addons, search})); | ||
|
||
const App = () => ( | ||
<Provider store={store}> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This redux stuff normally seems to go in an index.js file but we want to split it out per app. I'm not sure if this is the best place for it but apparently it works.
280a75f
to
0157809
Compare
bar: {slug: 'bar', title: 'The bar add-on'}, | ||
}; | ||
|
||
export default function addon(state = initialState, action) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would be nice to tweak the style settings so we don have spaces around default args e.g. so we have state=initialState
instead of state = initialState
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, I found this a little weird at first. I don't totally hate it but not what I went with first.
There's conflicts that need addressing but this is looking good. r+wc |
I don't see coveralls posting the status anymore, maybe it only does that when the coverage goes down? |
Redux with stub data (fixes #8)
This probably still needs some cleanup. Fixes mozilla/addons#5583.