-
Notifications
You must be signed in to change notification settings - Fork 22
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
[MTV-1880] Fix 'Project' name select issues #1433
base: main
Are you sure you want to change the base?
Conversation
c663a63
to
d93736c
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## main #1433 +/- ##
==========================================
- Coverage 36.81% 36.20% -0.62%
==========================================
Files 158 159 +1
Lines 2548 2580 +32
Branches 599 618 +19
==========================================
- Hits 938 934 -4
- Misses 1428 1464 +36
Partials 182 182 ☔ View full report in Codecov by Sentry. |
d93736c
to
5b5f5d5
Compare
…1 of migration plan wizard Signed-off-by: Jeff Puzzo <[email protected]>
5b5f5d5
to
1dd376a
Compare
Quality Gate passedIssues Measures |
Hey @jpuzz0 Do you know if we have a JIRA ticket for the MTV project? |
@jpuzz0 Can you please also breakdown this PR into small PRs? Much easy to review and verify by QE when the PR is small and related to one issue... Thanks! |
@metalice |
@sgratch Sure, I'll break it down into 2 PRs. I'll keep this one just for "Project name" fixes and break out the "Plan name" feature. |
Issue: https://issues.redhat.com/browse/MTV-1880
Design issue: https://issues.redhat.com/browse/HPUX-259
Summary
NOTE: All comments made here were addressed with the exception of #5 to rename the details "Namespace" label (was suggested to update to be named "Project). This is still being discussed as to what we might change the tooltip text to, since there are many references to the term "Namespace".
Demo
https://github.com/user-attachments/assets/7c95fbeb-1018-45aa-8984-0b79f3a12bce
Empty provider state