Commit d441b5e7 authored by Sean McGivern's avatar Sean McGivern

Merge branch 'rs-remove-usage-ping-pretty' into 'master'

Remove the `pretty` parameter for `usage_data`

Closes #36595

See merge request !14091
parents 8f763879 c7a77dce
...@@ -20,8 +20,7 @@ class Admin::ApplicationSettingsController < Admin::ApplicationController ...@@ -20,8 +20,7 @@ class Admin::ApplicationSettingsController < Admin::ApplicationController
def usage_data def usage_data
respond_to do |format| respond_to do |format|
format.html do format.html do
usage_data = Gitlab::UsageData.data usage_data_json = JSON.pretty_generate(Gitlab::UsageData.data)
usage_data_json = params[:pretty] ? JSON.pretty_generate(usage_data) : usage_data.to_json
render html: Gitlab::Highlight.highlight('payload.json', usage_data_json) render html: Gitlab::Highlight.highlight('payload.json', usage_data_json)
end end
......
...@@ -7,4 +7,4 @@ ...@@ -7,4 +7,4 @@
= succeed '.' do = succeed '.' do
= link_to 'application settings', admin_application_settings_path(anchor: 'usage-statistics') = link_to 'application settings', admin_application_settings_path(anchor: 'usage-statistics')
%pre.usage-data.js-syntax-highlight.code.highlight{ data: { endpoint: usage_data_admin_application_settings_path(format: :html, pretty: true) } } %pre.usage-data.js-syntax-highlight.code.highlight{ data: { endpoint: usage_data_admin_application_settings_path(format: :html) } }
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment