Skip to content

fix: server-render nested form value sets#15378

Merged
teemingc merged 2 commits intomainfrom
nested-set-form-fix
Feb 25, 2026
Merged

fix: server-render nested form value sets#15378
teemingc merged 2 commits intomainfrom
nested-set-form-fix

Conversation

@dummdidumm
Copy link
Member

Fixes #15327

  • make getter lazy instead of snapshotting once in advance, so we can deal with updates to the fields
  • deep_set is a "do-something" function that does not return something, but in the mutation case we just assigned the result of the function (which is undefined) to the input. Fixed accordingly

Fixes #15327
- make getter lazy instead of snapshotting once in advance, so we can deal with updates to the fields
- deep_set is a "do-something" function that does not return something, but in the mutation case we just assigned the result of the function (which is undefined) to the input. Fixed accordingly
@changeset-bot
Copy link

changeset-bot bot commented Feb 21, 2026

🦋 Changeset detected

Latest commit: 655fd7a

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@sveltejs/kit Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@teemingc teemingc added the forms Stuff relating to forms and form actions label Feb 24, 2026
@svelte-docs-bot
Copy link

@teemingc teemingc merged commit 244838c into main Feb 25, 2026
27 of 28 checks passed
@teemingc teemingc deleted the nested-set-form-fix branch February 25, 2026 08:59
@github-actions github-actions bot mentioned this pull request Feb 25, 2026
@teemingc teemingc linked an issue Feb 25, 2026 that may be closed by this pull request
Copilot AI pushed a commit to Stadly/kit that referenced this pull request Mar 6, 2026
Fixes sveltejs#15327
- make getter lazy instead of snapshotting once in advance, so we can
deal with updates to the fields
- deep_set is a "do-something" function that does not return something,
but in the mutation case we just assigned the result of the function
(which is undefined) to the input. Fixed accordingly

Co-authored-by: Tee Ming <chewteeming01@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

forms Stuff relating to forms and form actions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

form.fields.fieldName.set('value'); is not server-side rendered fields.value() data missing, despite setting values

2 participants