Skip to content

fix: include is_checked in query context note maps#167

Merged
careck merged 1 commit intomasterfrom
fix/is-checked-query-context
Apr 28, 2026
Merged

fix: include is_checked in query context note maps#167
careck merged 1 commit intomasterfrom
fix/is-checked-query-context

Conversation

@careck
Copy link
Copy Markdown
Member

@careck careck commented Apr 28, 2026

Summary

  • note_to_rhai_dynamic was missing the is_checked field, so get_children()/get_note() results always had is_checked undefined
  • Views filtering by checkbox state (e.g. task progress counters) got zero matches
  • Adds the missing field to match build_note_map which already includes it

Test plan

  • All 628 core tests pass
  • Import a workspace with checked Task notes, verify is_checked is accessible from views using get_children()

note_to_rhai_dynamic was missing the is_checked field, so
get_children/get_note results always had is_checked undefined.
This broke views that filter by checkbox state (e.g. task progress).
@careck careck merged commit 2194e96 into master Apr 28, 2026
2 checks passed
@careck careck deleted the fix/is-checked-query-context branch April 29, 2026 06:47
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