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 Broker details page #6945
Add Broker details page #6945
Conversation
2a0594c
to
19d42d3
Compare
b33959a
to
2f5daee
Compare
frontend/packages/knative-plugin/src/components/eventing/BrokerDetailsPage.tsx
Outdated
Show resolved
Hide resolved
2f5daee
to
4425bbe
Compare
frontend/packages/knative-plugin/src/components/eventing/BrokerDetailsPage.tsx
Outdated
Show resolved
Hide resolved
canCreate={false} | ||
{...props} | ||
kind={referenceForModel(EventingTriggerModel)} | ||
ListComponent={TriggerList} |
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.
you can use the fieldSelector field here to filter our the triggers based on the broker name
fieldSelector={`spec.broker=${<broker name>}`}
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.
fieldSelector
seems to have issue with spec.broker
, so will be going with current implementation.
Error Loading Triggers: field label not supported: spec.broker
if (customData?.broker && customData.broker === connectedBroker) { | ||
return null; |
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.
Don't need this if you use the fieldSelector
c0681c9
to
6617f23
Compare
4f222db
to
b7d9c34
Compare
thanks @rottencandy looks good, will verify it once |
b7d9c34
to
dc9da94
Compare
@rottencandy it seems like there might be some weird alignment issues with a few of the columns – other than that small fix it looks good to me! |
1b81ac4
to
3ba796d
Compare
3ba796d
to
1704c5f
Compare
Rebased. |
/lgtm Verified the changes , works as expected |
/retest |
/test analyze |
/retest |
5 similar comments
/retest |
/retest |
/retest |
/retest |
/retest |
/retest Please review the full test history for this PR and help us cut down flakes. |
1704c5f
to
950871a
Compare
Rebased. |
950871a
to
d718da0
Compare
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: invincibleJai, rottencandy, sahil143 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 |
/retest |
/retest Please review the full test history for this PR and help us cut down flakes. |
Fixes:
https://issues.redhat.com/browse/ODC-4994
https://issues.redhat.com/browse/ODC-5062
Description:
Add a details page for Broker.
Add a list page for Triggers.
Screen shots / Gifs for design review:
@openshift/team-devconsole-ux @invincibleJai
Browser conformance: