search

dwyl / learn-redux

:boom: Comprehensive Notes for Learning (how to use) Redux to manage state in your Web/Mobile (React.js) Apps.
GNU General Public License v2.0
446 stars 42 forks source link

notes edits #29

Closed manonthemat closed 8 years ago

manonthemat commented 8 years ago

just markdown changes in the notes for lesson 26...

biggest change is that <Provider> is visible in the title as such

iteles commented 8 years ago

@manonthemat Thank you for your contributions, we really appreciate them and so does our community!

In future, please assign your PRs to someone (usually @nelsonic or myself) for review - we like to ensure that all contributions are reviewed and aim to never merge our own PRs into master at dwyl 👍🏻

Thanks for reminding me that we need some serious updating to https://github.com/dwyl/contributing/blob/master/README.md 😊👌🏻

manonthemat commented 8 years ago

Got it and I agree that another pair of eyes is always a good idea :+1:

In this particular case I didn't want anybody to have to go through all these lines, as the benefit of the PR is minimal.