spree_cm_commissioner 2.0.3.pre.pre.pre7 → 2.1.0.pre.pre1

data/app/models/spree_cm_commissioner/vehicle.rb

@@ -1,29 +1,26 @@
-require_dependency 'spree_cm_commissioner'
-
 module SpreeCmCommissioner
-  class Vehicle < SpreeCmCommissioner::Base
-    include SpreeCmCommissioner::RouteType
+  class Vehicle < Base
+    include SpreeCmCommissioner::VehicleType
 
-    belongs_to :vehicle_type, class_name: 'SpreeCmCommissioner::VehicleType'
-    has_one :primary_photo, -> { order(position: :asc) }, class_name: 'SpreeCmCommissioner::VehiclePhoto', as: :viewable, dependent: :destroy
     belongs_to :vendor, class_name: 'Spree::Vendor'
 
-    after_commit :create_vehicle_option_value
+    has_one :primary_photo, -> { order(position: :asc) }, class_name: 'SpreeCmCommissioner::VehiclePhoto', as: :viewable, dependent: :destroy
+    has_one :seat_layout, as: :layoutable, class_name: 'SpreeCmCommissioner::SeatLayout', dependent: :destroy
 
-    has_many :vehicle_photo, class_name: 'SpreeCmCommissioner::VehiclePhoto', as: :viewable, dependent: :destroy
-    has_many :vehicle_seats, class_name: 'SpreeCmCommissioner::VehicleSeat', through: :vehicle_type
+    has_many :vehicle_photos, class_name: 'SpreeCmCommissioner::VehiclePhoto', as: :viewable, dependent: :destroy
 
     has_many :vehicle_option_types, class_name: 'SpreeCmCommissioner::VehicleOptionType', dependent: :destroy
     has_many :option_value_vehicles, class_name: 'SpreeCmCommissioner::OptionValueVehicle', dependent: :destroy
+    has_many :option_values, through: :option_value_vehicles, class_name: 'Spree::OptionValue'
+    has_many :amenities, through: :option_value_vehicles, source: :option_value
 
     validates :code, uniqueness: { scope: :vendor_id }, presence: true
-    validates :license_plate, uniqueness: {}, allow_blank: true
+    validates :license_plate, uniqueness: true, allow_blank: true
 
-    def create_vehicle_option_value
-      SpreeCmCommissioner::VehicleOptionValueCreator.call(self)
-    end
+    self.whitelisted_ransackable_attributes = %w[license_plate code vehicle_type]
 
-    self.whitelisted_ransackable_attributes = %w[license_plate code]
-    self.whitelisted_ransackable_associations = %w[vehicle_type]
+    def display_name
+      "#{code} (#{license_plate})"
+    end
   end
 end

data/app/models/spree_cm_commissioner/vendor_decorator.rb

@@ -36,21 +36,17 @@ module SpreeCmCommissioner
       base.has_many :vendor_kind_option_values,
                     through: :option_value_vendors, source: :option_value
 
-      base.has_many :branches, lambda {
-                                 where(cm_vendor_places: { place_type: 0 })
-                               }, through: :vendor_places, class_name: 'SpreeCmCommissioner::VendorPlace'
-      base.has_many :stops, -> { where(cm_vendor_places: { place_type: 1 }) }, through: :vendor_places, class_name: 'SpreeCmCommissioner::VendorPlace'
-      base.has_many :locations, -> { where(cm_vendor_places: { place_type: 2 }) },
-                    through: :vendor_places, class_name: 'SpreeCmCommissioner::VendorPlace'
+      base.has_many :branches, -> { branch }, class_name: 'SpreeCmCommissioner::VendorPlace'
+      base.has_many :stops, -> { stop }, class_name: 'SpreeCmCommissioner::VendorPlace'
+      base.has_many :locations, -> { location }, class_name: 'SpreeCmCommissioner::VendorPlace'
+
+      base.has_many :branch_places, through: :branches, class_name: 'SpreeCmCommissioner::Place', source: :place
+      base.has_many :stop_places, through: :stops, class_name: 'SpreeCmCommissioner::Place', source: :place
+      base.has_many :location_places, through: :locations, class_name: 'SpreeCmCommissioner::Place', source: :place
 
       base.has_many :places,
                     through: :nearby_places, source: :place, class_name: 'SpreeCmCommissioner::Place'
 
-      base.has_many :vendor_stops, class_name: 'SpreeCmCommissioner::VendorStop', dependent: :destroy
-      base.has_many :boarding_points, -> { where(cm_vendor_stops: { stop_type: 0 }) },
-                    through: :vendor_stops, source: :stop, class_name: 'SpreeCmCommissioner::Place'
-      base.has_many :drop_off_points, -> { where(cm_vendor_stops: { stop_type: 1 }) },
-                    through: :vendor_stops, source: :stop, class_name: 'SpreeCmCommissioner::Place'
       base.has_many :dynamic_fields, class_name: 'SpreeCmCommissioner::DynamicField', foreign_key: :vendor_id, dependent: :destroy
 
       base.has_one  :logo, as: :viewable, dependent: :destroy, class_name: 'SpreeCmCommissioner::VendorLogo'
@@ -81,12 +77,16 @@ module SpreeCmCommissioner
       base.has_many :promotion_categories, class_name: 'Spree::PromotionCategory', foreign_key: :vendor_id, dependent: :destroy
 
       base.has_many :homepage_section_relatables,
+                    as: :relatable,
                     class_name: 'SpreeCmCommissioner::HomepageSectionRelatable',
-                    dependent: :destroy, inverse_of: :relatable
+                    dependent: :destroy,
+                    inverse_of: :relatable
 
-      base.has_many :vehicle_types, class_name: 'SpreeCmCommissioner::VehicleType', dependent: :destroy
       base.has_many :vehicles, class_name: 'SpreeCmCommissioner::Vehicle', dependent: :destroy
-      # base.has_many :vehicles, through: :vehicle_types, class_name: 'SpreeCmCommissioner::Vehicle', dependent: :destroy
+      base.has_many :trips, class_name: 'SpreeCmCommissioner::Trip', dependent: :destroy
+      base.has_many :trip_stops, through: :trips, class_name: 'SpreeCmCommissioner::TripStop'
+      base.has_many :boarding_trip_stops, through: :trips, class_name: 'SpreeCmCommissioner::TripStop', source: :boarding_trip_stops
+      base.has_many :drop_off_trip_stops, through: :trips, class_name: 'SpreeCmCommissioner::TripStop', source: :drop_off_trip_stops
 
       base.validates :account_name, :account_number, presence: true, if: lambda {
                                                                            payment_qrcode.present? && Spree::Store.default.code.include?('billing')

data/app/models/spree_cm_commissioner/vendor_place.rb

@@ -5,15 +5,25 @@ module SpreeCmCommissioner
 
     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
+
+    has_many :branches, -> { branch }, class_name: 'SpreeCmCommissioner::VendorPlace', foreign_key: :location_id, dependent: :restrict_with_error
+    has_many :stops, -> { stop }, class_name: 'SpreeCmCommissioner::VendorPlace', foreign_key: :location_id, dependent: :restrict_with_error
 
     validates :place_type, presence: true
     validates :place_id, uniqueness: { scope: %i[vendor_id place_type] }
+    validates :location, presence: true, if: -> { branch? || stop? }
 
     accepts_nested_attributes_for :place, allow_destroy: true
 
-    scope :branches,  -> { where(place_type: :branch) }
-    scope :stops,     -> { where(place_type: :stop) }
-    scope :locations, -> { where(place_type: :location) }
+    # include place when using these delegate to avoid N+1
+    def display_name
+      place&.name
+    end
+
+    def self.ransackable_associations(_auth_object = nil)
+      ['place']
+    end
 
     def selected
       vendor.selected_place_references.include?(place&.reference)

data/app/overrides/spree/admin/shared/sub_menu/_configuration/locations_tab.html.erb.deface

@@ -0,0 +1,5 @@
+<!-- insert_top "[data-hook='admin_configurations_sidebar_menu']" -->
+
+<%= tab :locations, match_path: '/locations', label: 'locations' if can? :manage, SpreeCmCommissioner::Place %>
+
+

data/app/queries/spree_cm_commissioner/trip_query.rb

@@ -1,155 +1,134 @@
 module SpreeCmCommissioner
   class TripQuery
-    include SpreeCmCommissioner::Transit::TripHelper
+    attr_reader :origin_id, :destination_id, :date, :vendor_id, :number_of_guests, :params
 
-    attr_reader :origin_id, :destination_id, :travel_date, :params
-
-    def initialize(origin_id:, destination_id:, travel_date:, params: {})
+    def initialize(origin_id:, destination_id:, date:, vendor_id: nil, number_of_guests: nil, params: {}) # rubocop:disable Metrics/ParameterLists
       @origin_id = origin_id
       @destination_id = destination_id
-      @travel_date = travel_date
-      @travel_date = Time.zone.now if travel_date.to_date == Time.zone.now.to_date
+      @date = date.to_date == Time.zone.now.to_date ? Time.zone.now : Time.zone.parse(date.to_s)
+      @vendor_id = vendor_id
+      @number_of_guests = number_of_guests || 1
       @params = params
+      @page = (params[:page] || 1).to_i
+      @per_page = params[:per_page]
     end
 
     def call
-      return [] if travel_date.to_date < Date.current
+      return Kaminari.paginate_array([]) if date.to_date < Date.current
+
+      paginated_relation = direct_trips
+      return Kaminari.paginate_array([]) if paginated_relation.empty?
 
-      direct_results = direct_trips.map { |trip| SpreeCmCommissioner::TripQueryResult.new([trip]) }
-      connected_results = connected_trips || []
-      direct_results + connected_results
+      unique_trips = paginated_relation.uniq(&:id)
+      results_array = unique_trips.map do |trip|
+        result = build_trip_result(trip)
+        SpreeCmCommissioner::TripQueryResult.new([result])
+      end
+
+      Kaminari.paginate_array(
+        results_array,
+        total_count: unique_trips.size,
+        limit: unique_trips.size,
+        offset: paginated_relation.offset_value
+      )
     end
 
     def direct_trips
-      result = Spree::Variant
-               .select('spree_variants.id AS variant_id,
-                        vendors.id AS vendor_id,
-                        vendors.name AS vendor_name,
-                        routes.name AS route_name,
-                        routes.short_name AS short_name,
-                        boarding.stop_id AS origin_id,
-                        drop_off.stop_id AS destination_id,
-                        boarding.stop_name AS origin,
-                        drop_off.stop_name AS destination,
-                        trips.departure_time AS departure_time,
-                        trips.duration AS duration,
-                        trips.vehicle_id AS vehicle_id'
-                      )
-               .joins('INNER JOIN cm_trips AS trips ON trips.variant_id = spree_variants.id')
-               .joins('INNER JOIN cm_trip_stops AS boarding ON boarding.trip_id = spree_variants.id AND boarding.stop_type = 0')
-               .joins('INNER JOIN cm_trip_stops AS drop_off ON drop_off.trip_id = spree_variants.id AND drop_off.stop_type = 1')
-               .joins('INNER JOIN spree_vendors AS vendors ON vendors.id = spree_variants.vendor_id')
-               .joins('INNER JOIN spree_products AS routes ON routes.id = spree_variants.product_id')
-      result = result.where(vendors: { id: params[:vendor_id] }) if params[:vendor_id].present?
-      result = result.where('boarding.stop_id = ? AND drop_off.stop_id = ?', origin_id, destination_id)
-                     .where('trips.departure_time > ? AND trips.departure_time <= ?',
-                            travel_date.strftime('%H:%M:%S'),
-                            travel_date.end_of_day.strftime('%H:%M:%S')
-                           )
+      @inventory_sql ||= product_inventory_totals
+      result = 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
+                 INNER JOIN cm_places AS origin_places ON origin_places.id = cm_trips.origin_place_id
+                 INNER JOIN cm_places AS dest_places ON dest_places.id = cm_trips.destination_place_id
+                 INNER JOIN spree_variants AS master ON master.product_id = cm_trips.product_id AND master.is_master = true
+                 INNER JOIN spree_prices AS prices ON prices.variant_id = master.id AND prices.deleted_at IS NULL
+               SQL
+               .where('(boarding.location_place_id = ? OR boarding.stop_place_id = ? OR cm_trips.origin_place_id = ?)
+                        AND (drop_off.location_place_id = ? OR drop_off.stop_place_id = ? OR cm_trips.destination_place_id = ?)',
+                      origin_id, origin_id, origin_id, destination_id, destination_id, destination_id
+               )
+               .joins("INNER JOIN (#{@inventory_sql.to_sql}) iv ON cm_trips.product_id = iv.product_id")
 
-      result.map do |trip|
-        trip_result_options = {
-          trip_id: trip[:variant_id],
-          vendor_id: trip[:vendor_id],
-          vendor_name: trip[:vendor_name],
-          route_name: trip[:route_name],
-          short_name: trip[:short_name],
-          detail: trip[:variant_id],
-          origin_id: trip[:origin_id],
-          origin: trip[:origin],
-          destination_id: trip[:destination_id],
-          destination: trip[:destination],
-          vehicle_id: trip[:vehicle_id],
-          departure_time: parse_time(trip[:departure_time]),
-          duration: trip[:duration]
-        }
-        SpreeCmCommissioner::TripResult.new(trip_result_options)
-      end
+      result = result.where(vendor_id: vendor_id) if vendor_id.present?
+      @per_page.to_i.positive? ? result.page(@page).per(@per_page) : result.page(@page)
     end
 
-    def connected_trips # rubocop:disable Metrics/MethodLength
-      result = SpreeCmCommissioner::TripConnection
-               .joins('
-                INNER JOIN spree_variants variant1 ON variant1.id = cm_trip_connections.from_trip_id
-                INNER JOIN spree_variants variant2 ON variant2.id = cm_trip_connections.to_trip_id
-                INNER JOIN spree_products AS routes1 ON routes1.id = variant1.product_id
-                INNER JOIN spree_products AS routes2 ON routes2.id = variant2.product_id
-                INNER JOIN cm_trips AS trip1 ON trip1.variant_id = cm_trip_connections.from_trip_id
-                INNER JOIN cm_trips AS trip2 ON trip2.variant_id = cm_trip_connections.to_trip_id
-                INNER JOIN cm_trip_stops trip1_origin ON trip1_origin.trip_id = variant1.id AND trip1_origin.stop_type = 0
-                INNER JOIN cm_trip_stops trip2_origin ON trip2_origin.trip_id = variant2.id AND trip2_origin.stop_type = 0
-                INNER JOIN cm_trip_stops trip2_destination ON trip2_destination.trip_id = variant2.id AND trip2_destination.stop_type = 1
-                INNER JOIN spree_vendors AS vendor1 ON vendor1.id = variant1.vendor_id
-                INNER JOIN spree_vendors AS vendor2 ON vendor2.id = variant2.vendor_id'
-                     )
-               .select('cm_trip_connections.id AS id,
-                        trip1.variant_id AS trip1_id,
-                        trip1.origin_id AS trip1_origin_id,
-                        trip1.destination_id AS trip1_destination_id,
-                        trip1.departure_time AS trip1_departure_time,
-                        trip1.duration AS trip1_duration,
-                        routes1.short_name AS route1_short_name,
-                        routes1.name AS route1_name,
-                        trip1.vehicle_id AS trip1_vehicle,
-                        vendor1.name AS trip1_vendor_name,
-                        trip2.variant_id AS trip2_id,
-                        trip2.origin_id AS trip2_origin_id,
-                        trip2.destination_id AS trip2_destination_id,
-                        trip2.departure_time AS trip2_departure_time,
-                        trip2.duration AS trip2_duration,
-                        trip2.vehicle_id AS trip2_vehicle,
-                        routes2.short_name AS route2_short_name,
-                        routes2.name AS route2_name,
-                        trip1_origin.stop_name AS trip1_origin,
-                        trip2_origin.stop_name AS trip2_origin,
-                        trip2_destination.stop_name AS trip2_destination,
-                        vendor2.name AS trip2_vendor_name'
-                      ).where('trip1_origin.stop_id = ? AND trip2_destination.stop_id = ?', origin_id, destination_id)
-      result = result.where('vendor1.id = ? OR vendor2.id = ?', params[:vendor_id], params[:vendor_id]) if params[:vendor_id].present?
-      result = result.where('trip1.departure_time > ? AND trip1.departure_time <= ?', travel_date.strftime('%H:%M:%S'),
-                            travel_date.end_of_day.strftime('%H:%M:%S')
-      ).uniq
+    def product_inventory_totals
+      Spree::Product
+        .select(
+          'spree_products.id AS product_id,
+              SUM(cm_inventory_items.max_capacity) AS max_capacity,
+              SUM(cm_inventory_items.quantity_available) AS quantity_available'
+        )
+        .joins(variants: :inventory_items)
+        .where('cm_inventory_items.inventory_date = ? AND cm_inventory_items.quantity_available >= ?', @date.to_date, @number_of_guests)
+        .group('spree_products.id')
+    end
 
-      return [] if result.blank?
+    private
 
-      build_trip_query_result(result)
+    def build_trip_result(trip)
+      vehicle = trip&.vehicle
+      vendor = trip&.vendor
+      trip_result_options = {
+        id: trip&.id,
+        departure_time: trip&.departure_time,
+        duration: trip&.duration,
+        allow_seat_selection: trip&.allow_seat_selection,
+        route_type: trip&.route_type,
+        origin_place: {
+          id: trip&.origin_place_id,
+          name: trip&.origin_place_name
+        },
+        destination_place: {
+          id: trip&.destination_place_id,
+          name: trip&.destination_place_name
+        },
+        vehicle_id: trip&.vehicle_id,
+        vehicle: vehicle,
+        vendor_id: trip&.vendor_id,
+        vendor: vendor,
+        product_id: trip&.product_id,
+        price: trip&.amount,
+        currency: trip&.currency,
+        compare_at_amount: trip&.compare_at_amount,
+        boarding: build_boarding_info(trip),
+        drop_off: build_drop_off_info(trip),
+        quantity_available: trip&.quantity_available,
+        max_capacity: trip&.max_capacity,
+        amenities: (trip.vehicle&.option_values || [])
+      }
+      SpreeCmCommissioner::TripResult.new(trip_result_options)
     end
 
-    private
+    def build_boarding_info(trip)
+      {
+        stop_id: trip&.boarding_stop_id,
+        stop_name: trip&.boarding_stop_name,
+        departure_time: trip&.boarding_departure_time
+      }
+    end
 
-    def build_trip_query_result(connections)
-      connections.map do |trip|
-        from_trip = {
-          trip_id: trip[:trip1_id],
-          origin_id: trip[:trip1_origin_id],
-          destination_id: trip[:trip1_destination_id],
-          departure_time: parse_time(trip[:trip1_departure_time]),
-          duration: trip[:trip1_duration],
-          vendor_name: trip[:trip1_vendor_name],
-          route_name: trip[:route1_name],
-          short_name: trip[:route1_short_name],
-          vehicle_id: trip[:trip1_vehicle],
-          origin: trip[:trip1_origin],
-          destination: trip[:trip2_origin]
-        }
-        to_trip = {
-          trip_id: trip[:trip2_id],
-          origin_id: trip[:trip2_origin_id],
-          destination_id: trip[:trip2_destination_id],
-          departure_time: parse_time(trip[:trip2_departure_time]),
-          duration: trip[:trip2_duration],
-          vendor_name: trip[:trip2_vendor_name],
-          route_name: trip[:route2_name],
-          short_name: trip[:route2_short_name],
-          vehicle_id: trip[:trip2_vehicle],
-          origin: trip[:trip2_origin],
-          destination: trip[:trip2_destination]
-        }
-        data = [SpreeCmCommissioner::TripResult.new(from_trip),
-                SpreeCmCommissioner::TripResult.new(to_trip)
-                ]
-        SpreeCmCommissioner::TripQueryResult.new(data, connection_id: trip[:id])
-      end
+    def build_drop_off_info(trip)
+      {
+        stop_id: trip&.drop_off_stop_id,
+        stop_name: trip&.drop_off_stop_name,
+        arrival_time: trip&.drop_off_arrival_time
+      }
     end
   end
 end

data/app/request_schemas/spree_cm_commissioner/accommodation_request_schema.rb

@@ -3,6 +3,9 @@ module SpreeCmCommissioner
     params do
       required(:from_date).value(:date)
       required(:to_date).value(:date)
+      required(:province_id).value(:integer)
+      required(:adult).value(:integer)
+      required(:children).value(:integer)
     end
 
     rule(:from_date, :to_date) do

data/app/request_schemas/spree_cm_commissioner/application_request_schema.rb

@@ -11,7 +11,7 @@ module SpreeCmCommissioner
     end
 
     def error_message
-      errors.map(&:text).join(', ')
+      errors.map { |error| "#{error.path.join(', ')}: #{error.text}" }.to_sentence
     end
 
     private

data/app/request_schemas/spree_cm_commissioner/inventory_item_schema.rb

@@ -0,0 +1,8 @@
+module SpreeCmCommissioner
+  class InventoryItemSchema < ApplicationRequestSchema
+    params do
+      required(:variant_ids).value(:array).each(:integer)
+      optional(:inventory_dates).maybe(:array).each(:date)
+    end
+  end
+end

data/app/request_schemas/spree_cm_commissioner/trip_search_request_schema.rb

@@ -0,0 +1,20 @@
+module SpreeCmCommissioner
+  class TripSearchRequestSchema < ApplicationRequestSchema
+    params do
+      required(:origin_id).value(:integer)
+      required(:destination_id).value(:integer)
+      required(:date).value(:date)
+    end
+
+    rule(:date) do
+      key.failure(:must_be_in_future) if value < Time.zone.today
+    end
+
+    rule(:origin_id, :destination_id) do
+      origin_id = values[:origin_id]
+      destination_id = values[:destination_id]
+
+      key.failure(:must_be_different) if origin_id == destination_id
+    end
+  end
+end

data/app/request_schemas/spree_cm_commissioner/variant_request_schema.rb

@@ -0,0 +1,19 @@
+module SpreeCmCommissioner
+  class VariantRequestSchema < ApplicationRequestSchema
+    params do
+      required(:from_date).value(:date)
+      required(:to_date).value(:date)
+      required(:adult).value(:integer)
+      required(:children).value(:integer)
+    end
+
+    rule(:from_date, :to_date) do
+      from_date = values[:from_date]
+      to_date   = values[:to_date]
+
+      key.failure(:must_be_in_future) if from_date < Time.zone.today
+      key.failure(:must_be_later_than_start_date) if from_date > to_date
+      key.failure(:stay_is_too_long) if (to_date - from_date).to_i > ENV.fetch('ACCOMMODATION_MAX_STAY_DAYS', 10).to_i
+    end
+  end
+end

data/app/serializers/spree/v2/storefront/accommodation_serializer.rb

@@ -9,10 +9,12 @@ module Spree
 
         attributes :total_inventory, :service_availabilities
 
+        # Deprecated
         attribute :total_booking do |vendor|
           vendor.respond_to?(:total_booking) ? vendor.total_booking : 0
         end
 
+        # Deprecated
         attribute :remaining do |vendor|
           vendor.respond_to?(:remaining) ? vendor.remaining : vendor.total_inventory
         end

data/app/serializers/spree/v2/storefront/cart_serializer_decorator.rb

@@ -12,6 +12,7 @@ module Spree
 
           # override to return all payments instead of only valid_payments
           base.has_many :payments
+          base.has_many :saved_guests, serializer: ::SpreeCmCommissioner::V2::Storefront::SavedGuestSerializer
         end
       end
     end

data/app/serializers/spree/v2/storefront/line_item_serializer_decorator.rb

@@ -4,10 +4,12 @@ module Spree
       module LineItemSerializerDecorator
         def self.prepended(base)
           base.attributes :from_date, :to_date, :need_confirmation, :product_type, :event_status, :amount, :display_amount,
+                          :checkin_date, :checkout_date,
                           :number, :qr_data,
                           :kyc, :kyc_fields, :remaining_total_guests, :number_of_guests,
                           :completion_steps, :available_social_contact_platforms, :allow_anonymous_booking, :discontinue_on,
-                          :high_demand, :jwt_token
+                          :high_demand, :jwt_token,
+                          :direction, :trip_id, :boarding_trip_stop_id, :drop_off_trip_stop_id
 
           base.attribute :required_self_check_in_location, &:required_self_check_in_location?
           base.attribute :allowed_self_check_in, &:allowed_self_check_in?

data/app/serializers/spree/v2/storefront/user_serializer_decorator.rb

@@ -10,6 +10,7 @@ module Spree
           base.has_one :profile, serializer: ::Spree::V2::Storefront::UserProfileSerializer
           base.has_many :device_tokens, serializer: Spree::V2::Storefront::UserDeviceTokenSerializer
           base.has_many :spree_roles, serializer: Spree::V2::Storefront::RoleSerializer
+          base.has_many :user_identity_providers, serializer: SpreeCmCommissioner::V2::Storefront::UserIdentityProviderSerializer
         end
       end
     end

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

@@ -0,0 +1,15 @@
+module SpreeCmCommissioner
+  module V2
+    module Storefront
+      class AmenitySerializer < BaseSerializer
+        set_type :amenity
+        attributes :name, :presentation
+        attribute :display_icon do |option_value|
+          ::ActionController::Base.helpers.image_url(option_value.display_icon)
+        rescue StandardError
+          nil
+        end
+      end
+    end
+  end
+end

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

@@ -0,0 +1,11 @@
+module SpreeCmCommissioner
+  module V2
+    module Storefront
+      class BlockSerializer < BaseSerializer
+        attributes :label, :width, :height, :x, :y, :rotation, :block_type
+
+        attribute :seatable, &:seatable?
+      end
+    end
+  end
+end

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

@@ -7,7 +7,8 @@ module SpreeCmCommissioner
         attributes :first_name, :last_name, :dob, :gender, :kyc_fields, :other_occupation, :created_at, :updated_at,
                    :qr_data, :social_contact_platform, :social_contact, :available_social_contact_platforms,
                    :age, :emergency_contact, :other_organization, :expectation, :upload_later, :address, :formatted_bib_number, :phone_number,
-                   :token, :country_code, :contact, :data_fill_stage_phase
+                   :token, :country_code, :contact, :data_fill_stage_phase,
+                   :saved_guest_id
 
         # temporary, once app release after cm-app v1.9.1, we no longer need this.
         attribute :seat_number do |object|
@@ -21,6 +22,9 @@ module SpreeCmCommissioner
         belongs_to :occupation, serializer: Spree::V2::Storefront::TaxonSerializer
         belongs_to :nationality, serializer: Spree::V2::Storefront::TaxonSerializer
         belongs_to :line_item, serializer: Spree::V2::Storefront::LineItemSerializer
+        belongs_to :block, serializer: ::SpreeCmCommissioner::V2::Storefront::BlockSerializer
+
+        has_many :guest_dynamic_fields, serializer: SpreeCmCommissioner::V2::Storefront::GuestDynamicFieldSerializer
 
         has_one :id_card, serializer: Spree::V2::Storefront::IdCardSerializer
         has_one :check_in

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

@@ -0,0 +1,42 @@
+module SpreeCmCommissioner
+  module V2
+    module Storefront
+      class InventoryItemSerializer < BaseSerializer
+        include Spree::Api::V2::DisplayMoneyHelper
+
+        cache_options store: nil
+
+        attributes :variant_id,
+                   :inventory_date,
+                   :public_quantity_available
+
+        attribute :currency do |_item, params|
+          params[:currency]
+        end
+
+        # must includes(:prices)
+        attribute :price do |item, params|
+          price(item, params[:currency])
+        end
+
+        # must includes(:prices)
+        attribute :display_price do |item, params|
+          display_price(item, params[:currency])
+        end
+
+        # must includes(:prices)
+        attribute :compare_at_price do |item, params|
+          compare_at_price(item, params[:currency])
+        end
+
+        # must includes(:prices)
+        attribute :display_compare_at_price do |item, params|
+          display_compare_at_price(item, params[:currency])
+        end
+
+        has_many :reserved_blocks, serializer: ReservedBlockSerializer,
+                                   if: proc { |_, params| params && params[:include_reserved_blocks] == true }
+      end
+    end
+  end
+end

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

@@ -0,0 +1,9 @@
+module SpreeCmCommissioner
+  module V2
+    module Storefront
+      class ReservedBlockSerializer < BaseSerializer
+        attributes :block_id
+      end
+    end
+  end
+end

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

@@ -0,0 +1,21 @@
+module SpreeCmCommissioner
+  module V2
+    module Storefront
+      class SavedGuestSerializer < BaseSerializer
+        attributes :first_name,
+                   :last_name,
+                   :dob,
+                   :age,
+                   :gender,
+                   :email,
+                   :country_code,
+                   :phone_number,
+                   :intel_phone_number,
+                   :user_id
+
+        belongs_to :occupation, serializer: Spree::V2::Storefront::TaxonSerializer
+        belongs_to :nationality, serializer: Spree::V2::Storefront::TaxonSerializer
+      end
+    end
+  end
+end