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.

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

Dynamic form - Number validation is not working, if the field is set to minimum and maximum value #1571

Closed
NishkalankBezawada opened this issue Jun 28, 2023 · 4 comments
Assignees
Labels
status:fixed-next-drop Issue will be fixed in upcoming release. type:bug
Milestone

Comments

@NishkalankBezawada
Copy link
Contributor

Category

[ ] Enhancement

[x] Bug

[ ] Question

Version

[3.14.0]

Expected / Desired Behavior / Question

I have column in my list of type number, where I I have configured a max value of 10000.

image

This works fine on List form but with Dynamic form, this validation fails and the user can enter any value.

image

Observed Behavior

The user can enter any value even greater than 10000

Steps to Reproduce

  1. Create list list column of type number and set max value to 10000
  2. Render the list form via, PnP Dynamic form
  3. Add any value to this column even greater than 10000, the form submits the data.

Thanks,
Nishkalank Bezawada

@ghost
Copy link

ghost commented Jun 28, 2023

Thank you for reporting this issue. We will be triaging your incoming issue as soon as possible.

@ghost ghost added the Needs: Triage 🔍 label Jun 28, 2023
@github-actions
Copy link

Thank you for submitting your first issue to this project.

@joelfmrodrigues joelfmrodrigues added type:bug status:working-on-it Known issue / feature being addressed. Will use other "status:*" labels & comments for more detail. and removed Needs: Triage 🔍 labels Jun 29, 2023
@NishkalankBezawada
Copy link
Contributor Author

Hello @joelfmrodrigues

Created PR 1575 for the same.

Kindly review.

Thanks,
Nishkalank

@AJIXuMuK AJIXuMuK added this to the 3.15.0 milestone Jul 9, 2023
@AJIXuMuK AJIXuMuK added status:fixed-next-drop Issue will be fixed in upcoming release. and removed status:working-on-it Known issue / feature being addressed. Will use other "status:*" labels & comments for more detail. labels Jul 9, 2023
@joelfmrodrigues
Copy link
Collaborator

New release just published, closing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status:fixed-next-drop Issue will be fixed in upcoming release. type:bug
Projects
None yet
Development

No branches or pull requests

3 participants