Skip to content

feat(require-ion-error-text): lint Ionic validation messages - #30

Merged
rdlabo merged 1 commit into
mainfrom
feat/require-ion-error-text
Aug 26, 2026
Merged

feat(require-ion-error-text): lint Ionic validation messages#30
rdlabo merged 1 commit into
mainfrom
feat/require-ion-error-text

Conversation

@rdlabo

@rdlabo rdlabo commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

Overview

Adds an Angular template lint rule that requires a valid Ionic error-text source for controls participating in Signal Forms validation.

The default scope is intentionally limited to supported Ionic controls with [formField], avoiding false positives for filters and settings controls that do not represent validation fields.

Changes

  • Add require-ion-error-text
  • Support ion-input, ion-textarea, ion-select, ion-checkbox, ion-radio-group, and ion-toggle
  • Accept non-empty static errorText and property-bound [errorText]
  • Reject empty text and [attr.errorText]
  • Add adapter-aware, check-all, and readonly options
  • Enable the rule in the recommended preset
  • Document the 22.1 migration and configuration

Compatibility

The recommended preset gains a new error-level rule. The default only checks controls with [formField]; applications using KitIonicFormField can explicitly enable adapter-aware mode.

Validation

  • npm run lint
  • npm test -- --runInBand — 594 tests passed

@rdlabo
rdlabo merged commit a87aafc into main Aug 26, 2026
5 checks passed
@rdlabo
rdlabo deleted the feat/require-ion-error-text branch August 26, 2026 09:46
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.

1 participant