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

[Form Common API]: Remove disabled & invalidState properties #448

Merged
merged 8 commits into from
Sep 30, 2024

Conversation

videoeero
Copy link
Contributor

@videoeero videoeero commented Sep 28, 2024

Jira-ticket: https://funidata.atlassian.net/browse/DS-296

BREAKING CHANGE:

  • Affecting all form components (text-input, datepicker etc.) disabled and invalidState properties are removed. Disabling should be done straight from FormControl. InvalidState was previously used internally, but is no longer needed and form field's invalid state should come from FormControl as well.

IMPROVEMENT LocalizedTextGroup

  • If whole FormGroup is disabled (formGroup.disable()), it will disable the associated Select Component as well

@videoeero videoeero added the breaking change This change is not backwards-compatible label Sep 28, 2024
@MayaMarjut MayaMarjut self-requested a review September 30, 2024 05:46
MayaMarjut
MayaMarjut previously approved these changes Sep 30, 2024
@github-actions github-actions bot temporarily deployed to Documentation for branch DS-296-remove-disabled-input September 30, 2024 06:52 Destroyed
@github-actions github-actions bot temporarily deployed to Documentation for branch DS-296-remove-disabled-input September 30, 2024 07:53 Destroyed
@github-actions github-actions bot temporarily deployed to Documentation for branch DS-296-remove-disabled-input September 30, 2024 08:01 Destroyed
@github-actions github-actions bot temporarily deployed to Documentation for branch DS-296-remove-disabled-input September 30, 2024 08:14 Destroyed
@github-actions github-actions bot temporarily deployed to Documentation for branch DS-296-remove-disabled-input September 30, 2024 09:51 Destroyed
@MayaMarjut MayaMarjut merged commit de5f149 into main Sep 30, 2024
10 checks passed
@MayaMarjut MayaMarjut deleted the DS-296-remove-disabled-input branch September 30, 2024 10:36
@github-actions github-actions bot locked and limited conversation to collaborators Sep 30, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
breaking change This change is not backwards-compatible
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants