Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix data context handling in _load to align with Blaze@3.0.1 #121

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

aboire
Copy link

@aboire aboire commented Feb 4, 2025

In Blaze@3.0.1, dataVar.set() now requires an object { value: current.data ?? {} } instead of setting current.data directly.

This commit updates ostrio:flow-router-extra@3.11.0 to follow this change and ensure proper data propagation in templates and layouts.

#120

In Blaze@3.0.1, `dataVar.set()` now requires an object `{ value: current.data ?? {} }` instead of setting `current.data` directly.

This commit updates `ostrio:flow-router-extra@3.11.0` to follow this change and ensure proper data propagation in templates and layouts.
Blaze 3.0.1 introduced a change where `dataVar.set()` expects an object `{ value: ... }` instead of setting `current.data` directly. However, previous versions of Blaze (< 3.0.1) do not use this structure.
@jankapunkt
Copy link

This fixes param/queryParam reactivity in Blaze (which are back available in autorun -> Template.currentData())

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants