-
Notifications
You must be signed in to change notification settings - Fork 24
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
feat: added fieldTypes prop to form #218
Conversation
Introduce a way to configure the input types through a Form prop. Signed-off-by: Julien Terzibaschian <julien@terzibaschian.com>
✅ Deploy Preview for remix-forms ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
@danielweinmann could you please review this pr? I want it to get merged if everything is correct. |
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.
Hey, @ReptoxX! Thanks for the PR and sorry for the delay in reviewing it. It's looking real good! I just requested a few minor changes and we'll be good to go!
Hey @danielweinmann, Hope this works out. |
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.
LGTM! Can you resolve the conflicts before we merge?
Okay i've resolved the conflict. |
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.
@ReptoxX, can you fix the broken tests? It looks like it's just a label that's incorrect.
Thank you, @ReptoxX! 🎉 |
Introduce a way to configure the input types through a Form prop.
As mentioned in #217.