-
Notifications
You must be signed in to change notification settings - Fork 73
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
Fixes #15713: Show log informations next to reports and full compliance report #2501
Conversation
PR rebased |
7cd18fc
to
6c9bbaa
Compare
PR rebased |
1 similar comment
PR rebased |
84001a8
to
4358be6
Compare
Commit modified |
1 similar comment
Commit modified |
04aaae3
to
4e33cb9
Compare
Commit modified |
@@ -217,6 +219,7 @@ case class ReportLine ( | |||
|
|||
JsObj( | |||
( "executionDate", executionDate.toString("yyyy-MM-dd HH:mm:ss") ) | |||
, ( "runDate", runDate.toString("yyyy-MM-dd HH:mm:ss") ) |
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.
Can you use either ISODateTimeFormater .dateTime or our date formatter service with "getDisplayDate" ? We need at least timezone and it's more consistant that way
PR rebased |
4e33cb9
to
63913fd
Compare
Commit modified |
63913fd
to
7e60c46
Compare
When we first come to the page, all event logs are loaded, which leads to a very bad experience (huge latency...). |
bad PR, sorry ! |
OK, merging this PR |
https://issues.rudder.io/issues/15713
Replacing previous PR: #2493