spree_cm_commissioner 2.3.1.pre.pre2 → 2.3.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 26be0034848358f7b5de1d490862a398ed97a3078c8cf2fe57df4d7e6e3be5a9
-  data.tar.gz: 37a918ae7deb8ea489361e890effc51c8b8534ab469cc04594f37495688e4ace
+  metadata.gz: 87608fab14f473c498987cb8ecf0793926b031a1368d9a1d03f05fc3bf03e819
+  data.tar.gz: 9472eaf172e1346757684e9c2b69cec671053927d1a99739f13d0eea1fcb17ce
 SHA512:
-  metadata.gz: 8d6a2ddde23786c4a5bbc639d386d6337f8de52ddabd8b7da9149718d1ee61d7cf5b87ba2d0bfc63cf87f88abf9789adfd80465993c16a34a46f476045c79e22
-  data.tar.gz: 6d06c0ea1b09c4e957d5b2c40bb6cdc5cc2eab7f2a93a5e61bd5efd45e559c9fb0f2a354c5221308432bdeb6fc43f4ab164ca9205e339a595bedd7d9b9b14145
+  metadata.gz: b3da7fbad39575ce43d6b39e088c24ec878095d88a6652e4dd07d87ae0b474403ef368dba1233e0a2d3a2084a03c45d1932c20eee21ee6a65f99e0cb8e4d1e1d
+  data.tar.gz: 92f4f8a8a446c765568b23b8a1183f7f2f83a499a608a59a34ab315314bf4ea256fbb8264da0da2e80ad0dab527a9fed4c8498a9c77342e186f61386dfad735b

data/Gemfile.lock

@@ -34,7 +34,7 @@ GIT
 PATH
   remote: .
   specs:
-    spree_cm_commissioner (2.3.1.pre.pre2)
+    spree_cm_commissioner (2.3.1)
       activerecord-multi-tenant
       activerecord_json_validator (~> 2.1, >= 2.1.3)
       aws-sdk-cloudfront

data/app/controllers/spree/admin/product_completion_steps_controller.rb

@@ -4,7 +4,6 @@ module Spree
       belongs_to 'spree/product', find_by: :slug
 
       before_action :load_step_types, if: :member_action?
-      before_action :build_assets, only: %i[create update]
 
       def load_step_types
         @step_types = [
@@ -15,13 +14,10 @@ module Spree
 
       # override
       def permitted_resource_params
-        return @permitted_resource_params if defined?(@permitted_resource_params)
-
         key = ActiveModel::Naming.param_key(@object)
         permit_keys = params.require(key).keys
 
-        @permitted_resource_params = params.require(key).permit(permit_keys)
-        @permitted_resource_params
+        params.require(key).permit(permit_keys)
       end
 
       # @overrided
@@ -58,18 +54,6 @@ module Spree
       def location_after_save
         edit_object_url(@object)
       end
-
-      def remove_banner
-        @object.banner&.destroy
-        flash[:success] = Spree.t(:successfully_removed)
-        redirect_to edit_object_url(@object)
-      end
-
-      private
-
-      def build_assets
-        @object.build_banner(attachment: permitted_resource_params.delete(:banner)) if permitted_resource_params[:banner].present?
-      end
     end
   end
 end

data/app/controllers/spree/admin/tenants_controller.rb

@@ -1,7 +1,7 @@
 module Spree
   module Admin
     class TenantsController < Spree::Admin::ResourceController
-      before_action :load_vector_icons, only: %i[new edit create update]
+      before_action :load_vector_icons, only: %i[new edit]
 
       # override
       def collection

data/app/controllers/spree/api/v2/storefront/intercity_taxi/draft_orders_controller.rb

@@ -20,6 +20,7 @@ module Spree
 
             def update
               order = SpreeCmCommissioner::IntercityTaxiOrder::Update.call(
+                order_number: params[:id],
                 params: params
               )
 

data/app/interactors/spree_cm_commissioner/create_ticket.rb

@@ -48,8 +48,6 @@ module SpreeCmCommissioner
         name: 'ticket-type',
         presentation: 'Ticket Type'
       )
-      # auto assign the option type to the ticket
-      @ticket.option_types << @option_type unless @ticket.option_types.include?(@option_type)
 
       @option_value = Spree::OptionValue.find_or_create_by!(
         name: @ticket.name,

data/app/interactors/spree_cm_commissioner/host_matcher.rb

@@ -22,7 +22,7 @@ module SpreeCmCommissioner
     end
 
     def find_tenant(normalized_host)
-      SpreeCmCommissioner::Tenant.find_by(host: normalized_host)
+      SpreeCmCommissioner::Tenant.find_by(host: "https://#{normalized_host}")
     end
   end
 end

data/app/interactors/spree_cm_commissioner/vendor_creation_telegram_alert_sender.rb

@@ -22,7 +22,7 @@ module SpreeCmCommissioner
     end
 
     def admin_chat_id
-      Spree::Store.default&.preferred_telegram_new_vendor_alert_chat_id
+      ENV.fetch('EXCEPTION_NOTIFIER_TELEGRAM_CHANNEL_ID', nil)
     end
   end
 end

data/app/mailers/spree/order_mailer_decorator.rb

@@ -37,21 +37,22 @@ module Spree
 
     def setup_tenant_and_store
       @tenant = @order.tenant
-      @current_store = @order.store
-      return if @tenant.blank?
-
-      @brand_color = @tenant.preferences[:brand_primary_color]
-      @vendor_logo_url = @tenant.active_vendor&.logo&.original_url
+      if @tenant.present?
+        @brand_color = @tenant.preferences[:brand_primary_color]
+        @vendor_logo_url = @tenant.active_vendor&.logo&.original_url
+        @current_store = @tenant
+      else
+        @current_store = @order.store
+      end
     end
 
     def build_subject(resend)
       prefix = resend ? "[#{Spree.t(:resend).upcase}] " : ''
-      store_name = @tenant.present? ? @tenant.name : @current_store&.name
-      "#{prefix}#{store_name} Booking Confirmation ##{@order.number}"
+      "#{prefix}#{@current_store&.name} Booking Confirmation ##{@order.number}"
     end
 
     def store_url
-      @tenant.present? ? @tenant.url : @current_store.url
+      @tenant.present? ? @current_store.host : @current_store.url
     end
 
     def ticket_email(guest, email)

data/app/models/concerns/spree_cm_commissioner/service_recommendations.rb

@@ -50,9 +50,9 @@ module SpreeCmCommissioner
       private
 
       def tenant_link_or_default(tenant, default_link)
-        return default_link unless tenant.respond_to?(:url)
+        return default_link unless tenant.respond_to?(:host)
 
-        site = tenant.url.to_s.strip
+        site = tenant.host.to_s.strip
         site.presence || default_link
       rescue StandardError
         default_link

data/app/models/concerns/spree_cm_commissioner/store_preference.rb

@@ -6,7 +6,6 @@ module SpreeCmCommissioner
       preference :sms_sender_id, :string
       preference :telegram_order_alert_chat_id, :string
       preference :telegram_order_request_alert_chat_id, :string
-      preference :telegram_new_vendor_alert_chat_id, :string
       preference :assetlinks, :string, default: ''
       preference :apple_app_site_association, :string, default: ''
     end

data/app/models/spree_cm_commissioner/product_completion_step.rb

@@ -4,8 +4,6 @@ module SpreeCmCommissioner
 
     belongs_to :product, class_name: '::Spree::Product', optional: false
 
-    has_one :banner, as: :viewable, dependent: :destroy, class_name: 'SpreeCmCommissioner::ProductCompletionStepBanner'
-
     # When a completion step is changed, regenerate completion steps for all line items
     # of the product so they reflect the latest step configuration.
     after_destroy :regenerate_line_items_completion_steps
@@ -30,8 +28,7 @@ module SpreeCmCommissioner
         action_label: action_label,
         action_url: action_url_for(line_item),
         completed_at: existing_data&.dig('completed_at'),
-        completed: completed?(line_item),
-        banner_url: banner.present? ? banner.original_url : nil
+        completed: completed?(line_item)
       }
     end
 

data/app/models/spree_cm_commissioner/product_completion_steps/social_entry_url.rb

@@ -5,7 +5,6 @@ module SpreeCmCommissioner
       # - https://t.me/ThePlatformKHBot?start={order.number}
       # - https://example.com?guest_name={guests[0].first_name}&phone={guests[0].phone_number}
       # - https://example.com?line_item_id={line_item.id}&quantity={line_item.quantity}
-      # - https://example.com?bill_address.first_name={bill_address.first_name}&bill_address.lastname={bill_address.lastname}
       preference :entry_point_link, :string
 
       # Allowed fields per object (excludes private/reference fields like *_id, *_by_id, metadata, etc.)
@@ -15,11 +14,7 @@ module SpreeCmCommissioner
         state
         email
         total
-        display_total
         item_total
-        display_item_total
-        adjustment_total
-        display_adjustment_total
         created_at
         completed_at
         token
@@ -28,31 +23,12 @@ module SpreeCmCommissioner
         currency
         item_count
         channel
-        special_instructions
-      ].freeze
-
-      BILL_ADDRESS_ALLOWED_FIELDS = %w[
-        first_name
-        last_name
-        full_name
-        address1
-        address2
-        city
-        zipcode
-        phone
-        state_name
-        alternative_phone
-        company
-        label
-        age
-        gender
       ].freeze
 
       LINE_ITEM_ALLOWED_FIELDS = %w[
         qr_data
         quantity
         price
-        display_price
         from_date
         to_date
         number
@@ -69,6 +45,7 @@ module SpreeCmCommissioner
         dob
         gender
         age
+        email
         phone_number
         address
         seat_number
@@ -83,6 +60,9 @@ module SpreeCmCommissioner
         expectation
         other_occupation
         other_organization
+        entry_type
+        upload_later
+        data_fill_stage_phase
         created_at
         updated_at
       ].freeze
@@ -92,79 +72,32 @@ module SpreeCmCommissioner
         return nil if preferred_entry_point_link.blank?
 
         url = preferred_entry_point_link.dup
-        url = replace_order_placeholders(url, line_item)
-        url = replace_bill_address_placeholders(url, line_item)
-        url = replace_line_item_placeholders(url, line_item)
-
-        replace_guest_placeholders(url, line_item)
-      end
-
-      def completed?(line_item)
-        return false if line_item.completion_steps.blank?
-
-        step_data = line_item.completion_steps.find { |step| step['position'].to_s == position.to_s }
-        step_data&.dig('completed_at').present?
-      end
-
-      # Returns a hash of all supported placeholder fields grouped by object type.
-      # Used by the admin UI to display available placeholders when configuring the entry_point_link template.
-      # Example usage in template: {order.number}, {bill_address.first_name}, {line_item.quantity}, {guests[0].email}
-      #
-      # @return [Hash] Hash with keys :order, :bill_address, :line_item, :guests
-      #                Each key maps to an array of allowed field names for that object type
-      def supported_fields
-        {
-          order: ORDER_ALLOWED_FIELDS,
-          bill_address: BILL_ADDRESS_ALLOWED_FIELDS,
-          line_item: LINE_ITEM_ALLOWED_FIELDS,
-          guests: GUEST_ALLOWED_FIELDS
-        }
-      end
-
-      private
 
-      def replace_order_placeholders(url, line_item)
-        return url if line_item.order.blank?
-
-        ORDER_ALLOWED_FIELDS.each do |field|
-          value = line_item.order.public_send(field)
-          url.gsub!("{order.#{field}}", value.to_s) if value.present?
+        # Replace order placeholders: {order.field_name}
+        line_item.order.attributes.each do |field, value|
+          url.gsub!("{order.#{field}}", value.to_s) if ORDER_ALLOWED_FIELDS.include?(field)
         end
 
-        url
-      end
-
-      def replace_bill_address_placeholders(url, line_item)
-        return url if line_item.order&.bill_address.blank?
-
-        BILL_ADDRESS_ALLOWED_FIELDS.each do |field|
-          value = line_item.order.bill_address.public_send(field)
-          url.gsub!("{bill_address.#{field}}", value.to_s) if value.present?
+        # Replace line_item placeholders: {line_item.field_name}
+        line_item.attributes.each do |field, value|
+          url.gsub!("{line_item.#{field}}", value.to_s) if LINE_ITEM_ALLOWED_FIELDS.include?(field)
         end
 
-        url
-      end
-
-      def replace_line_item_placeholders(url, line_item)
-        LINE_ITEM_ALLOWED_FIELDS.each do |field|
-          value = line_item.public_send(field)
-          url.gsub!("{line_item.#{field}}", value.to_s) if value.present?
+        # Replace guest placeholders: {guests[index].field_name}
+        line_item.guests.each_with_index do |guest, index|
+          guest.attributes.each do |field, value|
+            url.gsub!("{guests[#{index}].#{field}}", value.to_s) if GUEST_ALLOWED_FIELDS.include?(field)
+          end
         end
 
         url
       end
 
-      def replace_guest_placeholders(url, line_item)
-        return url if line_item.guests.blank?
-
-        line_item.guests.each_with_index do |guest, index|
-          GUEST_ALLOWED_FIELDS.each do |field|
-            value = guest.public_send(field)
-            url.gsub!("{guests[#{index}].#{field}}", value.to_s) if value.present?
-          end
-        end
+      def completed?(line_item)
+        return false if line_item.completion_steps.blank?
 
-        url
+        step_data = line_item.completion_steps.find { |step| step['position'].to_s == position.to_s }
+        step_data&.dig('completed_at').present?
       end
     end
   end

data/app/models/spree_cm_commissioner/tenant.rb

@@ -12,10 +12,7 @@ module SpreeCmCommissioner
                                       through: :vendors,
                                       source: :payment_methods
 
-    validates :host, presence: true, uniqueness: true,
-                     format: { without: %r{\Ahttps?://},
-                               message: :format
-                             }
+    validates :host, uniqueness: true, presence: true
 
     enum state: { enabled: 0, disabled: 1 }
 
@@ -25,12 +22,6 @@ module SpreeCmCommissioner
       vendors.where(state: :active).first
     end
 
-    def url
-      return if host.blank?
-
-      Rails.env.development? || Rails.env.test? ? "http://#{host}" : "https://#{host}"
-    end
-
     private
 
     def generate_slug

data/app/models/spree_cm_commissioner/vendor_place.rb

@@ -1,9 +1,8 @@
 module SpreeCmCommissioner
   class VendorPlace < Base
-    enum place_type: { venue: 0, branch: 1, stop: 2, location: 3 }
+    enum place_type: { location: 0, branch: 1, stop: 2 }
     acts_as_list scope: :vendor
 
-    belongs_to :tenant, class_name: 'SpreeCmCommissioner::Tenant', optional: true
     belongs_to :vendor, class_name: 'Spree::Vendor', optional: false
     belongs_to :place, class_name: 'SpreeCmCommissioner::Place', optional: false
     belongs_to :location, class_name: 'SpreeCmCommissioner::VendorPlace', optional: true
@@ -17,8 +16,6 @@ module SpreeCmCommissioner
 
     accepts_nested_attributes_for :place, allow_destroy: true
 
-    before_save :set_tenant_id
-
     # include place when using these delegate to avoid N+1
     def display_name
       place&.name
@@ -31,11 +28,5 @@ module SpreeCmCommissioner
     def selected
       vendor.selected_place_references.include?(place&.reference)
     end
-
-    private
-
-    def set_tenant_id
-      self.tenant_id = vendor&.tenant_id
-    end
   end
 end

data/app/overrides/spree/admin/stores/_form/store_preferences.html.erb.deface

@@ -37,15 +37,6 @@
             <% end %>
           </div>
 
-          <!-- Telegram New Vendor Alert Chat ID -->
-          <div class="col-6">
-            <%= f.field_container :preferred_telegram_new_vendor_alert_chat_id do %>
-              <%= f.label :preferred_telegram_new_vendor_alert_chat_id, Spree.t(:telegram_new_vendor_alert_chat_id) %>
-              <%= f.text_field :preferred_telegram_new_vendor_alert_chat_id, class: 'form-control' %>
-              <%= f.error_message_on :preferred_telegram_new_vendor_alert_chat_id %>
-            <% end %>
-          </div>
-
           <!-- Assetlinks -->
           <div class="col-12">
             <%= f.field_container :preferred_assetlinks do %>

data/app/queries/spree_cm_commissioner/trip_query.rb

@@ -1,13 +1,12 @@
 module SpreeCmCommissioner
   class TripQuery
-    attr_reader :origin_id, :destination_id, :date, :vendor_id, :tenant_id, :number_of_guests, :route_type, :params
+    attr_reader :origin_id, :destination_id, :date, :vendor_id, :number_of_guests, :route_type, :params
 
-    def initialize(origin_id:, destination_id:, date:, route_type: nil, vendor_id: nil, tenant_id: nil, number_of_guests: nil, params: {}) # rubocop:disable Metrics/ParameterLists
+    def initialize(origin_id:, destination_id:, date:, route_type: nil, vendor_id: nil, number_of_guests: nil, params: {}) # rubocop:disable Metrics/ParameterLists
       @origin_id = origin_id
       @destination_id = destination_id
       @date = date.to_date == Time.zone.now.to_date ? Time.zone.now : Time.zone.parse(date.to_s)
       @vendor_id = vendor_id
-      @tenant_id = tenant_id
       @number_of_guests = number_of_guests || 1
       @route_type = route_type
       @params = params
@@ -37,9 +36,8 @@ module SpreeCmCommissioner
 
     def direct_trips
       result = trip_scope
-      result = result.where({ vendor_id: vendor_id }.compact) if vendor_id.present?
+      result = result.where({ vendor_id: vendor_id }.compact)
       result = result.where(route_type: route_type) if route_type.present?
-      result = result.where(spree_vendors: { tenant_id: tenant_id }) if tenant_id.present?
       paginate(result)
     end
 
@@ -58,24 +56,20 @@ module SpreeCmCommissioner
     private
 
     def trip_scope
-      scope = SpreeCmCommissioner::Trip
-              .select(<<~SQL.squish)
-                cm_trips.*,
-                boarding.departure_time AS boarding_departure_time,
-                boarding.stop_place_id AS boarding_stop_id, boarding.stop_name AS boarding_stop_name,
-                drop_off.stop_name AS drop_off_stop_name, drop_off.stop_place_id AS drop_off_stop_id,
-                drop_off.arrival_time AS drop_off_arrival_time,
-                COALESCE(iv.quantity_available, 0) AS quantity_available,
-                COALESCE(iv.max_capacity, 0) AS max_capacity,
-                origin_places.name AS origin_place_name, dest_places.name AS destination_place_name,
-                prices.amount AS amount, prices.compare_at_amount AS compare_at_amount,
-                prices.currency AS currency
-              SQL
-              .includes(vendor: :logo, vehicle: :option_values)
-
-      scope = scope.joins(:vendor) if tenant_id.present?
-
-      scope
+      SpreeCmCommissioner::Trip
+        .select(<<~SQL.squish)
+          cm_trips.*,
+          boarding.departure_time AS boarding_departure_time,
+          boarding.stop_place_id AS boarding_stop_id, boarding.stop_name AS boarding_stop_name,
+          drop_off.stop_name AS drop_off_stop_name, drop_off.stop_place_id AS drop_off_stop_id,
+          drop_off.arrival_time AS drop_off_arrival_time,
+          COALESCE(iv.quantity_available, 0) AS quantity_available,
+          COALESCE(iv.max_capacity, 0) AS max_capacity,
+          origin_places.name AS origin_place_name, dest_places.name AS destination_place_name,
+          prices.amount AS amount, prices.compare_at_amount AS compare_at_amount,
+          prices.currency AS currency
+        SQL
+        .includes(vendor: :logo, vehicle: :option_values)
         .joins(<<~SQL.squish)
           INNER JOIN cm_trip_stops AS boarding ON boarding.trip_id = cm_trips.id AND boarding.stop_type = 0
           INNER JOIN cm_trip_stops AS drop_off ON drop_off.trip_id = cm_trips.id AND drop_off.stop_type = 1

data/app/serializers/spree_cm_commissioner/v2/storefront/trip_vehicle_serializer.rb

@@ -4,7 +4,7 @@ module SpreeCmCommissioner
       class TripVehicleSerializer < BaseSerializer
         set_type :vehicle
 
-        attributes :id, :code, :vehicle_type, :number_of_seats
+        attributes :id, :code, :vehicle_type
 
         has_many :vehicle_photos, serializer: ::SpreeCmCommissioner::V2::Storefront::AssetSerializer
         has_many :amenities, serializer: ::SpreeCmCommissioner::V2::Storefront::AmenitySerializer

data/app/services/spree_cm_commissioner/intercity_taxi_order/create.rb

@@ -42,7 +42,6 @@ module SpreeCmCommissioner
           )
 
           build_guests_for!(line_item, quantity)
-          assign_saved_guests_to_line_item(line_item)
 
           order.save!
           order.update_with_updater!
@@ -63,13 +62,6 @@ module SpreeCmCommissioner
       def build_guests_for!(line_item, qty)
         Array.new(qty) { line_item.guests.new }
       end
-
-      def assign_saved_guests_to_line_item(line_item)
-        line_item.guests.each do |guest|
-          saved_guest = SpreeCmCommissioner::SavedGuest.new
-          guest.saved_guest = saved_guest
-        end
-      end
     end
   end
 end

data/app/services/spree_cm_commissioner/intercity_taxi_order/update.rb

@@ -1,20 +1,23 @@
 module SpreeCmCommissioner
   module IntercityTaxiOrder
     class Update
-      attr_reader :params
+      attr_reader :order_number, :params
 
-      def self.call(params:)
-        new(params: params).call
+      def self.call(order_number:, params:)
+        new(order_number: order_number, params: params).call
       end
 
-      def initialize(params:)
+      def initialize(order_number:, params:)
+        @order_number = order_number
         @params = ensure_parameters(params)
       end
 
       def call
-        order = find_order!
+        order = Spree::Order.find_by!(number: order_number)
         order.update!(order_params)
 
+        create_billing_address_if_missing(order)
+
         order
       end
 
@@ -26,41 +29,48 @@ module SpreeCmCommissioner
         ::ActionController::Parameters.new(params)
       end
 
-      def find_order!
-        raise(ArgumentError, 'Order token is required') if order_token.blank?
-
-        Spree::Order.find_by!(token: order_token)
-      end
+      def create_billing_address_if_missing(order)
+        return if order.bill_address.present?
 
-      def order_token
-        params[:order_token]
+        SpreeCmCommissioner::BillingAddressCreator.call(order: order)
       end
 
       def order_params
         params.require(:order).permit(
-          line_items_attributes: %i[
-            id
-            pickup_place_name
-            drop_off_place_name
-            from_date
-            pickup_lat
-            pickup_lng
-            pickup_oob_confirmed
-            drop_off_lat
-            drop_off_lng
-            drop_off_oob_confirmed
-            distance_km
-            ordered_points
-            base_km
-            detour_pickup_km
-            detour_dropoff_km
-            extra_pickup_km
-            extra_dropoff_km
-            extra_pickup_charge_usd
-            extra_dropoff_charge_usd
-            estimated_time_minutes
-            remark
-            passenger_count
+          :intel_phone_number,
+          :email,
+          line_items_attributes: [
+            :id,
+            :pickup_place_name,
+            :drop_off_place_name,
+            :from_date,
+            :pickup_lat,
+            :pickup_lng,
+            :pickup_oob_confirmed,
+            :drop_off_lat,
+            :drop_off_lng,
+            :drop_off_oob_confirmed,
+            :distance_km,
+            :ordered_points,
+            :base_km,
+            :detour_pickup_km,
+            :detour_dropoff_km,
+            :extra_pickup_km,
+            :extra_dropoff_km,
+            :extra_pickup_charge_usd,
+            :extra_dropoff_charge_usd,
+            :estimated_time_minutes,
+            :remark,
+            :passenger_count,
+            { guests_attributes: %i[
+              id
+              first_name
+              last_name
+              gender
+              nationality_id
+              age
+            ]
+            }
           ]
         )
       end

data/app/views/shared/_asset_field.html.erb

@@ -3,8 +3,6 @@
 <%# :asset %>
 <%# :remove_url, optional %>
 <%# :classes, optional %>
-<%# :aspect_ratio, optional %>
-<%# :size_info, optional %>
 <%# :form %>
 <%# :required, optional, default: false %>
 
@@ -18,17 +16,6 @@
     class: 'rounded border mb-4 mw-100',
     style: 'max-height: 100px') if asset.present? && asset.attachment.present? %>
 
-  <% if defined?(aspect_ratio) || defined?(size_info) %>
-    <small class="mb-2 form-text text-muted d-block">
-      <% if defined?(aspect_ratio) && aspect_ratio.present? %>
-        <strong>Aspect Ratio:</strong> <%= aspect_ratio %><br>
-      <% end %>
-      <% if defined?(size_info) && size_info.present? %>
-        <strong>Recommended Size:</strong> <%= size_info %>
-      <% end %>
-    </small>
-  <% end %>
-
   <div data-hook="file" class="form-group">
     <%= form.label field, label || Spree.t(field) %>
     <%= form.file_field field, required: required %>