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
fix: execution log crashes for logs with no uuid #13998
Conversation
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
Codecov Report
@@ Coverage Diff @@
## master #13998 +/- ##
==========================================
+ Coverage 79.02% 79.03% +0.01%
==========================================
Files 936 936
Lines 47408 47408
Branches 5938 5939 +1
==========================================
+ Hits 37465 37471 +6
+ Misses 9817 9811 -6
Partials 126 126
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
🏷️ 2021.13 |
* Fix execution log when no execution id * Use single quotes * Pretty (cherry picked from commit 5dd971e)
* Fix execution log when no execution id * Use single quotes * Pretty
* Fix execution log when no execution id * Use single quotes * Pretty
SUMMARY
An issue was introduced in #13752 where logs that were created before the
executionId
was created cause the execution log list view to give the following error:This PR fixes this issue, instead displaying "none" for logs without an
executionId
.