{"push_id":2392783321,"size":1,"distinct_size":1,"ref":"refs/heads/master","head":"1b86546ddb38931e941bf77ddb40a4c8376bc933","before":"9bf1c6f74f9fbbee6504c6c65175587be02296d8","commits":[{"sha":"1b86546ddb38931e941bf77ddb40a4c8376bc933","author":{"name":"Kevin Lau","email":"eac79e57f53165ceab4a5447ccc12495a2cd9233@gmail.com"},"message":"Fix tutorial code to match text\n\nSpread the previous state into the next state returned by the reducer.\r\nEven though this redux state only has one property, `counter`, keeping\r\nthe other state is good to do in general. Also, this keeps the code\r\nconsistent. Currently, the code sometimes grabs the whole `state` and\r\nother times destructures just `{ counter }` from the `state`. This fix\r\nalways grabs the whole `state` to be able to spread it.\r\n\r\nThe current code for `createActions()` for `'INCREMENT'` and\r\n`'DECREMENT'` that accept an argument always returns a `payload`\r\nof `{ amount: 1 }` and `{ amount: -1 }` respectively. I am assuming the\r\nintent is to default to 1 or -1. This fix uses a default parameter\r\nand matches the example in README.md.\r\n\r\nThe `+ amount` was changed into `- amount` for the `'DECREMENT'` reducer\r\nin a previous commit. `+ amount` is correct because `amount` is negative\r\nfor the `'DECREMENT'` action. It also makes sense like this because\r\nbelow, `combineActions()` is used meaning the `'INCREMENT'` and\r\n`'DECREMENT'` reducers should be the same.","distinct":true,"url":"https://api.github.com/repos/dosentmatter/redux-actions/commits/1b86546ddb38931e941bf77ddb40a4c8376bc933"}]}
{"actor":{"display_login":"dosentmatter"}}