-
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
🐛 Add a fallback for issue title #1273
Conversation
ibolton336
commented
Aug 9, 2023
- In the event of a blank ruleReport.description, fallback to use the name or a hardcoded string to prevent an app crash
119e8c4
to
6c87751
Compare
Codecov ReportPatch and project coverage have no change.
Additional details and impacted files@@ Coverage Diff @@
## main #1273 +/- ##
=======================================
Coverage 43.14% 43.14%
=======================================
Files 143 143
Lines 4299 4299
Branches 1000 1000
=======================================
Hits 1855 1855
Misses 2364 2364
Partials 80 80
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
...pages/issues/issue-detail-drawer/file-incidents-detail-modal/file-incidents-detail-modal.tsx
Outdated
Show resolved
Hide resolved
7aac028
to
4ca4ea4
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.
Looks good just a couple of suggestions
6caaeba
to
a47aa8d
Compare
Signed-off-by: ibolton336 <ibolton@redhat.com>
a47aa8d
to
c0c309c
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