Skip to content
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

Infer the data type of User defined variables #3599

Open
CharString opened this issue Nov 10, 2023 · 1 comment
Open

Infer the data type of User defined variables #3599

CharString opened this issue Nov 10, 2023 · 1 comment
Assignees

Comments

@CharString
Copy link
Contributor

As far as I can tell, the only reason to define a variable is if you intend to use it.

This would partially close #2707, because together with #3597, #3598 and open-formulieren/InferNoLogic#2, just the assignment to userVar in the action would be enough to infer that it is in fact a date type.

Some advanced Form Designers, may like to specify the type they intend for their variables, so the type checker may help them catch it when they make a mistake. For instance when they intend to use the variable in a ServiceFetch or a registration backend.

Adding a data type "auto" and make it the default for user defined variables, would enable both use cases.

Adding the types a registration backend expects and for servicefetch services that have a OpenAPI spec to the context, analogue to #3597, would make "auto" sufficient for everything but unspecified services.

@CharString CharString converted this from a draft issue Nov 10, 2023
@CharString CharString self-assigned this Nov 10, 2023
@joeribekker joeribekker added this to the Release 2.5.0 milestone Nov 13, 2023
@joeribekker
Copy link
Contributor

Type inference feature was cancelled due to implementation difficulties vs benefits.

@joeribekker joeribekker removed this from the Release 2.5.0 milestone Jan 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants