-
Notifications
You must be signed in to change notification settings - Fork 39
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
fixes: #76bug add validation for all input field's #77
fixes: #76bug add validation for all input field's #77
Conversation
…anges in regex link validation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Make sure all the functions are working properly.
- After making the changes, make sure the code is not broken or getting affected due to changes. - Also, try to check everything is working by adding a few entries from your end.
@MayankBansal12 are we good to merge? |
yes we can merge this PR |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You only need to modify the add categories. Other than that, everything else looks good.
new list of category
@love-sonkar - thanks for your contribution! |
fixes #76 added validation in all input field's and for the category i use a dropdown selection for better user experiance and for validation.