Skip to content

feature/prep reflex update #377

New issue

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

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

Already on GitHub? Sign in to your account

Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open

Conversation

cutoffthetop
Copy link
Contributor

@cutoffthetop cutoffthetop commented Jul 23, 2025

Changes

  • prep reflex update:
    • avoid byte operators on Vars, use rx.cond instead
    • avoid Var.to_string(), use f-strings to render vars instead
    • avoid magical setters/getters on State, implement them explicitly
    • avoid calling event handlers from other event handlers, use private methods / direct assignments instead
    • avoid kwargs on event handlers, use positional Mapping argument instead
    • avoid getting granian warnings, set env var explicitly
    • avoid defaults for state model fields, making a swap to dataclasses easier
  • fix some typing
  • de-duplicate pagination component
  • de-duplicate ingestion roundtrip tests

@cutoffthetop cutoffthetop added the blocked This cannot be resolved yet label Jul 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked This cannot be resolved yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant