-
Notifications
You must be signed in to change notification settings - Fork 3
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
validation #37
Comments
What type of validation are you talking about in our form |
first of all - we cannot submit an empty form. |
let's mark this like an open thing. i mean we need validation, for sure. so implement checking if form not empty and add ability to set required fields. also maybe we need to have some notifications, like : "please fill form", etc.... |
hold on. i think antd have validation stuff. |
Okay I'll check it out and tell u asap.Will that be fine |
awesome - thank you!
On 28-May-2018 at 07:34 PM, Rahul Thakur wrote:
Okay I'll check it out and tell u asap.Will that be fine
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#37 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABZrDhb8tUVMFJDFod3mD77b2e1CUqviks5t3FFRgaJpZM4UDhhq>
.
|
I still think that this is unnecessary stuff because ingredients, allergy, diet, holiday are unnecessary stuff and not everyone wants to search a specific stuff
The only thing I can think of keeping as required is course but in that also we have two options(which according to me one is enough) and we are only going to display the cuisines that user wants, am I right? then why are we asking the user which cuisines they don't want |
https://ant.design/components/form/ contains validation example which we can use |
if you are thinking regarding hundreds of recepies that the server will bombard then we can do some stuff like google and only display one page at a time so that the load from server is also reduced and the user also only sees a few at a time |
https://github.com/25th-floor/spected
The text was updated successfully, but these errors were encountered: