Skip to content

feat: zero v1.0.0 and disabled query support#216

Open
Gerbuuun wants to merge 8 commits intomainfrom
feat/zero-v1.0
Open

feat: zero v1.0.0 and disabled query support#216
Gerbuuun wants to merge 8 commits intomainfrom
feat/zero-v1.0

Conversation

@Gerbuuun
Copy link
Copy Markdown
Collaborator

@Gerbuuun Gerbuuun commented Mar 25, 2026

  • upgrade @rocicorp/zero to v1.0
  • updates all other dependencies
  • adds disabled query feature

@socket-security
Copy link
Copy Markdown

socket-security Bot commented Mar 25, 2026

@maxstevens-nl maxstevens-nl self-requested a review March 27, 2026 09:25
Copy link
Copy Markdown
Collaborator

@maxstevens-nl maxstevens-nl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • We should also bump the version in playground/package.json.
  • #217 should fix the broken typecheck, if that's merged this should be good to go

@Gerbuuun
Copy link
Copy Markdown
Collaborator Author

Before releasing I want to add the disabled query feature. This happens when undefined is passed in useQuery. I've added the disabled status to useQuery which is not used by the other zero packages. What do you think? (using unknown otherwise would be weird, and complete also doesn't feel right)

Copy link
Copy Markdown
Collaborator

@maxstevens-nl maxstevens-nl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you add a test for the disabled query state and rebase the branch this is good to go

- Allow `useQuery` to accept falsy queries and expose a disabled state
- Use `ErroredQuery.message` when surfacing query errors
- Allow `useInterval` to accept reactive delay values and clean up intervals reliably
- Pass `cacheURL` to Zero config instead of `server` in the playground
@Gerbuuun Gerbuuun changed the title chore (deps): update dependency @rocicorp/zero to v1.0.0 feat: zero v1.0.0 and disabled query support Apr 28, 2026
@Gerbuuun Gerbuuun requested a review from maxstevens-nl April 28, 2026 11:45
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.

2 participants