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

Fix FormComponent showing error indicators in all fields when using a customValidate function #15464

Merged
merged 1 commit into from Dec 12, 2023

Conversation

mmichilot
Copy link
Contributor

References

Fixes: #15447

Code changes

Also check the length of rawErrors since rawErrors will be an empty array rather than undefined when a customValidate function is passed to FormComponent.

Copy link

Thanks for making a pull request to jupyterlab!
To try out this branch on binder, follow this link: Binder

@mmichilot
Copy link
Contributor Author

This PR should be ready, is there anything else that I should do?

Copy link
Member

@krassowski krassowski left a comment

Choose a reason for hiding this comment

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

Indeed, this looks good thanks @mmichilot!

@krassowski krassowski merged commit 2dc294c into jupyterlab:main Dec 12, 2023
76 of 81 checks passed
@krassowski
Copy link
Member

@meeseeksdev please backport to 4.0.x

meeseeksmachine pushed a commit to meeseeksmachine/jupyterlab that referenced this pull request Dec 12, 2023
…ators in all fields when using a `customValidate` function
krassowski pushed a commit that referenced this pull request Dec 12, 2023
…ll fields when using a `customValidate` function (#15522)

Co-authored-by: Matthew Michilot <matthew.michilot@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

All FormComponent fields have an error indicator when customValidate function is passed
2 participants