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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

polish index.html #4204

Closed
wants to merge 1 commit into from
Closed

polish index.html #4204

wants to merge 1 commit into from

Conversation

koraytugay
Copy link


name: 馃摑 Documentation Fix
about: Fixing a problem in an existing docs page

Checklist

  • Is there an existing issue for this PR?
    • link issue here
  • Have the files been linted and formatted?

What docs page needs to be fixed?

  • Section:
  • Page:

What is the problem?

What changes does this PR make to fix the problem?

@codesandbox-ci
Copy link

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit a405fd7:

Sandbox Source
Vanilla Configuration
Vanilla Typescript Configuration

@netlify
Copy link

netlify bot commented Oct 30, 2021

鉁旓笍 Deploy Preview for redux-docs ready!

馃敤 Explore the source changes: a405fd7

馃攳 Inspect the deploy log: https://app.netlify.com/sites/redux-docs/deploys/617ca6e900377700087cf792

馃槑 Browse the preview: https://deploy-preview-4204--redux-docs.netlify.app

@markerikson
Copy link
Contributor

Hi. Appreciate the PR, but I don't think these changes are needed atm.

That said, we would be interested in trying to update some of the other examples to match our current "Modern Redux" approaches:

#3917

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.

None yet

2 participants