-
Notifications
You must be signed in to change notification settings - Fork 10
Update template.ts to include Layout schema #15
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
base: master
Are you sure you want to change the base?
Changes from 4 commits
fd76fc4
6193a8c
225b076
26598c7
26b3a7c
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,42 @@ | ||
| export interface UiConfig { | ||
| /** | ||
| * CEL expression to add dynamic behaviour. | ||
| * This input will be displayed only when this condition is true. | ||
| * All inputs in this template are available to reference here. | ||
| */ | ||
| visible?: string | ||
|
|
||
| /** | ||
| * Component defines the form element that should be used to | ||
| * override the default renderer for the input. | ||
| */ | ||
| component?: "dropdown" | "text" | "number" | "date" | "datetime" | "select" | string; | ||
|
|
||
| /** | ||
| * Autofocus configures the form element autofocus attribute. | ||
| */ | ||
| autofocus?: boolean; | ||
|
|
||
| /** | ||
| * Placeholder configures the form element placeholder attribute. | ||
| */ | ||
| placeholder?: string; | ||
|
|
||
| /** | ||
| * Tooltip configures the form element alt attribute. | ||
| */ | ||
| tooltip?: string; | ||
|
|
||
| /** | ||
| * Types of values allowed for this input. | ||
| * Fixed: User must enter value when configuring the pipeline | ||
| * Runtime: User must enter value when running the pipeline | ||
| * Expression: Value will be derived by evaluating this CEL / JEXL expression | ||
| */ | ||
| allowedValueTypes?: Array<'fixed' | 'runtime' | 'expression'> | ||
|
|
||
| /** | ||
| * Options to populate the dropdown if the `component` is "select" | ||
| */ | ||
| options?: string[] | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. options is also available in input interface outside of ui
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @shaurya-harness i've added this here because the UI code supports it here today. Ideally yes, we should remove this and only use the |
||
| } | ||
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.
enum and options are duplicated
also shall we standardize enum / list / options - all seem to be doing the same thing