From f39219e6e5423d1540c170ed6c8d94456cf857a7 Mon Sep 17 00:00:00 2001 From: Molly Struve Date: Wed, 14 Oct 2020 15:33:33 -0500 Subject: [PATCH] [deploy] Optimization:Break up SiteConfig Updates by Groups (#10851) --- .../admin/configs/_form_submission.html.erb | 8 ++ app/views/admin/configs/show.html.erb | 97 ++++++++++++++++--- 2 files changed, 92 insertions(+), 13 deletions(-) create mode 100644 app/views/admin/configs/_form_submission.html.erb diff --git a/app/views/admin/configs/_form_submission.html.erb b/app/views/admin/configs/_form_submission.html.erb new file mode 100644 index 000000000..3edbd8a36 --- /dev/null +++ b/app/views/admin/configs/_form_submission.html.erb @@ -0,0 +1,8 @@ +<% if current_user.has_role?(:single_resource_admin, Config) %> +
+ <%= label_tag :confirmation_update %> + <%= text_field_tag :confirmation, nil, class: "form-control", placeholder: "Confirmation text", autocomplete: "off", required: true, pattern: @confirmation_text %> +
Type the sentence: <%= @confirmation_text %>
+ <%= f.submit "Update Site Configuration", class: "btn btn-danger btn-lg" %> +
+<% end %> diff --git a/app/views/admin/configs/show.html.erb b/app/views/admin/configs/show.html.erb index fb35ccc2a..cb4b9eabb 100644 --- a/app/views/admin/configs/show.html.erb +++ b/app/views/admin/configs/show.html.erb @@ -112,9 +112,12 @@ placeholder: Constants::SiteConfig::DETAILS[:video_encoder_key][:placeholder] %>
<%= Constants::SiteConfig::DETAILS[:video_encoder_key][:description] %>
+ <%= render "form_submission", f: f %> + <% end %> + <%= form_for(SiteConfig.new, url: admin_config_path) do |f| %>
<%= render partial: "card_header", locals: { @@ -313,9 +316,13 @@
Changing authentication keys will not take affect until this forem instance is restarted.
+ + <%= render "form_submission", f: f %>
+ <% end %> + <%= form_for(SiteConfig.new, url: admin_config_path) do |f| %>
<%= render partial: "card_header", locals: { @@ -379,9 +386,13 @@ placeholder: Constants::SiteConfig::DETAILS[:campaign_featured_tags][:placeholder] %>
<%= Constants::SiteConfig::DETAILS[:campaign_featured_tags][:description] %>
+ + <%= render "form_submission", f: f %> + <% end %> + <%= form_for(SiteConfig.new, url: admin_config_path) do |f| %>
<%= render partial: "card_header", locals: { @@ -444,9 +455,13 @@ placeholder: Constants::SiteConfig::DETAILS[:staff_user_id][:placeholder] %>
<%= Constants::SiteConfig::DETAILS[:staff_user_id][:description] %>
+ + <%= render "form_submission", f: f %> + <% end %> + <%= form_for(SiteConfig.new, url: admin_config_path) do |f| %>
<%= render partial: "card_header", locals: { @@ -485,9 +500,13 @@
<%= Constants::SiteConfig::DETAILS[:credit_prices_in_cents][:xlarge][:description] %>
<% end %> + + <%= render "form_submission", f: f %> + <% end %> + <%= form_for(SiteConfig.new, url: admin_config_path) do |f| %>
<%= render partial: "card_header", locals: { @@ -516,9 +535,13 @@ <% end %> <% end %>
+ + <%= render "form_submission", f: f %> + <% end %> + <%= form_for(SiteConfig.new, url: admin_config_path) do |f| %>
<%= render partial: "card_header", locals: { @@ -545,9 +568,13 @@ placeholder: Constants::SiteConfig::DETAILS[:periodic_email_digest_max][:placeholder] %>
<%= Constants::SiteConfig::DETAILS[:periodic_email_digest_max][:description] %>
+ + <%= render "form_submission", f: f %> + <% end %> + <%= form_for(SiteConfig.new, url: admin_config_path) do |f| %>
<%= render partial: "card_header", locals: { @@ -570,9 +597,13 @@ <%= f.check_box :display_jobs_banner, checked: SiteConfig.display_jobs_banner %>
<%= Constants::SiteConfig::DETAILS[:display_jobs_banner][:description] %>
+ + <%= render "form_submission", f: f %> + <% end %> + <%= form_for(SiteConfig.new, url: admin_config_path) do |f| %>
<%= render partial: "card_header", locals: { @@ -589,9 +620,13 @@ value: SiteConfig.ga_tracking_id %>
<%= Constants::SiteConfig::DETAILS[:ga_tracking_id][:description] %>
+ + <%= render "form_submission", f: f %> + <% end %> + <%= form_for(SiteConfig.new, url: admin_config_path) do |f| %>
<%= render partial: "card_header", locals: { @@ -615,9 +650,13 @@ value: SiteConfig.recaptcha_secret_key %>
<%= Constants::SiteConfig::DETAILS[:recaptcha_secret_key][:description] %>
+ + <%= render "form_submission", f: f %> + <% end %> + <%= form_for(SiteConfig.new, url: admin_config_path) do |f| %>
<%= render partial: "card_header", locals: { @@ -709,9 +748,13 @@ <%= SiteConfig.right_navbar_svg_icon.html_safe %>
+ + <%= render "form_submission", f: f %> + <% end %> + <%= form_for(SiteConfig.new, url: admin_config_path) do |f| %>
<%= render partial: "card_header", locals: { @@ -782,9 +825,13 @@ value: SiteConfig.mascot_image_description %>
<%= Constants::SiteConfig::DETAILS[:mascot_image_description][:description] %>
+ + <%= render "form_submission", f: f %> + <% end %> + <%= form_for(SiteConfig.new, url: admin_config_path) do |f| %>
<%= render partial: "card_header", locals: { @@ -808,9 +855,13 @@ <% end %> <% end %>
+ + <%= render "form_submission", f: f %> + <% end %> + <%= form_for(SiteConfig.new, url: admin_config_path) do |f| %>
<%= render partial: "card_header", locals: { @@ -855,9 +906,13 @@ placeholder: Constants::SiteConfig::DETAILS[:payment_pointer][:placeholder] %>
<%= Constants::SiteConfig::DETAILS[:payment_pointer][:description] %>
+ + <%= render "form_submission", f: f %> + <% end %> + <%= form_for(SiteConfig.new, url: admin_config_path) do |f| %>
<%= render partial: "card_header", locals: { @@ -898,9 +953,13 @@ value: SiteConfig.mailchimp_community_moderators_id %>
<%= Constants::SiteConfig::DETAILS[:mailchimp_community_moderators_id][:description] %>
+ + <%= render "form_submission", f: f %> + <% end %> + <%= form_for(SiteConfig.new, url: admin_config_path) do |f| %>
<%= render partial: "card_header", locals: { @@ -966,8 +1025,13 @@ placeholder: Constants::SiteConfig::DETAILS[:suggested_users][:placeholder] %>
<%= Constants::SiteConfig::DETAILS[:suggested_users][:description] %>
+ + <%= render "form_submission", f: f %> + <% end %> + + <%= form_for(SiteConfig.new, url: admin_config_path) do |f| %>
<%= render partial: "card_header", locals: { @@ -996,8 +1060,13 @@ placeholder: Constants::SiteConfig::DETAILS[:spam_trigger_terms][:placeholder] %>
<%= Constants::SiteConfig::DETAILS[:spam_trigger_terms][:description] %>
+ + <%= render "form_submission", f: f %> + <% end %> + + <%= form_for(SiteConfig.new, url: admin_config_path) do |f| %>
<%= render partial: "card_header", locals: { @@ -1029,9 +1098,13 @@ <% end %> <% end %>
+ + <%= render "form_submission", f: f %> + <% end %> + <%= form_for(SiteConfig.new, url: admin_config_path) do |f| %>
<%= render partial: "card_header", locals: { @@ -1049,8 +1122,13 @@ placeholder: Constants::SiteConfig::DETAILS[:sponsor_headline][:placeholder] %>
<%= Constants::SiteConfig::DETAILS[:sponsor_headline][:description] %>
+ + <%= render "form_submission", f: f %> + <% end %> + + <%= form_for(SiteConfig.new, url: admin_config_path) do |f| %>
<%= render partial: "card_header", locals: { @@ -1069,9 +1147,13 @@ pattern: "[a-z0-9,]+" %>
<%= Constants::SiteConfig::DETAILS[:sidebar_tags][:description] %>
+ + <%= render "form_submission", f: f %> + <% end %> + <%= form_for(SiteConfig.new, url: admin_config_path) do |f| %>
<%= render partial: "card_header", locals: { @@ -1122,21 +1204,10 @@ <%= f.check_box :public, checked: SiteConfig.public %>
Are most of the site pages (posts, profiles, etc. public?) — BE VERY CAUTIOUS IN CHANGING THIS
+ + <%= render "form_submission", f: f %> - <% if current_user.has_role?(:single_resource_admin, Config) %> -
-
Confirm and Submit
-
-
- <%= label_tag :confirmation %> - <%= text_field_tag :confirmation, nil, class: "form-control", placeholder: "Confirmation text", autocomplete: "off", required: true, pattern: @confirmation_text %> -
Type the sentence: <%= @confirmation_text %>
- <%= f.submit "Update Site Configuration", class: "btn btn-danger btn-lg" %> -
-
-
- <% end %> <% end %>