-
Notifications
You must be signed in to change notification settings - Fork 178
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
Remove invalid jump link #3559
base: main
Are you sure you want to change the base?
Remove invalid jump link #3559
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #3559 +/- ##
==========================================
- Coverage 85.54% 85.17% -0.37%
==========================================
Files 1371 1382 +11
Lines 31190 31571 +381
Branches 8731 8824 +93
==========================================
+ Hits 26680 26892 +212
- Misses 4510 4679 +169
... and 318 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
7a04943
to
6f21f4a
Compare
It's ok to remove the the Instead of trying to do the clean up of data connections here, we should keep the code around until we tackle the story to remove it so that we can update tests at the same time. |
@christianvogt okay I'll restore the test. Would you like me to comment that part out then? Otherwise the tests fail. |
@ashley-o0o the test should not fail if all you're doing is removing the jump link |
ee0f61c
to
a738b57
Compare
a738b57
to
714da2a
Compare
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: christianvogt 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 |
Closes: RHOAIENG-16189
Description
Removes invalid jump link to Data connections in the create workbench page. In doing so, also cleaned up some unused data connections code.
How Has This Been Tested?
Tested locally on cluster.
Test Impact
No tests changed
Screenshots
Before:
After:
Request review criteria:
Self checklist (all need to be checked):
If you have UI changes:
After the PR is posted & before it merges:
main