spree_cm_commissioner 2.8.3.pre.pre4 → 2.8.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8bd441927a9bb64c823a003b5a1f0ecbb7591a90de805f3e3edfdcaaa48180ba
-  data.tar.gz: 3a5b5097a6b2daf1edf5d2b8a55d3d87cc6de0127022b3859817ffe3994ae75d
+  metadata.gz: 0774f6d0c4fad7ae396f14326fd6d0965164b74d89f355c3966bc8d1fd9563f9
+  data.tar.gz: 002d84c9f1273b54f01d5a25bdecc5f3b475a12e07b32d8e1846c19107fe776f
 SHA512:
-  metadata.gz: 37bcbc34200db68f4647ff333e5e923939b42d46f669ea04b721be9652f7280e12f047ad9541b538054723cd16a63350f62cc586d978fa8ad94a7e4625695ca9
-  data.tar.gz: ab2ee51768d3b2d9c7b1145120b0d70defa14c6cbb6ea7c9a550a11f7ec2496b3674bdbba48f0a37f452c11f7fc5c7ba2d567eacd4f9de0380a18c06ce1fe68e
+  metadata.gz: 260d54db1f9dd63dbfef02edf2944d02e2c582972b8903815462e438be9cbfb6f5561a7b4a4b64c2cda4af18908c8c7a91958213c06764d1f7ee4d8e2c16dec3
+  data.tar.gz: 003d6ecd547b0215851266f06e823f2f7c5e7a8da35f5b4a8051784855a4d095d088b79f700dcadca8820d561d7a9989a3ee0d9eb7940943c3c5777cc7a4742c

data/.env.example

@@ -34,6 +34,7 @@ DISTANCE_SIGNING_KEY=hei********************VY
 
 # Cache durations (in seconds) for different content types
 # See: app/controllers/concerns/spree_cm_commissioner/content_cachable.rb
+CACHE_AUTO_INVALIDATION_DISABLED="" # Set to yes to disable CDN cache invalidation entirely (kill switch)
 CACHE_CDN_MAX_AGE=86400 # CDN/server-side cache duration for all responses
 CACHE_STATIC_MAX_AGE=86400 # Static content (countries, provinces, seat layouts, CMS pages) - 1 day
 CACHE_SEMI_STATIC_MAX_AGE=3600 # Semi-static content (menus, homepage backgrounds, routes) - 1 hour

data/.gitignore

@@ -34,7 +34,3 @@ vendor/bundle/
 # Cursor
 .cursor
 dump.rdb
-# Added by code-review-graph
-.code-review-graph/
-.cursorrules
-.claude

data/Gemfile.lock

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

data/app/controllers/concerns/spree_cm_commissioner/content_cachable.rb

@@ -16,7 +16,6 @@ module SpreeCmCommissioner
       GuestCardClassesController
       SeatLayoutController
       CmsPagesController
-      AdvertisementsController
     ].freeze
 
     # Priority 2: Semi-Static (1 hour)
@@ -26,7 +25,6 @@ module SpreeCmCommissioner
       HomepageDataController
       PopularRoutesController
       RoutePlacesController
-      ShowsController
     ].freeze
 
     # Priority 3: Moderate Freshness (30 minutes)

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

@@ -51,10 +51,7 @@ module Spree
       def permitted_resource_params
         segment_value = helpers.calculate_segment_value(params[:spree_cm_commissioner_homepage_section])
 
-        params
-          .require(:spree_cm_commissioner_homepage_section)
-          .permit(:title, :description, :active, :tenant_id, :preview)
-          .merge(segment: segment_value)
+        params.require(:spree_cm_commissioner_homepage_section).permit(:title, :description, :active, :tenant_id).merge(segment: segment_value)
       end
 
       def load_tenants

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

@@ -3,7 +3,6 @@ module Spree
     module TaxonsControllerDecorator
       def self.prepended(base)
         base.before_action :build_assets, only: %i[create update]
-        base.before_action :normalize_sti_taxon_params, only: %i[create update]
       end
 
       # override
@@ -51,24 +50,6 @@ module Spree
         @taxon.build_home_banner(attachment: permitted_resource_params.delete(:home_banner)) if permitted_resource_params[:home_banner]
         @taxon.build_video_banner(attachment: permitted_resource_params.delete(:video_banner)) if permitted_resource_params[:video_banner]
       end
-
-      # Some admin forms submit STI records under their class key
-      # (e.g. :spree_cm_commissioner_show) while Spree update reads :taxon.
-      # Merge both into :taxon so all edited fields are persisted.
-      def normalize_sti_taxon_params
-        return unless @taxon
-
-        sti_key = @taxon.class.model_name.param_key.to_sym
-        sti_params = params[sti_key]
-        return unless sti_params.is_a?(::ActionController::Parameters)
-
-        base_params = params[:taxon]
-        base_hash = base_params.is_a?(::ActionController::Parameters) ? base_params.to_unsafe_h : {}
-        sti_hash = sti_params.to_unsafe_h
-        merged_params = ::ActionController::Parameters.new(base_hash.merge(sti_hash))
-
-        params[:taxon] = merged_params
-      end
     end
   end
 end

data/app/controllers/spree/api/v2/storefront/homepage_sections_controller.rb

@@ -19,7 +19,6 @@ module Spree
             @collection ||= model_class.filter_by_segment(params[:homepage_id] || :general)
                                        .active
                                        .where(tenant_id: nil)
-                                       .where(preview: false)
                                        .order(position: :asc)
           end
         end

data/app/controllers/spree/api/v2/storefront/trip_search_controller.rb

@@ -39,7 +39,6 @@ module Spree
               params[:date],
               params[:route_type],
               params[:vendor_id],
-              params[:featured],
               params[:number_of_guests],
               resource_includes&.sort&.join(','),
               sparse_fields&.sort&.join(','),
@@ -65,8 +64,7 @@ module Spree
               'trips.vendor',
               'trips.vendor.logo',
               'trips.vehicle_type',
-              'trips.vehicle_type.vehicle_photos',
-              'trips.amenities'
+              'trips.vehicle_type.vehicle_photos'
             ]
           end
 
@@ -100,7 +98,11 @@ module Spree
 
           # override
           def serializer_params
-            params.permit(:include).to_hash
+            super.merge(
+              exclude_vendor_kind_option_types: true,
+              exclude_vendor_kind_option_values: true,
+              exclude_vehicle_type_amenities: true
+            )
           end
 
           # override

data/app/controllers/spree/api/v2/tenant/base_controller.rb

@@ -16,10 +16,6 @@ module Spree
             set_current_tenant(@tenant)
           end
 
-          def current_vendor
-            @current_vendor ||= @tenant.active_vendor
-          end
-
           def render_serialized_payload(status = 200)
             render json: yield, status: status, content_type: content_type
           end

data/app/controllers/spree/api/v2/tenant/homepage_sections_controller.rb

@@ -5,7 +5,6 @@ module Spree
         class HomepageSectionsController < BaseController
           def collection
             @collection ||= scope.filter_by_segment(params[:homepage_id] || :general)
-                                 .where(preview: false)
                                  .active
                                  .order(position: :asc)
           end

data/app/controllers/spree/api/v2/tenant/products_controller.rb

@@ -21,7 +21,7 @@ module Spree
 
           def scope
             MultiTenant.with(@tenant) do
-              model_class.where(tenant_id: MultiTenant.current_tenant_id, preview: false)
+              model_class.where(tenant_id: MultiTenant.current_tenant_id)
             end
           end
 

data/app/controllers/spree/api/v2/tenant/taxons_controller.rb

@@ -31,7 +31,7 @@ module Spree
 
           def scope
             MultiTenant.with(@tenant) do
-              model_class.where(preview: false)
+              model_class
             end
           end
 

data/app/controllers/spree_cm_commissioner/admin/products_controller_decorator.rb

@@ -6,7 +6,6 @@ module SpreeCmCommissioner
         base.around_action :set_writing_role, only: %i[index]
         base.after_action :set_tenant_after_update, only: %i[update]
         base.before_action :merge_industry_taxons_into_taxons, only: [:update]
-        base.before_action :normalize_sti_product_params, only: %i[create update]
       end
 
       # Override
@@ -52,24 +51,6 @@ module SpreeCmCommissioner
 
       protected
 
-      # Some admin forms submit STI records under their class key while
-      # Spree update reads :product. Merge both into :product so all edited
-      # fields are persisted for STI product subclasses.
-      def normalize_sti_product_params
-        return unless @product
-
-        sti_key = @product.class.model_name.param_key.to_sym
-        sti_params = params[sti_key]
-        return unless sti_params.is_a?(::ActionController::Parameters)
-
-        base_params = params[:product]
-        base_hash = base_params.is_a?(::ActionController::Parameters) ? base_params.to_unsafe_h : {}
-        sti_hash = sti_params.to_unsafe_h
-        merged_params = ::ActionController::Parameters.new(base_hash.merge(sti_hash))
-
-        params[:product] = merged_params
-      end
-
       def merge_industry_taxons_into_taxons
         return if params[:product].blank?
 

data/app/factory/spree_cm_commissioner/event_telegram_message_factory.rb

@@ -0,0 +1,68 @@
+# 🎉 ---[Event]---
+#
+# 📋 Event: {{event.name}}
+#
+# 👤 Organizer: {{vendor.name}}
+#
+# 🗓️ Start Date: {{from_date}}
+# 🗓️ End Date: {{to_date}}
+# 🔗 URL: ......
+#
+module SpreeCmCommissioner
+  class EventTelegramMessageFactory < TelegramMessageFactory
+    attr_reader :event
+
+    def initialize(event:)
+      @event = event
+
+      title = "✨ #{event.name} ✨"
+
+      super(title: title)
+    end
+
+    def body
+      [
+        event_name_line,
+        organizer_line,
+        [start_date_line, end_date_line].compact.join("\n"),
+        event_url_line
+      ].compact_blank.join("\n\n")
+    end
+
+    private
+
+    def event_name_line
+      "📋 Event: #{event.name}" if event.name.present?
+    end
+
+    def organizer_line
+      return if event.vendor&.name.blank?
+
+      "👤 Organizer: #{bold(event.vendor.name)}"
+    end
+
+    def start_date_line
+      return if event.from_date.blank?
+
+      "🗓️ Start Date: #{pretty_datetime(event.from_date)}"
+    end
+
+    def end_date_line
+      return if event.to_date.blank?
+
+      "🗓️ End Date: #{pretty_datetime(event.to_date)}"
+    end
+
+    def pretty_datetime(date)
+      return '' if date.blank?
+
+      date.strftime('%b %d, %Y · %H:%M')
+    end
+
+    def event_url_line
+      return if event.event_url.blank?
+
+      "🔗 URL: #{event.event_url}"
+    end
+  end
+end

data/app/factory/spree_cm_commissioner/order_telegram_message_factory.rb

@@ -48,7 +48,7 @@ module SpreeCmCommissioner
     end
 
     def line_item_content(line_item)
-      return intercity_taxi_line_item_content(line_item) if intercity_taxi?(line_item)
+      return transit_line_item_content(line_item) if line_item.transit?
 
       text = []
 
@@ -64,8 +64,8 @@ module SpreeCmCommissioner
     def pretty_date_for(line_item)
       return nil unless line_item.date_present?
 
-      from_date = pretty_date(line_item.from_date)
-      to_date = pretty_date(line_item.to_date)
+      from_date = pretty_datetime(line_item.from_date)
+      to_date = pretty_datetime(line_item.to_date)
 
       if from_date == to_date
         "🗓️ #{from_date}"
@@ -84,7 +84,7 @@ module SpreeCmCommissioner
       text << "Email: #{inline_code(order.email)}" if order.email.present?
       text << "Delivery Address: #{formatted_shipping_address.presence || 'N/A'}" if order.delivery_required?
 
-      append_intercity_taxi_footer(text) if intercity_taxi_order?
+      append_transit_footer(text) if transit_order?
 
       if show_details_link && order.guests.any?
         text << ''
@@ -132,12 +132,12 @@ module SpreeCmCommissioner
 
     private
 
-    def intercity_taxi?(line_item)
-      trip_for(line_item)&.intercity_taxi?
+    def transit_order?
+      selected_line_items.any?(&:transit?)
     end
 
     def intercity_taxi_order?
-      selected_line_items.any? { |li| intercity_taxi?(li) }
+      selected_line_items.any? { |li| trip_for(li)&.intercity_taxi? }
     end
 
     def trip_for(line_item)
@@ -147,10 +147,10 @@ module SpreeCmCommissioner
       @trip_cache[line_item.trip_id] ||= SpreeCmCommissioner::Trip.find_by(id: line_item.trip_id)
     end
 
-    def intercity_taxi_line_item_content(line_item)
+    def transit_line_item_content(line_item)
       text = []
-      route_text = taxi_route_text(line_item)
-      departure_text = taxi_departure_text(line_item)
+      route_text = transit_route_text(line_item)
+      departure_text = transit_departure_text(line_item)
       text << bold('🚗 Trip Details')
       text << "Route: #{route_text}" if route_text.present?
       text << "Passengers: #{line_item.passenger_count || line_item.quantity}"
@@ -159,7 +159,7 @@ module SpreeCmCommissioner
       text.compact.join("\n")
     end
 
-    def taxi_route_text(line_item)
+    def transit_route_text(line_item)
       trip = trip_for(line_item)
       return line_item.product.name if trip.blank?
 
@@ -171,21 +171,35 @@ module SpreeCmCommissioner
       vehicle.present? ? "#{parts} (#{vehicle})" : parts
     end
 
-    def taxi_departure_text(line_item)
+    def transit_departure_text(line_item)
       return nil unless line_item.date_present?
 
       line_item.from_date.strftime('%b %d, %Y · %H:%M')
     end
 
     def nationality_text
-      values = order.saved_guests.map { |sg| sg.nationality&.name.presence || sg.nationality_group&.humanize }.compact.uniq
-      return nil if values.empty?
+      grouped = order.saved_guests.group_by { |sg| sg.nationality_group&.humanize }
+      grouped.delete(nil)
+      return nil if grouped.empty?
 
-      values.join(', ')
+      grouped.map { |nationality_group, guests| customer_type_summary(nationality_group, guests) }.join(', ')
     end
 
-    def append_intercity_taxi_footer(text)
-      text << "Nationality: #{nationality_text}" if nationality_text.present?
+    # e.g. "Local (Adult x1, Child x1)". age_group is NOT NULL, so it is always present.
+    def customer_type_summary(nationality_group, guests)
+      age_breakdown = guests
+                      .group_by { |sg| sg.age_group.humanize }
+                      .map { |age_group, age_guests| "#{age_group} x#{age_guests.count}" }
+                      .join(', ')
+
+      "#{nationality_group} (#{age_breakdown})"
+    end
+
+    def append_transit_footer(text)
+      nationality = nationality_text
+      text << "Nationality: #{nationality}" if nationality.present?
+
+      return unless intercity_taxi_order?
 
       pickup = selected_line_items.filter_map(&:pickup_map_place).first
       dropoff = selected_line_items.filter_map(&:dropoff_map_place).first

data/app/factory/spree_cm_commissioner/telegram_message_factory.rb

@@ -46,5 +46,11 @@ module SpreeCmCommissioner
 
       date.to_date.strftime('%b %d, %Y')
     end
+
+    def pretty_datetime(datetime)
+      return '' if datetime.blank?
+
+      datetime&.in_time_zone&.strftime('%b %d, %Y - %H:%M')
+    end
   end
 end

data/app/finders/spree_cm_commissioner/events/find_matches.rb

@@ -7,7 +7,6 @@ module SpreeCmCommissioner
       def execute
         scope.joins(:integration_mappings).where(
           kind: :event,
-          preview: false,
           integration_mappings: { status: :active }
         ).distinct
       end

data/app/helpers/spree_cm_commissioner/admin/homepage_segment_helper.rb

@@ -15,8 +15,6 @@ module SpreeCmCommissioner
           'badge badge-success text-uppercase'
         when :football
           'badge badge-dark text-uppercase'
-        when :show
-          'badge badge-danger text-uppercase'
         else
           'badge'
         end

data/app/interactors/spree_cm_commissioner/invalidate_cache_request.rb

@@ -19,13 +19,17 @@ module SpreeCmCommissioner
       context.response = client.create_invalidation(
         distribution_id: ENV.fetch('ASSETS_SYNC_CF_DIST_ID'),
         invalidation_batch: {
-          caller_reference: Time.now.to_i.to_s,
+          # Must be globally unique per distribution,
+          # Time.now.to_i collides when multiple jobs fire in the same second.
+          caller_reference: SecureRandom.uuid,
           paths: {
             quantity: patterns.size,
             items: patterns
           }
         }
       )
+    rescue Aws::CloudFront::Errors::TooManyInvalidationsInProgress => e
+      context.fail!(message: "Too many invalidations in progress: #{e}")
     end
   end
 end

data/app/jobs/spree_cm_commissioner/event_creation_telegram_alert_sender_job.rb

@@ -0,0 +1,10 @@
+module SpreeCmCommissioner
+  class EventCreationTelegramAlertSenderJob < ApplicationUniqueJob
+    queue_as :telegram_bot
+
+    def perform(options = {})
+      event = Spree::Taxon.find(options[:event_id])
+      SpreeCmCommissioner::TelegramAlerts::EventCreation.call(event: event)
+    end
+  end
+end

data/app/jobs/spree_cm_commissioner/invalidate_cache_request_job.rb

@@ -1,7 +1,10 @@
 module SpreeCmCommissioner
   class InvalidateCacheRequestJob < ApplicationUniqueJob
+    queue_as :cache_invalidation
+
     def perform(options = {})
-      SpreeCmCommissioner::InvalidateCacheRequest.call(patterns: options[:patterns])
+      result = SpreeCmCommissioner::InvalidateCacheRequest.call(patterns: options[:patterns])
+      raise result.error if result.failure?
     end
   end
 end

data/app/jobs/spree_cm_commissioner/inventory_items/bulk_adjust_quantities_job.rb

@@ -1,13 +1,15 @@
 module SpreeCmCommissioner
   module InventoryItems
     class BulkAdjustQuantitiesJob < ApplicationUniqueJob
-      # :line_item_ids, :inventory_id_and_quantities
+      # :line_item_ids, :inventory_id_and_quantities, :caller_source
       #
       # :line_item_ids is included for unique job key generation to prevent duplicate jobs,
       # though it's not used in the perform method implementation.
+      # :caller_source is a string like "ClassName#method" identifying who enqueued the job.
       def perform(options = {})
         SpreeCmCommissioner::InventoryItems::BulkAdjustQuantities.call!(
-          inventory_id_and_quantities: options[:inventory_id_and_quantities]
+          inventory_id_and_quantities: options[:inventory_id_and_quantities],
+          caller_source: options[:caller_source]
         )
       end
     end

data/app/jobs/spree_cm_commissioner/inventory_items/bulk_adjust_quantities_on_hold_job.rb

@@ -3,16 +3,18 @@ module SpreeCmCommissioner
     class BulkAdjustQuantitiesOnHoldJob < ApplicationUniqueJob
       queue_as :default
 
-      # :order_id, :inventory_id_and_quantities
+      # :order_id, :inventory_id_and_quantities, :caller_source
       #
       # :order_id is included for unique job key generation to prevent duplicate jobs,
       # though it's not used in the perform method implementation.
+      # :caller_source is a string like "ClassName#method" identifying who enqueued the job.
       def perform(options = {})
         raise ArgumentError, 'order_id is required' if options[:order_id].blank?
         raise ArgumentError, 'inventory_id_and_quantities is required' if options[:inventory_id_and_quantities].blank?
 
         SpreeCmCommissioner::InventoryItems::BulkAdjustQuantitiesOnHold.call(
-          inventory_id_and_quantities: options[:inventory_id_and_quantities]
+          inventory_id_and_quantities: options[:inventory_id_and_quantities],
+          caller_source: options[:caller_source]
         )
       end
     end

data/app/jobs/spree_cm_commissioner/oauth_access_tokens/cleanup_expired_job.rb

@@ -0,0 +1,11 @@
+module SpreeCmCommissioner
+  module OauthAccessTokens
+    class CleanupExpiredJob < SpreeCmCommissioner::ApplicationJob
+      queue_as :default
+
+      def perform
+        SpreeCmCommissioner::OauthAccessTokens::CleanupExpired.new.call
+      end
+    end
+  end
+end

data/app/jobs/spree_cm_commissioner/route_metrics/decrease_trip_count_job.rb

@@ -2,8 +2,14 @@ module SpreeCmCommissioner
   module RouteMetrics
     class DecreaseTripCountJob < ApplicationUniqueJob
       def perform(options = {})
-        trip = SpreeCmCommissioner::Trip.find(options[:trip_id])
-        SpreeCmCommissioner::RouteMetrics::DecreaseTripCount.call(trip: trip)
+        route_metric = SpreeCmCommissioner::RouteMetric.find_by(
+          origin_place_id: options[:origin_place_id],
+          destination_place_id: options[:destination_place_id],
+          route_type: options[:route_type]
+        )
+        return if route_metric.nil?
+
+        SpreeCmCommissioner::RouteMetrics::DecreaseTripCount.call(route_metric: route_metric)
       end
     end
   end

data/app/jobs/spree_cm_commissioner/route_metrics/increase_trip_count_job.rb

@@ -2,8 +2,11 @@ module SpreeCmCommissioner
   module RouteMetrics
     class IncreaseTripCountJob < ApplicationUniqueJob
       def perform(options = {})
-        trip = SpreeCmCommissioner::Trip.find(options[:trip_id])
-        SpreeCmCommissioner::RouteMetrics::IncreaseTripCount.call(trip: trip)
+        SpreeCmCommissioner::RouteMetrics::IncreaseTripCount.call(
+          origin_place_id: options[:origin_place_id],
+          destination_place_id: options[:destination_place_id],
+          route_type: options[:route_type]
+        )
       end
     end
   end

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

@@ -8,8 +8,7 @@ module SpreeCmCommissioner
       tour: 0b00100,
       accommodation: 0b01000,
       things_to_do: 0b10000,
-      football: 0b100000,
-      show: 0b1000000
+      football: 0b100000
     }.freeze
 
     BIT_SEGMENT.each do |segment, bit_value|

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

@@ -23,6 +23,8 @@ module SpreeCmCommissioner
     TripKey = Struct.new(:trip_id, :direction, :boarding_trip_stop_id, :drop_off_trip_stop_id)
 
     included do
+      include SpreeCmCommissioner::ServiceOrigin
+
       validates :direction, inclusion: { in: DIRECTION }, allow_nil: true
       validates :trip_id, numericality: { only_integer: true, greater_than: 0 }, allow_nil: true
       validates :service_origin_id, numericality: { only_integer: true, greater_than: 0 }, allow_nil: true
@@ -125,6 +127,12 @@ module SpreeCmCommissioner
 
     private
 
+    def resolve_service_origin_id
+      return nil if trip_id.blank?
+
+      SpreeCmCommissioner::Trip.find_by(id: trip_id)&.service_origin_id
+    end
+
     def validate_trip_stops_for_scheduled_trips
       return if trip_id.blank? || trip_id.zero?
 

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

@@ -45,8 +45,7 @@ module SpreeCmCommissioner
       'color' => 'color',
       'ticket-type' => 'string',
       'seat-type' => 'string',
-      'intercity-taxi' => 'string',
-      'vote-package' => 'integer'
+      'intercity-taxi' => 'string'
     }.freeze
 
     included do

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

@@ -26,10 +26,8 @@ module SpreeCmCommissioner
 
       state_machine.after_transition to: :canceled, do: :precalculate_conversion
       state_machine.after_transition to: :canceled, do: :restock_inventory!
-      state_machine.after_transition to: :canceled, do: :de_allocate_resources
 
       state_machine.after_transition to: :complete, do: :increment_route_fulfilled_order_count
-      state_machine.after_transition to: :complete, do: :allocate_resources
 
       scope :accepted, -> { where(request_state: 'accepted') }
 
@@ -330,13 +328,5 @@ module SpreeCmCommissioner
       factory = OrderTelegramMessageFactory.new(title: title, order: self)
       TelegramNotificationSenderJob.perform_later(chat_id: chat_id, message: factory.message, parse_mode: factory.parse_mode)
     end
-
-    def allocate_resources
-      SpreeCmCommissioner::VotingCreditAllocationJob.perform_later(order_id: id)
-    end
-
-    def de_allocate_resources
-      SpreeCmCommissioner::VotingCreditDeAllocationJob.perform_later(order_id: id)
-    end
   end
 end

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

@@ -5,7 +5,7 @@ module SpreeCmCommissioner
   module ProductType
     extend ActiveSupport::Concern
 
-    PRODUCT_TYPES = %i[accommodation service ecommerce transit show_episode vote_package advertisement].freeze
+    PRODUCT_TYPES = %i[accommodation service ecommerce transit].freeze
     PERMANENT_STOCK_PRODUCT_TYPES = %w[accommodation service transit].freeze
     PRE_INVENTORY_DAYS = { 'transit' => 90, 'accommodation' => 365, 'service' => 30 }.freeze