data fetching with the delay message & Feature Validate AddBooking #53
Conversation
| const [error, setError] = useState(null); | ||
|
|
||
| // Function to check the email format | ||
| const isValidEmail = (email) => { |
There was a problem hiding this comment.
Why didn't you use the pattern attribute to add validation on email input indead of creating a new funtion? Like here https://developer.mozilla.org/en-US/docs/Web/HTML/Element/input/email
| alert("Please fill in all required fields."); | ||
|
|
||
| // Validation for first name, last name, email, and room ID | ||
| if ( |
There was a problem hiding this comment.
Same here. Input element have attribute required to validate that input id filled or not.
look into this https://www.w3schools.com/js/js_validation.asp
There was a problem hiding this comment.
That because it's only works on the HTML on react I'm trying to check if any of the fields are empty, that's what I understood when I was testing the validation function. It would make an extra line if I added the required attribute on the input fields
| @@ -1,97 +1,97 @@ | |||
| // Bookings.scss | |||
|
|
|||
| .bookings { | |||
| padding: 20px; | |||
There was a problem hiding this comment.
The styles looks good.
I did not know that input validation is another feature called validate AddBookings. so this PR is for two features. and I did a stretch validation to check if the room is booked