-
-
Notifications
You must be signed in to change notification settings - Fork 101
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’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
CYF-London | seerat junaid | Module-User-Focused-Data | Week 2 #319
base: main
Are you sure you want to change the base?
Conversation
ITP | SEERAT FATIMA | Module User Focused Data | Sprint 1
Revert "ITP | SEERAT FATIMA | Module User Focused Data | Sprint 1"
✅ Deploy Preview for cyf-ufd ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Form looks great. May I suggest using a HTML validator (https://validator.w3.org/) to check for potential syntax errors in your HTML code? Also, any reason to accept two or more spaces as a valid name? |
Thank you for taking the time to review my form. Your suggestion to use the HTML Validator was invaluable, especially since it was new to me. The tool made it easy to identify and correct my mistakes. Additionally, I was experimenting with some new features, which led to unintentionally allowing multiple spaces in the name field. I have now addressed this issue and ensured that only single spaces are accepted. Thanks again for your helpful feedback! latest commit here |
If you think you have made all the suggested changes, you can mark your work as completed. |
Learners, PR Template
Self checklist
Changelist
Briefly explain your PR.
Questions
Ask any questions you have for your reviewer.