-
Notifications
You must be signed in to change notification settings - Fork 43
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
🐛 Adjust custom target edit to reflect api changes #1489
Conversation
3a16c34
to
bbbb61e
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1489 +/- ##
=======================================
Coverage 40.29% 40.29%
=======================================
Files 143 143
Lines 4532 4532
Branches 1096 1096
=======================================
Hits 1826 1826
Misses 2609 2609
Partials 97 97
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
bd94ea5
to
43f9d4a
Compare
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
One suggestion for the tooltip, but it isn't blocking.
tooltipMessage={ | ||
"Cannot delete credential assigned to a JIRA tracker." | ||
"Cannot delete credential as it is currently in use." |
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.
Suggested improvement: Add the name of the thing that is using the credential so it is easy to find.
"Credential is in use by application Foxtrot Oscar and cannot be deleted."
Signed-off-by: ibolton336 <ibolton@redhat.com>
Signed-off-by: ibolton336 <ibolton@redhat.com>
Signed-off-by: ibolton336 <ibolton@redhat.com>
891d936
to
5ac2206
Compare
https://issues.redhat.com/browse/MTA-1542
https://issues.redhat.com/browse/MTA-1543