-
Notifications
You must be signed in to change notification settings - Fork 34
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
Adjust readme #54
Adjust readme #54
Conversation
sgnilreutr
commented
Nov 22, 2021
- Move Abstract section into Main Goal
- Add Table of Contents
- Structure document with correct headings
- Textual changes, grammar.
- Add Saga + Thunk examples
Add saga and redux thunk examples
Add table of contents
Adjust indent
Textual changes
Hi, @sgnilreutr thanks for the great work! I'll review it soon! |
@@ -151,12 +180,14 @@ const App = () => ( | |||
); | |||
|
|||
export default App; | |||
|
|||
//...History can now be used via other components using history.push() etc. | |||
``` |
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.
What's the goal of this comment?
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.
I wanted to emphasise that by implementing redux first history the behaviour of a function such as history.push or navigate() remains the same.
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.
@salvoravida Is there anything else you'd like to have changed before it can be merged?
Thank you! |