-
Notifications
You must be signed in to change notification settings - Fork 886
Rich parameters: remove old parameters #6075
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
Comments
Hey @kylecarbs, I'd like to hear your thoughts about the execution here. Once we have all blockers resolved, should we just remove old parameters, or give users a month to move their workspaces. I would like to spend time on the implementation of auto-migrators as I'm sure that we won't cover all edge cases. |
Let's give users 1 month to migrate their templates + display a warning for template admins that their workspace uses template parameters that will be deprecated by I have some open questions, though:
|
To be honest, this is one of those issues that are tangled and "expensive" to implement, so I'm rather hesitant. The migration to the new workspace will happen only once, and sometimes it's easier/better to just have a second pair of eyes instead of having a logic that may be buggy.
Currently, it's required to set a default value for AFAIR We had a chat that some complex cases are more predictable/harmful to solve by creating the 2nd workspace.
There are a few paths:
|
All makes sense to me. |
Extracted from: #5574
Blockers:
variable
#5980The goal of this issue is to deprecate and remove old parameters in favor of rich parameters.
To be reviewed and adjusted:
Once this issue is resolved, it won't be possible to use templates with old parameters, and all relevant workspaces must be switched to templates with rich parameters.
The text was updated successfully, but these errors were encountered: