Skip to content

Conversation

@Talos0248
Copy link
Contributor

@Talos0248 Talos0248 commented Dec 10, 2025

…tsx to maintain consistency with EditPostForm.tsx


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

Checklist

  • Is there an existing issue for this PR?
    • Nope! I think directly opening a pull request and discussing there is a better approach, according to my best judgement.
  • Have the files been linted and formatted?
    • Ran yarn lint just in case but nothing changed.

What docs page needs to be fixed?

4 sections across 3 pages involving the AddPostForm.tsx component file

Occurrence 1: https://redux.js.org/tutorials/essentials/part-3-data-flow#adding-new-posts

  • Section: Adding New Posts
  • Page: Redux Essentials, Part 3: Basic Redux Data Flow

Occurrence 2: https://redux.js.org/tutorials/essentials/part-3-data-flow#dispatching-the-post-added-action

  • Section: Dispatching the Post Added Action
  • Page: Redux Essentials, Part 3: Basic Redux Data Flow

Occurrence 3: https://redux.js.org/tutorials/essentials/part-4-using-data#adding-authors-for-posts

  • Section: Adding Authors for Posts
  • Page: Redux Essentials, Part 4: Using Redux Data

Occurrence 4: https://redux.js.org/tutorials/essentials/part-7-rtk-query-basics#using-mutation-hooks-in-components

  • Section: Using Mutation Hooks in Components
  • Page: Redux Essentials, Part 7: RTK Query Basics

What is the problem?

To preface, the code already works without the name attribute, since the submission handling code intercepts the default form event and instead relies on the element ids. However, I've noticed that the similar EditPostForm.tsx component has the name attribute for both the text and textarea input. I figured adding the name attribute to the input element to AddPostForm.tsx may help with consistency.

Admittedly, as I was going through the essential docs as a learner, there was a moment of confusion/self-doubt around the lines of "wait why does this input not have 'name', but the one on EditPostForm does? is this a special redux thing i'm missing?" haha

Also, I'm not sure what I'm doing to fail github workflow checks, only .md files have been edited :(

What changes does this PR make to fix the problem?

Adds name="postTitle" in all relevant AddPostForm.tsx snippets I could find.

…tsx to maintain consistency with EditPostForm.tsx
@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.

@netlify
Copy link

netlify bot commented Dec 10, 2025

Deploy Preview for redux-docs ready!

Name Link
🔨 Latest commit 1447204
🔍 Latest deploy log https://app.netlify.com/projects/redux-docs/deploys/6939482c0eca21000844257f
😎 Deploy Preview https://deploy-preview-4829--redux-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

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.

1 participant