Skip to content
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

Remove duplicated page titles, fix sidebar menu #1033

Merged
merged 3 commits into from
Oct 6, 2018

Conversation

BTMPL
Copy link
Contributor

@BTMPL BTMPL commented Oct 6, 2018

No description provided.

@@ -24,7 +25,7 @@ yarn add react-redux

<!-- perhaps add link to an extra quick start section? -->

## `<Provider />` and `connect`
Copy link
Contributor Author

@BTMPL BTMPL Oct 6, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Using the less-than/greater-than signs tries to render HTML tag; changing to &lt; and &gt; just shows the escaped strings :(

@BTMPL
Copy link
Contributor Author

BTMPL commented Oct 6, 2018

Updated the PR a bit more to generate proper sidebar on API page:

image

@markerikson markerikson merged commit 2e4465a into reduxjs:master Oct 6, 2018
@markerikson
Copy link
Contributor

Sweet!

albertodev7 pushed a commit to albertodev7/react-redux that referenced this pull request Dec 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants