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

add ui schema property to scaffolder templates #79

Merged
merged 1 commit into from
Sep 2, 2024

Conversation

ydayagi
Copy link
Contributor

@ydayagi ydayagi commented Aug 26, 2024

@batzionb
Copy link
Contributor

batzionb commented Sep 1, 2024

Adding the ui:autofocus is something that currently the UI already does automatically for the first property of each step
Maybe a better example would be to add

ui:widget: textarea

@@ -7,7 +7,8 @@
"sampleText": {
"title": "A sample text",
"type": "string",
"description": "Sample text data"
"description": "Sample text data",
"ui:autofocus": true
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
"ui:autofocus": true
"ui:widget": "textarea"

@@ -38,6 +38,9 @@ instance of the ${{ values.workflowId }} workflow:
curl -v -X POST -H "Content-Type: application/json" http://localhost:8080/${{ values.workflowId }} -d@input.json
```

## Adding UI schema properties
You can add React UI schema properties to the input schema. See the 'ui:autofocus' example at [${{ values.artifactId }}-schema.json](./src/main/resources/schemas/${{ values.artifactId }}-schema.json ).
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
You can add React UI schema properties to the input schema. See the 'ui:autofocus' example at [${{ values.artifactId }}-schema.json](./src/main/resources/schemas/${{ values.artifactId }}-schema.json ).
You can add React UI schema properties to the input schema. See the 'ui:widget' example at [${{ values.artifactId }}-schema.json](./src/main/resources/schemas/${{ values.artifactId }}-schema.json ).

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

@batzionb
Copy link
Contributor

batzionb commented Sep 2, 2024

/lgtm

@batzionb
Copy link
Contributor

batzionb commented Sep 2, 2024

/approve

FLPATH-1615
https://issues.redhat.com/browse/FLPATH-1615

Signed-off-by: Yaron Dayagi <ydayagi@redhat.com>
Copy link

openshift-ci bot commented Sep 2, 2024

New changes are detected. LGTM label has been removed.

@ydayagi ydayagi changed the title add ui schema property to basic template add ui schema property to scaffolder templates Sep 2, 2024
@masayag masayag merged commit f382f1b into parodos-dev:main Sep 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants