-
Notifications
You must be signed in to change notification settings - Fork 0
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
Update validity state functionality and documentation #165
Conversation
Reviewer's Guide by SourceryThis pull request updates the validity state functionality and documentation for form components. The key change is the replacement of the Class diagram showing updated form validation componentsclassDiagram
class BaseInput {
+String error
+String validity
+String setCustomValidity
+String setCustomValidityCustomError
+String setCustomValidityPatternMismatch
+String setCustomValidityValueMissing
+String setCustomValidityTooShort
+String setCustomValidityTooLong
+String setCustomValidityRangeOverflow
+String setCustomValidityRangeUnderflow
+validate()
}
class AuroInput {
+render()
}
class AuroSelect {
+String error
+String validity
+String setCustomValidity
+String setCustomValidityCustomError
+String setCustomValidityValueMissing
+reset()
+validate()
}
class AuroDatePicker {
+String error
+String validity
+String setCustomValidity
+String setCustomValidityCustomError
+String setCustomValidityValueMissing
+validate()
}
BaseInput <|-- AuroInput
BaseInput <|-- AuroSelect
BaseInput <|-- AuroDatePicker
note for BaseInput "Base class with common validation functionality"
State diagram for form validation statesstateDiagram-v2
[*] --> Valid
Valid --> Invalid: Validation fails
Invalid --> Valid: Validation passes
state Invalid {
[*] --> CustomError: error attribute set
[*] --> ValueMissing: required but empty
[*] --> PatternMismatch: pattern validation fails
[*] --> RangeOverflow: value > max
[*] --> RangeUnderflow: value < min
[*] --> TooLong: value too long
[*] --> TooShort: value too short
}
Invalid --> Valid: Validation passes
Valid --> [*]
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
cd34721
to
e69fb46
Compare
7c87570
to
009587a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @jordanjones243 - I've reviewed your changes and they look great!
Here's what I looked at during the review
- 🟢 General issues: all looks good
- 🟢 Security: all looks good
- 🟢 Testing: all looks good
- 🟢 Complexity: all looks good
- 🟢 Documentation: all looks good
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
6ad2251
to
c2c4c56
Compare
BREAKING CHANGE: patternMismatch replaces badInput state when user inputs do not match the expected pattern.
BREAKING CHANGE: input's errorMessage is now a private property.
c2c4c56
to
510ad18
Compare
🎉 This PR is included in version 2.0.0-beta.3 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Alaska Airlines Pull Request
Before Submitting this pull request:
Development
sectionnote: all pull requests require at least one linked ticket
Ready For Review
, all ticket's linked underDevelopment
must have their status changed toReady For Review
as wellBy submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license and I have performed a self-review of my own update.
Summary by Sourcery
Update the validity state functionality and documentation for form components.
Summary by Sourcery
Update the handling of validation states and error messages for form components.
New Features:
Enhancements:
Documentation: