-
Notifications
You must be signed in to change notification settings - Fork 117
feature core/input: validation in input components #2278
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
base: main
Are you sure you want to change the base?
feature core/input: validation in input components #2278
Conversation
…ithub.com/1307-Dev/ix into (feature/core)validation-input-components
…ithub.com/1307-Dev/ix into (feature/core)validation-input-components
|
…ithub.com/1307-Dev/ix into (feature/core)validation-input-components
…ithub.com/1307-Dev/ix into (feature/core)validation-input-components
…ithub.com/1307-Dev/ix into (feature/core)validation-input-components
|



💡 What is the current behavior?
There are some errors which prevent validations from happening has expected.
GitHub Issue Number: #1799
JIRA: https://agileworld.siemens.cloud/jira/browse/IX-2595
🆕 What is the new behavior?
The acceptance criteria for this ticket has been met-
Required input:
Invalid input > Removing value with keyboard > Stays invalid
Invalid input > Remove touched state (e.g. clear button) > Valid again
Invalid input > Programmatically setting it to empty > Stays invalid
Valid input > Removing value with keyboard > It is invalid
Valid input > Remove touched state (e.g. clear button) > Valid
Valid input > Programmatically setting it to empty > It is invalid
Not required input:
Invalid input > Removing value with keyboard > Valid
Invalid input > Remove touched state (e.g. clear button) > Valid again
Invalid input > Programmatically setting it to empty > Valid
Valid input > Removing value with keyboard > Valid
Valid input > Remove touched state (e.g. clear button) > Valid
Valid input > Programmatically setting it to empty > Valid
🏁 Checklist
A pull request can only be merged if all of these conditions are met (where applicable):
pnpm test)pnpm lint)pnpm build, changes pushed)👨💻 Help & support