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
adds support for sinkBinding Source #4829
adds support for sinkBinding Source #4829
Conversation
5612f65
to
3b8f130
Compare
3b8f130
to
198686f
Compare
Thanks @debsmita1 , have updated |
198686f
to
4a8f054
Compare
/kind feature |
In a new namespace there will not be any D/DC/KnativeService, in this case if user selectes event source option in add flow, there is no way to submit the form as knative service field is required.
|
4a8f054
to
660d056
Compare
@karthikjeeyar yes we need to show some info to user in this scenario , will file an issue to track and follow up with UXD |
})); | ||
describe('SinkBindingSection', () => { | ||
it('should render FormSection', () => { | ||
const wrapper = shallow(<SinkBindingSection />); |
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.
nit: we could have this in beforeEach
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.
updated
yes i saw it too and addressed in #4810 |
660d056
to
5e3c173
Compare
Verified locally, works as expected |
thanks @karthikjeeyar , will add hold for verification from UXD |
/hold for approval from UXD , @serenamarie125 PTAL |
General comment @invincibleJai - we will not have final text available for labels/help text at this time, so please don't worry about updating those now. We will have a follow up design story to work on that. |
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.
LGTM @invincibleJai !
I think we may have follow up work based on UX being clarified, but this looks like a great initial iteration!
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: invincibleJai, karthikjeeyar, serenamarie125 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 |
/hold cancel thanks @serenamarie125 |
/retest |
/test e2e-gcp-console |
/retest Please review the full test history for this PR and help us cut down flakes. |
Fixes:
https://issues.redhat.com/browse/ODC-2659
Analysis / Root cause:
User can't create SinkBinding source via web console
Solution Description:
Adds support for sinkBinding source via web console
Screen shots / Gifs for design review:
cc @serenamarie125
Unit test coverage report:
Browser conformance: