-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
components as literal vars #4223
Conversation
return Bare.create("") | ||
|
||
|
||
def render_dict_to_var(tag: dict | Component | str, imported_names: set[str]) -> Var: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it would be nice if we could move some of this special specific logic closer to the components that will actually depend on it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
at the current stage there's no standardization of the dict format, so the checks are very tight to what we have right now, i can restructure the rendering logic in the future if needed
should be merged with reflex-dev/reflex-web#1047 |
waiting on reflex-dev/reflex-web#1050 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can we pare this down and keep the nextjs15 and turbo changes in the other PR #4243
done |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why are those tests removed? Is there a replacement that cover same usecases?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
they are removed because it's annoyingly specific, since there are integration test cases I thought it's not worth the effort of having test cases that just repeat the logic of the code.
No description provided.