forked from reduxjs/redux
-
Notifications
You must be signed in to change notification settings - Fork 40
Update docs #26
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
Open
neighborhood999
wants to merge
182
commits into
new-doc-zh-tw
Choose a base branch
from
update-docs
base: new-doc-zh-tw
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Update docs #26
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Fixed wrong unsubscribe example
Added word for better sentence flow.
from ”headlines for a select subreddit” to: => ”headlines for a selected subreddit”.
small wording fix
Also update to latest version
Fixed a typo
Updated `npmcdn.com` links to `unpkg.com`
* Rename examples => examples_old * Port counter and counter-vanilla * Port async example * Port todos example * Port todos-with-undo example * Port some tests over * Add todomvc example * Ported shopping-cart example * Ported tree-view * Ported real-world * Add missing dep * Temp fix install until examples get an equivalent to buildAll.js * Restore example build scripts. * Bump react-scripts * Finish porting examples
Update Gitbook version
* Documentation : first draft for UsageWithReactRouter.md * uses React Router in writting, emphazises React Router in the introduction * transforms title to CamelCase * changes 'back-end' to 'development server', adds missing backticks * adds extension .js to title, fix grammar * adds missing semi-colons in code examples, adds space on self-closing tag, use PascalCase in configuration part titles * fix grammar on 'the todo list is not filtered' * fix innacurate information about Provider and Router relationship * adds a Next Steps section, adds a foot note about other routing libraries * clarifies 'the Router' > 'React Router' * mentions react-router package version * precises description in fallback url * rephrase in more concrete way the fallback URL configuration, adjust create-react-app header * uses 'url'> 'URL' * explains in a more conversational way how bind with React Router, 'Express.js' > 'Express', 'Webpack Dev Server' > 'WebpackDevServer' * fix grammar * removes space before colon
dist/bundle.js -> static/bundle.js the HTML portion says it's in "static" directory.
loss one single quote beside here:`<FilterLink />`
Fix some markdown issue. I added '``' into `[<Link />]` to make it be rendered by github-page. From > React Router comes with a component to > React Router comes with a **<Link />** component (Sorry, I should have pushed this commit with the one yesterday. However, I just found the 'feature' today. I have translated this article to Chinese. [Link](nefe/redux-in-chinese#145)) :D
Update ComputingDerivedData.md
Add missing commas
This short piece of code renders all html tags in JSON harmless, by replacing the `<` character with its unicode escape `\u003c`. Since the only place where JSON can have `<` is in strings, this is sufficient for protecting JSON from XSS injection.
Provide example of initial state SSR scrub
…-doc Doc: Improve readability of Usage with React Router section
Fixed typo and increased fidelity of description with respect to example.
…outer-doc Doc: clarify Navigating with React Router section
Added a line in handleChange() which fixes the issue of posts not being requested when a new subreddit is selected.
Updated ExampleRedditAPI.md
grammar fix - missing word
Update ReducingBoilerplate.md
Remove `export` from requestPosts, receivePosts, and fetchPosts actions, as these are not exported in the final version. Also suggest adding brief explanation as to why they are not exported.
reduxjs#2142) * Modification to docs/basics/Reducers to make imports more clear. This should help reduce the need to scroll down to the completed code at the bottom of the file, as mentioned in reduxjs#2107 * Removed instructions on importing SET_VISIBILITY_FILTER
Code typo: returns => return
…ave nothing to return
Simplify composer
fixed condition where multiple bad values would be filtered out and h…
Having unique IDs on the sub-headings allows linking to sections of the documentation easier. Without this change, the following link is ambiguous: http://redux.js.org/docs/basics/UsageWithReact.html#container-components
Changed sub-headings so that the IDs are unique
* Footer component filename is listed as `containers/Footer.js` in docs * Directory in the example app is `components/Footer.js` * Update docs to match example app
Fix filename typo in UsageWithReactRouter docs
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.