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

Picker dialog validation #3242

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

Conversation

adids1221
Copy link
Contributor

@adids1221 adids1221 commented Sep 4, 2024

Description

Picker Dialog validation feature.
New UsePickerDialogProps hook to determine the default props of the dialog instead of the index file.
UsePickerSelection hook handles the validation and trigger the selectionValidation function.
Also in this PR there is a fix, Picker Dialog shouldn't render Modal.TopBar (at least when the user pass migrateDialog).

Changelog

Picker Dialog validation feature.

Additional info

MADS-4430

@ethanshar ethanshar added the Important for Next Release PR that must be included in the release version label Sep 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Important for Next Release PR that must be included in the release version
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants