feat: implement multi roster

Author: Bulletdev

Gemfile.lock

@@ -521,7 +521,7 @@ DEPENDENCIES
   webmock
 
 RUBY VERSION
-   ruby 3.4.5p51
+   ruby 3.4.8p72
 
 BUNDLED WITH
    2.3.27

app/controllers/api/v1/organizations_controller.rb

@@ -55,6 +55,21 @@ def upload_logo
         )
       end
 
+      # PATCH /api/v1/organizations/:id/lines
+      def update_lines
+        lines = Array(params[:enabled_lines]).select { |l| l.in?(Constants::Player::LINES) }
+
+        if lines.empty?
+          return render_error(message: 'At least one valid line is required', code: 'VALIDATION_ERROR',
+                              status: :unprocessable_entity)
+        end
+
+        lines = (['main'] | lines).uniq
+        @organization.update!(enabled_lines: lines)
+
+        render json: { message: 'Roster lines updated', enabled_lines: @organization.enabled_lines }, status: :ok
+      end
+
       private
 
       def set_organization
@@ -63,10 +78,10 @@ def set_organization
       end
 
       def require_admin_or_owner
-        return if %w[admin owner].include?(@current_user.role)
+        return if %w[admin owner coach].include?(@current_user.role)
 
         render_error(
-          message: 'Only admins and owners can update organization settings',
+          message: 'Only coaches, admins and owners can update organization settings',
           code: 'FORBIDDEN',
           status: :forbidden
         )

app/models/concerns/constants.rb

@@ -30,7 +30,7 @@ module Organization
 
   SOURCE_APP_URLS = {
     'prostaff' => ENV.fetch('PROSTAFF_URL', 'https://prostaff.gg'),
-    'scrims'   => ENV.fetch('SCRIMS_URL', 'https://scrims.lol'),
+    'scrims' => ENV.fetch('SCRIMS_URL', 'https://scrims.lol'),
     'arena_br' => ENV.fetch('ARENA_BR_URL', 'https://arena-br.vercel.app')
   }.freeze
 
@@ -51,6 +51,7 @@ module User
   module Player
     ROLES = %w[top jungle mid adc support].freeze
     STATUSES = %w[active inactive benched trial removed].freeze
+    LINES = %w[main academy farm female other].freeze
     QUEUE_RANKS = %w[I II III IV].freeze
     QUEUE_TIERS = %w[IRON BRONZE SILVER GOLD PLATINUM EMERALD DIAMOND MASTER GRANDMASTER CHALLENGER].freeze
 

app/modules/core/serializers/organization_serializer.rb

@@ -6,7 +6,7 @@ class OrganizationSerializer < Blueprinter::Base
   identifier :id
 
   fields :name, :slug, :team_tag, :region, :tier, :subscription_plan, :subscription_status,
-         :logo_url, :settings, :created_at, :updated_at,
+         :logo_url, :settings, :enabled_lines, :created_at, :updated_at,
          :trial_expires_at, :trial_started_at
 
   field :region_display do |org|

app/modules/players/controllers/players_controller.rb

@@ -20,6 +20,7 @@ def index
 
         players = players.by_role(params[:role]) if params[:role].present?
         players = players.by_status(params[:status]) if params[:status].present?
+        players = players.by_line(params[:line]) if params[:line].present?
 
         if params[:search].present?
           search_term = "%#{params[:search]}%"
@@ -178,13 +179,14 @@ def import
         summoner_name = params[:summoner_name]&.strip
         role = params[:role]
         region = params[:region] || 'br1'
+        line = params[:line].presence_in(Constants::Player::LINES) || 'main'
 
         # Validations
         return unless validate_import_params(summoner_name, role)
         return unless validate_player_uniqueness(summoner_name)
 
         # Import from Riot API
-        result = import_player_from_riot(summoner_name, role, region)
+        result = import_player_from_riot(summoner_name, role, region, line)
 
         # Handle result
         result[:success] ? handle_import_success(result) : handle_import_error(result)
@@ -426,12 +428,13 @@ def validate_player_uniqueness(summoner_name)
       end
 
       # Import player from Riot API
-      def import_player_from_riot(summoner_name, role, region)
+      def import_player_from_riot(summoner_name, role, region, line = 'main')
         RiotSyncService.import(
           summoner_name: summoner_name,
           role: role,
           region: region,
-          organization: current_organization
+          organization: current_organization,
+          line: line
         )
       end
 

app/modules/players/models/player.rb

@@ -56,6 +56,7 @@ class Player < ApplicationRecord # rubocop:disable Metrics/ClassLength
   validates :role, presence: true, inclusion: { in: Constants::Player::ROLES }
   validates :country, length: { maximum: 2 }
   validates :status, inclusion: { in: Constants::Player::STATUSES }
+  validates :line, inclusion: { in: Constants::Player::LINES }
   validates :riot_puuid, uniqueness: true, allow_blank: true
   validates :riot_summoner_id, uniqueness: true, allow_blank: true
   validates :jersey_number, uniqueness: { scope: :organization_id }, allow_blank: true
@@ -75,6 +76,7 @@ class Player < ApplicationRecord # rubocop:disable Metrics/ClassLength
   # Scopes
   scope :by_role, ->(role) { where(role: role) }
   scope :by_status, ->(status) { where(status: status) }
+  scope :by_line, ->(line) { where(line: line) }
   scope :active, -> { where(status: 'active') }
   scope :with_contracts, -> { where.not(contract_start_date: nil) }
   scope :contracts_expiring_soon, lambda { |days = 30|

app/modules/players/serializers/player_serializer.rb

@@ -5,7 +5,7 @@
 class PlayerSerializer < Blueprinter::Base
   identifier :id
 
-  fields :summoner_name, :real_name, :role, :status,
+  fields :summoner_name, :real_name, :role, :status, :line,
          :jersey_number, :birth_date, :country,
          :contract_start_date, :contract_end_date,
          :solo_queue_tier, :solo_queue_rank, :solo_queue_lp,

app/modules/players/services/riot_sync_service.rb

@@ -55,13 +55,13 @@ def initialize(organization, region = nil)
   end
 
   # Class method to import a new player from Riot API
-  def self.import(summoner_name:, role:, region:, organization:)
+  def self.import(summoner_name:, role:, region:, organization:, line: 'main')
     service = new(organization, region)
-    service.import_player(summoner_name, role)
+    service.import_player(summoner_name, role, line: line)
   end
 
   # Import a new player from Riot API
-  def import_player(summoner_name, role)
+  def import_player(summoner_name, role, line: 'main')
     parsed_name = parse_summoner_name(summoner_name)
     return parsed_name unless parsed_name[:success]
 
@@ -85,7 +85,8 @@ def import_player(summoner_name, role)
       solo_queue_losses: riot_data[:rank_data]['losses'] || 0,
       last_sync_at: Time.current,
       sync_status: 'success',
-      region: @region
+      region: @region,
+      line: line
     )
 
     {

app/modules/players/services/roster_management_service.rb

@@ -62,15 +62,14 @@ def remove_from_roster(reason:)
   # @param jersey_number [Integer] Jersey number (optional)
   # @return [Hash] Result with success status and player
   def self.hire_from_scouting(scouting_target:, organization:, contract_start:, contract_end:,
-                              salary: nil, jersey_number: nil, current_user: nil)
+                              salary: nil, jersey_number: nil, line: 'main', current_user: nil)
     ActiveRecord::Base.transaction do
-      # Check if this is a free agent or needs to be restored
       player = find_or_restore_player(scouting_target, organization)
 
-      # Update player with new contract details
       player.update!(
         organization: organization,
         status: 'active',
+        line: line.presence_in(Constants::Player::LINES) || 'main',
         contract_start_date: contract_start,
         contract_end_date: contract_end,
         salary: salary,

app/modules/scouting/controllers/players_controller.rb

@@ -115,6 +115,7 @@ def import_to_roster
           contract_end: params[:contract_end].present? ? Date.parse(params[:contract_end]) : nil,
           salary: params[:salary]&.to_d,
           jersey_number: params[:jersey_number]&.to_i,
+          line: params[:line],
           current_user: current_user
         )