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

UI + Workflow + Engine - Review Params + Config elements #421

Open
tlawrie opened this issue Oct 27, 2023 · 0 comments
Open

UI + Workflow + Engine - Review Params + Config elements #421

tlawrie opened this issue Oct 27, 2023 · 0 comments
Assignees

Comments

@tlawrie
Copy link
Member

tlawrie commented Oct 27, 2023

Map the types better

In the ParameterUtil class, everything maps to text. We need to use the InputType in the front end to map to the param types

export const InputType = Object.freeze({
Boolean: "boolean",
Email: "email",
Number: "number",
Password: "password",
Select: "select",
TextArea: "textarea",
Text: "text",
TextEditor: "texteditor",
TextEditorJs: "texteditor::javascript",
TextEditorText: "texteditor::text",
TextEditorShell: "texteditor::shell",
TextEditorYaml: "texteditor::yaml",
URL: "url",
});

Combine the elements

We currently have Params and Config as two separate elements, one for the UI and one for the system. We could potentially merge them into one with

Params: [{
  name:
  description
  type
  defaultValue
  config: {
    .. ui config ..
  }
}]
@tlawrie tlawrie changed the title UI + Workflow + Engine - revisit the Params + Config elements UI + Workflow + Engine - Review Params + Config elements Nov 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Ready
Development

No branches or pull requests

2 participants