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
Logs: Add Download logs
button to log-browser
#55163
Logs: Add Download logs
button to log-browser
#55163
Conversation
Drone build failed: https://drone.grafana.net/grafana/grafana-enterprise/33388 |
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 great!
reportInteraction('grafana_logs_download_logs_clicked', { | ||
app, | ||
format: 'logs', | ||
}); | ||
const logsModel = dataFrameToLogsModel(data || [], undefined); |
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.
Great cleanup.
Downloads
button to log browserDownloads
button to log browser
Drone build failed: https://drone.grafana.net/grafana/grafana-enterprise/33427 |
@@ -173,6 +173,7 @@ | |||
"babel-loader": "8.2.5", | |||
"babel-plugin-angularjs-annotate": "0.10.0", | |||
"babel-plugin-macros": "3.1.0", | |||
"blob-polyfill": "7.0.20220408", |
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.
I added that in order to add methods to the Blob
object, which seem to be missing on JSDom.
Ref jsdom/jsdom#2555
Drone build failed: https://drone.grafana.net/grafana/grafana-enterprise/33839 |
…nloads-button-more-prominent
Downloads
button to log browserDownloads
button to log browser
Downloads
button to log browserDownload logs
button to log log-browser
Drone build failed: https://drone.grafana.net/grafana/grafana-enterprise/35521 |
Download logs
button to log log-browserDownload logs
button to log-browser
WIP:
This is still work in progress as it
What this PR does / why we need it:
In #50188 we decided to add a new
Downloads
button to the logs browser. This browser supports only txt-format for now.Which issue(s) this PR fixes:
Fixes #50188
Preview of the feature:
Screen.Recording.2022-09-14.at.14.06.22.mov