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: πŸ› SQFormDialogStepper validation works properly #791

Merged
merged 1 commit into from
Sep 12, 2022

Conversation

20BBrown14
Copy link
Contributor

@20BBrown14 20BBrown14 commented Aug 31, 2022

SQFormDialogStepper validation now respects validation schema properly.
SQFormDialogStepper now validations on mount.

BREAKING CHANGE: 🧨 SQFormDialogStepper respects validation schema

βœ… Closes: #790

Loom: https://www.loom.com/share/6cd90ed9ec124121b0a72a187fbe0dc8

DO NOT MERGE UNTIL #793 is merged

@20BBrown14 20BBrown14 linked an issue Aug 31, 2022 that may be closed by this pull request
Copy link
Contributor

@laurelbean laurelbean left a comment

Choose a reason for hiding this comment

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

One of the tests is failing. Once that's fixed everything else looks good!

Copy link
Contributor

@aharpt aharpt left a comment

Choose a reason for hiding this comment

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

Approved pending fixing the build issue.

SQFormDialogStepper validation now respects validation schema properly.
SQFormDialogStepper now validations on mount.

BREAKING CHANGE: 🧨 SQFormDialogStepper respects validation schema

βœ… Closes: #790
@20BBrown14 20BBrown14 merged commit bf6b18e into issue-447/MUI5_migration Sep 12, 2022
@20BBrown14 20BBrown14 deleted the issue-790 branch September 12, 2022 14:43
kaylamc2 pushed a commit that referenced this pull request Dec 2, 2022
## [10.0.0](v9.7.0...v10.0.0) (2022-12-02)

### ⚠ BREAKING CHANGES

* 🧨 SQFormDialogStepper respects validation schema
* 🧨 Renamed rows and maxRows props on SQFormTextarea

### Features

* 🎸 Migrate InclusionList to MUI5 ([7c8b982](7c8b982)), closes [#729](#729)
* 🎸 Migrate SQFormAutoComplete to MUI5 ([0657e4b](0657e4b)), closes [#723](#723)

### Bug Fixes

* πŸ› SQFormDialogStepper validation works properly ([#791](#791)) ([bf6b18e](bf6b18e)), closes [#790](#790)
* πŸ› SQFormScrollable cards accepts value and label props ([#793](#793)) ([1a669b3](1a669b3)), closes [#792](#792)

### Code Refactoring

* πŸ’‘ SQFormTextarea to MUIv5 ([30d2301](30d2301)), closes [#710](#710)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

SQFormDialogStepper validation not working properly
4 participants