-
Notifications
You must be signed in to change notification settings - Fork 15
Conversation
rename list to counterIds
|
||
const initialState = { counterIds: ['Counter one', 'Counter two'] } | ||
|
||
export const pageReducer = (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.
move to the reducers
folder?
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.
Also not sure on the page
name. counters
?
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.
moved into reducers/index.js
un-defaulted previous reducer and renamed to counterInstanceReducer
renamed state slice from page
to counters
renamed id array value from counterIds
to allIds
} | ||
} | ||
|
||
class DynaCounter extends React.Component { |
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.
Move to the compnents
folder. Could add a link to it directly in the README.
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.
moved and linked
Great work! Don't forget to add yourself as a contributor using the "Examples" option. |
Adds another example, based on redux-saga/cancellable-counters.
This example demonstrates a dynamic list of dynostore dynamic components.