-
Notifications
You must be signed in to change notification settings - Fork 27
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
Add new default macro to declare read-write variables #285
Comments
The As such:
|
What about a different solution, like better errors/debugging? Ex in dev, if we try to access a variable that is not exposed, show a message: "Looks like you might be attempting to update the |
We could also add aliases as |
I want to stress that this is software for building data apps, targeting users and enterprises with highly sensitive data. We must 1000% err on the side of safety. |
Ok I see, I understand now why we need both in/out. I agree with showing better error messages, that'd completely solve this issue. |
@essenciary @hhaensel
Right now we have
@in
and@out
to declare read-write and read-only variables. Many times I've declared an@out
variable by mistake and then wondered why the UI component it was bound to wasn't working. I assume this could happen to other people as well.I'd like to have a different macro that would declare read-write variables and is named something other than
@in
, to avoid having to think whether the variable is read-only/read-write. Something like@io
,@bind
,@state
.And now that I think about it, given its name,
@in
should only take values from the browser to the backend and not the other way around. But this would be a breaking change for people's apps so let's not do that.The text was updated successfully, but these errors were encountered: