Form Validation with JavaScript: <clarify Instructions for an exercise>

[GaneshS288]

Checks

• [x] This is not a duplicate of an existing issue (please have a look through our open issues list to make sure)
• [x] I have thoroughly read and understand The Odin Project Contributing Guide
• [x] Would you like to work on this issue?

Describe your suggestion

Problem

There is an exercise about adding form validation to a previous project 'Library' (link below) https://www.theodinproject.com/lessons/node-path-javascript-library

The instructions tell learners to 'not let users submit the form without filling in all fields`. However prior to the Library projects you have learned native form validation so most learners must've used required attribute on their form input fields.

This can create confusion for the learners as the form is already handling empty fields.

Suggestions

We could change the instructions to tell learners that they shoudl display custom error messages instead of the default "Please fill out this field". So the exercise paragraph would look like this.

• Go back to your ‘Library’ project and add validation to that form! Display a custom error message when the user tries to submit an empty form field (eg. "The author name must be filled"). Don’t forget to use the git branch workflow you learned in Revisiting Rock Paper Scissors from Foundations to work on a new feature

Path

Node / JS

Lesson Url

https://www.theodinproject.com/lessons/node-path-javascript-form-validation-with-javascript

(Optional) Discord Name

evilganesh

(Optional) Additional Comments

No response

[github-actions[bot]]

This issue is stale because it has had no activity for the last 30 days.

[GaneshS288]

Bumped

[CouchofTomato]

@TheOdinProject/javascript Does anyone have an opinion on this?

[github-actions[bot]]

This issue is stale because it has had no activity for the last 30 days.

[GaneshS288]

Bump

[github-actions[bot]]

This issue is stale because it has had no activity for the last 30 days.

[github-actions[bot]]

This issue is stale because it has had no activity for the last 30 days.

[mao-sz]

Sorry this slipped through the cracks, @GaneshS288 I like the suggested change