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

[WIP] fix create service webview #2857

Closed

Conversation

datho7561
Copy link
Contributor

Fixes:

  • hide the commands from palette (they don't work from palette)
  • replace odo application list with a version that uses oc
  • mui5 -ify the create service form
  • Add a Patternfly-like dropdown for nested objects, it's collapsed by default.

Still broken:

  • When I submit, it adds all the booleans (check boxes) into the formData, even the ones that the user hasn't interacted with. This results in invalid JSON for the custom resource, so the call to create the service fails.
  • form could use a few more UI clean ups

Signed-off-by: David Thompson davthomp@redhat.com

Fixes:
- hide the commands from palette (they don't work from palette)
- replace `odo application list` with a version that uses `oc`
- mui5 -ify the create service form
- Add a Patternfly-like dropdown for nested objects, it's collapsed by
  default.

Still broken:
- When I submit, it adds all the booleans (check boxes) into the
  formData, even the ones that the user hasn't interacted with.
  This results in invalid JSON for the custom resource,
  so the call to create the service fails.
- form could use a few more UI clean ups

Signed-off-by: David Thompson <davthomp@redhat.com>
@datho7561
Copy link
Contributor Author

Broken part might be related to rjsf-team/react-jsonschema-form#675, still need to investigate further

@datho7561
Copy link
Contributor Author

Replaced by #3435

@datho7561 datho7561 closed this Oct 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant