Skip to content

use valueOrDefault in attribute hooks, update tests#357

Merged
hayleigh-dot-dev merged 5 commits intomainfrom
fix-356
Aug 3, 2025
Merged

use valueOrDefault in attribute hooks, update tests#357
hayleigh-dot-dev merged 5 commits intomainfrom
fix-356

Conversation

@yoshi-monster
Copy link
Contributor

@yoshi-monster yoshi-monster commented Jul 30, 2025

also updated all the tests to use the new assert. The reconciler tests are now also all run with the server-component json encoding in-between to test that code path too.

fixes #356

@yoshi-monster yoshi-monster changed the title Fix #356 use valueOrDefault in attribute hooks, update tests Jul 30, 2025
@hayleigh-dot-dev hayleigh-dot-dev merged commit 6994b5f into main Aug 3, 2025
1 check passed
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.

It seems empty-string value-attribute-values end up as undefined (for server side components)

2 participants