Skip to content

Comments

signal forms: align blog posts with submit-makeover changes#74

Merged
d-koppenhagen merged 5 commits intomainfrom
signal-forms-submit-makeover
Feb 21, 2026
Merged

signal forms: align blog posts with submit-makeover changes#74
d-koppenhagen merged 5 commits intomainfrom
signal-forms-submit-makeover

Conversation

@fmalcher
Copy link
Member

@fmalcher fmalcher commented Feb 20, 2026

Aligns all four Signal Forms blog posts with the submit-makeover branch changes: angular-buch/signal-forms-registration#14

  • Use FormRoot directive and submission: { action } config as primary submission pattern
  • Add manual submit() function as documented alternative (Part 1)
  • Rename ValidationErrorWithFieldWithFieldTree<ValidationError> (Part 2)
  • Add trailing periods to all validation messages
  • Update validateAsync params: ({ value }) => value()(ctx) => ctx.value()
  • Code snippets build up incrementally (no premature FormRoot or schema references)

Update all four blog posts to use the FormRoot directive and
submission config pattern as the primary approach. Add submit()
as an alternative. Rename ValidationErrorWithField to
WithFieldTree<ValidationError>, add trailing periods to validation
messages, and update validateAsync params syntax.
@fmalcher fmalcher force-pushed the signal-forms-submit-makeover branch from 73e205f to a058059 Compare February 20, 2026 22:31
@fmalcher fmalcher changed the title Align Signal Forms blog posts with submit-makeover changes signal forms: align blog posts with submit-makeover changes Feb 20, 2026
Use email().value.update() instead of email.value.update()
to correctly call the FieldTree as a function before accessing
the FieldState's value signal.
@fmalcher fmalcher marked this pull request as ready for review February 21, 2026 12:30
@fmalcher
Copy link
Member Author

@d-koppenhagen Fertig, bitte mergen wenn alles passt 😊

@d-koppenhagen d-koppenhagen merged commit d99c164 into main Feb 21, 2026
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.

3 participants