Add Contact information section on stock locations form

Author: chaimann

admin/app/components/solidus_admin/stock_locations/form/component.html.erb

@@ -12,7 +12,11 @@
       <% end %>
 
       <%= render component('ui/panel').new(title: t('.address')) do %>
-        <%= render component('ui/forms/address').new(addressable: @stock_location, form_field_name: 'stock_location', fieldset: :location, excludes: :email) %>
+        <%= render component('ui/forms/address').new(addressable: @stock_location, form_field_name: 'stock_location', fieldset: :location, excludes: %i[phone email]) %>
+      <% end %>
+
+      <%= render component('ui/panel').new(title: t('.contact_info')) do %>
+        <%= render component('ui/forms/address').new(addressable: @stock_location, form_field_name: 'stock_location', fieldset: :contact_info) %>
       <% end %>
     <% end %>
 

admin/app/components/solidus_admin/stock_locations/form/component.yml

@@ -4,6 +4,7 @@ en:
   availability: "Availability"
   backorderable_default: "Backorderable default"
   check_stock_on_transfer: "Check stock on transfer"
+  contact_info: "Contact information"
   default: "Set as default for all products"
   fulfillable: "Fulfillable"
   hints:

admin/app/controllers/solidus_admin/stock_locations_controller.rb

@@ -18,6 +18,7 @@ def permitted_resource_params
         :state_name,
         :state_id,
         :phone,
+        :email,
         :active,
         :default,
         :backorderable_default,

admin/spec/features/stock_locations/form_spec.rb

@@ -65,6 +65,7 @@
           solidus_select "Canada", from: "Country"
           solidus_select "Fictitious State in CA", from: "State"
           fill_in "Phone", with: "123123123"
+          fill_in "Email", with: "example@example.com"
           check "Set as default for all products"
           check "Backorderable default"
 
@@ -86,6 +87,7 @@
           expect(solidus_select_control("Country")).to have_content("Canada")
           expect(solidus_select_control("State")).to have_content("Fictitious State in CA")
           expect(find_field("Phone").value).to eq "123123123"
+          expect(find_field("Email").value).to eq "example@example.com"
           expect(checkbox("Active")).to be_checked
           expect(checkbox("Propagate all variants")).to be_checked
           expect(checkbox("Restock inventory")).to be_checked