Skip to content

Property Editors: Added form control and mandatory support(User, Member, Member Group)#20672

Merged
nielslyngsoe merged 12 commits intomainfrom
v17/improvement/property-editors-people-group
Dec 2, 2025
Merged

Property Editors: Added form control and mandatory support(User, Member, Member Group)#20672
nielslyngsoe merged 12 commits intomainfrom
v17/improvement/property-editors-people-group

Conversation

@engijlr
Copy link
Contributor

@engijlr engijlr commented Oct 28, 2025

Summary

Update some Property Editors to align with the FormControl validation pattern.

Changes

  • Extended each component with UmbFormControlMixin.
  • Bound the inner input element to the form control (addFormControlElement).
  • Added support for mandatory and mandatoryMessage properties.
  • Ensured that if a field is not mandatory, it supports undefined values.
  • Added a value-missing validator inside the inner input for proper required-state validation.
  • Implemented consistent handling for required, requiredMessage, and readonly.
  • Fixed small typos and attribute mismatches across components.

@iOvergaard iOvergaard changed the base branch from v17/dev to main October 30, 2025 10:51
@nielslyngsoe nielslyngsoe enabled auto-merge (squash) December 1, 2025 14:38
@nielslyngsoe nielslyngsoe merged commit da66cbd into main Dec 2, 2025
27 of 28 checks passed
@nielslyngsoe nielslyngsoe deleted the v17/improvement/property-editors-people-group branch December 2, 2025 07:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants