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.

Sign up for GitHub

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

Jump to bottom

Issue Updated - Multiple ids with the same value #138

Open
wants to merge 2 commits into
base: 16-working-with-forms
Choose a base branch
from
Open

Issue Updated - Multiple ids with the same value #138

wants to merge 2 commits into from

Commits on Apr 25, 2023

  1. Issue Updated - Multiple ids with the same value 

    When I click on one input it always points to name field.
In the starting structure provided, the id field for all 3 input elements is the same. I guess this is unintentional typo and not a best practice.
Please merge it so other people like don't get confuse, great course as always.
    @RushikeshGandhmal
    RushikeshGandhmal committed Apr 25, 2023
    Configuration menu
    Copy the full SHA
    95126c1 View commit details
    Browse the repository at this point in the history

  2. return initialInputState and not inputStateReducer 

    you must return initialInputState and not inputStateReducer
    @RushikeshGandhmal
    RushikeshGandhmal committed Apr 25, 2023
    Configuration menu
    Copy the full SHA
    7e8a3ee View commit details
    Browse the repository at this point in the history