-
Notifications
You must be signed in to change notification settings - Fork 881
rich parameters: add /prompt API #7099
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
Hi, @mtojek I believe this is now possible with the new |
Ephemeral fields are not related to this initiative. The goal of this issue is to move logic to determine parameters for a template from FE to BE. This simplifies FE and makes it less error-prone. Let's keep it. |
We still need this issue. Otherwise we leak business logic to site/frontend and CLI. |
Is anyone interested in writing an RFC for this? |
The goal of this issue is to research (and possibly implement) a new "prompt" API endpoint that determines parameter fields that coder UI/CLI must prompt for. Currently, we have the same logic rules encoded in UI and CLI, so there is a risk of having a mismatch.
There are different conditions under which specific fields should be displayed:
prompt_user
(will be added in the nearest future), one-time parameter valuesThere is also some coupling with legacy parameters, so ideally we should wait with implementation of this task until legacy params are removed
Slack thread:
The text was updated successfully, but these errors were encountered: