Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Input validity #254

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

Input validity #254

wants to merge 1 commit into from

Conversation

Ericson2314
Copy link
Member

@Ericson2314 Ericson2314 commented Nov 15, 2018 •
edited
Loading

Happens to include #253 that PR is now merged.

Ericson2314 added a commit to obsidiansystems/reflex-dom-forms that referenced this pull request Nov 16, 2018
@Ericson2314
Use reflex-dom PR branch
bb011e1 
Need reflex-frp/reflex-dom#254
@Ericson2314 Ericson2314 changed the base branch from develop to ModalsT January 3, 2019 20:03
@Ericson2314 Ericson2314 changed the base branch from ModalsT to develop January 3, 2019 20:03
@hSloan hSloan added the opinionated Requires judgement from senior developer or developer community label Jan 23, 2019
@Ericson2314
Expose a bare minimum of HTML 5 Form validity
aa58261 
Users can set custom custom validity status. There are a number of
way/reason to get the validity info, and so nothing is exposed to get
it at this time.
@ali-abrar
Copy link
Member

@luigy could you have a look at this?

@ApolloUnicorn ApolloUnicorn requested review from JBetz and matthewbauer and removed request for luigy December 3, 2019 19:00
JBetz
JBetz reviewed Nov 1, 2024
View reviewed changes
Copy link
Contributor

@JBetz JBetz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just leaving a comment to get this off of my review queue, since GitHub doesn't seem to let me remove myself as a reviewer.

@luigy luigy removed the request for review from matthewbauer November 1, 2024 13:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JBetz JBetz JBetz left review comments

Assignees
No one assigned
Labels
breaking change opinionated Requires judgement from senior developer or developer community
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Ericson2314 @ali-abrar @JBetz @hSloan