Rework promotion batches views

spree · Dec 4, 2023 · c01af53 · c01af53
1 parent 0058fed
commit c01af53
Show file tree

Hide file tree

Showing 5 changed files with 31 additions and 17 deletions.
diff --git a/app/views/spree/admin/promotion_batches/_form.html.erb b/app/views/spree/admin/promotion_batches/_form.html.erb
@@ -0,0 +1,6 @@
+<div data-hook="new_promotion_batch_template_promotion" class="col-12 col-md-4">
+  <%= f.field_container :template_promotion do %>
+    <%= f.label :template_promotion_id, Spree.t(:template_promotion) %>
+    <%= f.collection_select :template_promotion_id, Spree::Promotion.all, :id, :name, { include_blank: true }, { class: 'select2-clear w-100' } %>
+  <% end %>
+</div>
diff --git a/app/views/spree/admin/promotion_batches/edit.html.erb b/app/views/spree/admin/promotion_batches/edit.html.erb
@@ -0,0 +1,21 @@
+<% content_for :page_title do %>
+  <%= link_to Spree.t(:promotion_batches), admin_promotion_batches_path %> /
+  <%= @promotion_batch.id %>
+<% end %>
+
+<div class="card">
+  <div class="card-body">
+    <%= form_tag csv_import_admin_promotion_batch_path, method: :post, multipart: true do %>
+      <%= file_field_tag :file, accept: ".csv" %>
+      <%= submit_tag 'Import codes from CSV' %>
+    <% end %>
+  </div>
+</div>
+
+<%= form_for @promotion_batch, url: object_url, method: :put do |f| %>
+  <div class="card mb-4">
+    <div class="card-body">
+      <%= render partial: 'spree/admin/shared/edit_resource_links' %>
+    </div>
+  </div>
+<% end %>
diff --git a/app/views/spree/admin/promotion_batches/index.html.erb b/app/views/spree/admin/promotion_batches/index.html.erb
@@ -35,7 +35,8 @@
             <td><%= link_to promotion_batch.template_promotion.name, spree.edit_admin_promotion_path(promotion_batch.template_promotion) if promotion_batch.template_promotion %></td>
             <td class="actions" data-hook="admin_promotion_batches_index_row_actions">
               <span class="d-flex justify-content-end">
-               <%= link_to_delete promotion_batch, no_text: true if can?(:delete, promotion_batch) %>
+                <%= link_to_edit promotion_batch, no_text: true if can?(:edit, promotion_batch) %>
+                <%= link_to_delete promotion_batch, no_text: true if can?(:delete, promotion_batch) %>
               </span>
             </td>
           </tr>

diff --git a/app/views/spree/admin/promotion_batches/new.html.erb b/app/views/spree/admin/promotion_batches/new.html.erb
@@ -1,17 +1,12 @@
 <% content_for :page_title do %>
-  <%= link_to Spree.t(:promotion_batch), admin_promotion_batches_path %> /
+  <%= link_to Spree.t(:promotion_batches), admin_promotion_batches_path %> /
   <%= Spree.t(:new_promotion_batch) %>
 <% end %>
 
 <div class="card">
   <div class="card-body">
     <%= form_for :promotion_batch, url: collection_url do |f| %>
-      <div data-hook="new_promotion_batch_template_promotion" class="col-12 col-md-4">
-        <%= f.field_container :template_promotion do %>
-          <%= f.label :template_promotion_id, Spree.t(:template_promotion) %>
-          <%= f.collection_select :template_promotion_id, Spree::Promotion.all, :id, :name, { include_blank: true }, { class: 'select2-clear w-100' } %>
-        <% end %>
-      </div>
+      <%= render partial: 'form', locals: { f: f } %>
       <%= render partial: 'spree/admin/shared/new_resource_links' %>
     <% end %>
   </div>

diff --git a/app/views/spree/admin/promotion_batches/show.html.erb b/app/views/spree/admin/promotion_batches/show.html.erb
@@ -15,15 +15,6 @@
   <% end %>
 <% end %>
 
-<div class="card">
-  <div class="card-body">
-    <%= form_tag csv_import_admin_promotion_batch_path, method: :post, multipart: true do %>
-      <%= file_field_tag :file, accept: ".csv" %>
-      <%= submit_tag 'Import codes from CSV' %>
-    <% end %>
-  </div>
-</div>
-
 <table class="table">
   <thead class="text-muted">
     <tr>