-
Notifications
You must be signed in to change notification settings - Fork 163
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
[RHOAIENG-10312] Connection type table view #3067
[RHOAIENG-10312] Connection type table view #3067
Conversation
de18c50
to
9e9c333
Compare
d033eeb
to
58cbe32
Compare
Everything is looking good to me /lgtm |
/lgtm |
/approve |
58cbe32
to
749d1fa
Compare
frontend/src/__tests__/cypress/cypress/tests/mocked/connectionTypes/connectionTypes.cy.ts
Outdated
Show resolved
Hide resolved
f29765b
to
97b3150
Compare
frontend/src/pages/pipelines/global/experiments/executions/ExecutionsTableToolbar.tsx
Outdated
Show resolved
Hide resolved
frontend/src/pages/connectionTypes/create/DuplicateConnectionTypePage.tsx
Outdated
Show resolved
Hide resolved
97b3150
to
70bc9a9
Compare
70bc9a9
to
c522a89
Compare
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: christianvogt, manosnoam 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 |
Closes RHOAIENG-10312
Description
Adds the initial version of the Connection Types view.
How Has This Been Tested?
Login as a product admin, set the devFeaturesFlag on and then uncheck
disableConnectionTypes
, navigate to Settings -> Connection typesTest Impact
Added cypress e2e tests
Screenshots
Request review criteria:
Self checklist (all need to be checked):
If you have UI changes:
/cc @simrandhaliw