-
Notifications
You must be signed in to change notification settings - Fork 83
Return CSV if requested from realm stats. #922
Conversation
/hold hold. I'm going to add a link from the realm stats page. |
b984b2e
to
623efbe
Compare
/unhold |
/hold while I write the JSON approach back in. |
623efbe
to
f5fa290
Compare
/unhold |
97ef955
to
820d635
Compare
@@ -23,7 +23,16 @@ <h1>Realm stats</h1> | |||
</p> | |||
|
|||
<div class="card mb-3 shadow-sm"> | |||
<div class="card-header">Statistics</div> | |||
<div class="card-header">Statistics |
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'm wondering what this would look like more like this:
https://gist.github.com/whaught/7ad634843cc4de808796bdce5cdd8a33
just to get things out of the 'card-header' div and have an all-encompassing body div
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.
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
Let's go with this, we can tweak later
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
/hold
in case @whaught wants to re-review
/unhold |
Supports 4 modes: ${SERVER}/realm/stats.csv ${SERVER}/realm/stats.csv?user ${SERVER}/realm/stats.json ${SERVER}/realm/stats.json?user Fixes google#916
820d635
to
37a4088
Compare
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jeremyfaller, sethvargo, whaught The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Supports 2 modes:
${SERVER}/realm/stats?csv
Returns overall realm stats.
${SERVER}/realm/stats?csv&user
Returns the per-user realm stats.
Fixes #916