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 Kamelets- schema generated forms & yaml/form switching #7344
Conversation
/cc @invincibleJai |
<EventSourceSection | ||
namespace={namespace} | ||
kameletSource={kameletSource} | ||
catalogFlow |
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.
we may not need to pass catalogFlow
, as there will be just one flow now. I'll work on it to get this cleaned across
@@ -167,6 +177,7 @@ export const EventSource: React.FC<Props> = ({ | |||
namespace={namespace} | |||
eventSourceStatus={eventSourceStatus} | |||
eventSourceMetaDescription={eventSourceMetaDescription} | |||
kameletSource={kameletSource} |
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.
do we need to pass as prop or shall we set formSchema in initial values if it's kamelet
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.
We need this. We can not set it in the initial value as it has been not used in the form.
frontend/packages/knative-plugin/src/components/add/EventSourcePage.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/knative-plugin/src/components/add/EventSourcePage.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/knative-plugin/src/components/add/EventSourcePage.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/knative-plugin/src/components/add/event-sources/EventSourceSection.tsx
Show resolved
Hide resolved
frontend/packages/knative-plugin/src/utils/create-eventsources-utils.ts
Outdated
Show resolved
Hide resolved
frontend/packages/knative-plugin/src/utils/create-eventsources-utils.ts
Outdated
Show resolved
Hide resolved
frontend/packages/knative-plugin/src/components/add/event-sources/EventSourceSection.tsx
Show resolved
Hide resolved
frontend/packages/knative-plugin/src/components/add/EventSourcePage.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/knative-plugin/src/components/add/EventSourcePage.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/knative-plugin/src/components/add/event-sources/EventSourceSection.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/knative-plugin/src/components/add/event-sources/EventSourceSection.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/knative-plugin/src/utils/create-eventsources-utils.ts
Outdated
Show resolved
Hide resolved
0521f51
to
27011dd
Compare
/lgtm Verified all 6 kamelets[0] , works as expected |
frontend/packages/knative-plugin/src/components/add/EventSourcePage.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/knative-plugin/src/components/add/EventSourceForm.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/knative-plugin/src/components/add/event-sources/EventSourceSection.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/knative-plugin/src/components/add/event-sources/EventSourceSection.tsx
Outdated
Show resolved
Hide resolved
/lgtm verified the changes !! |
/assign @rohitkrai03 |
frontend/packages/console-shared/src/components/formik-fields/DynamicFormField.tsx
Outdated
Show resolved
Hide resolved
/test analyze |
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.
/approve
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: invincibleJai, rohitkrai03, vikram-raj The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Fixes:
https://issues.redhat.com/browse/ODC-5063
Description:
when creating an source for kamelets, I should be presented with a form/yaml view and ability to switch between two
Screen shots / Gifs for design review:
Test setup: