akadmin/chatwoot
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

fix: guard migration against non-numeric max_assignment_limit values

Browse Source
This commit is contained in:
Tanmay Deep Sharma
2026-03-16 19:51:02 +05:30
parent 3a99d7fad6
commit b8d35e8749
1 changed files with 6 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
db/migrate/20260316000001_migrate_max_assignment_limit_to_policies.rb
+6 -2
View File
@@ -1,4 +1,7 @@
class MigrateMaxAssignmentLimitToPolicies < ActiveRecord::Migration[7.1]
# Ruby's max for a 32-bit signed integer, matching PostgreSQL's integer column limit
INT_MAX = (2**31) - 1
def up
Account.find_each do |account|
migrate_account(account)
@@ -13,7 +16,8 @@ class MigrateMaxAssignmentLimitToPolicies < ActiveRecord::Migration[7.1]
def migrate_account(account)
inboxes_with_limit = account.inboxes
.where("(auto_assignment_config->>'max_assignment_limit')::int > 0")
.where("auto_assignment_config->>'max_assignment_limit' ~ '^[0-9]+$'")
.where("(auto_assignment_config->>'max_assignment_limit')::numeric > 0")
return if inboxes_with_limit.empty?
@@ -31,7 +35,7 @@ class MigrateMaxAssignmentLimitToPolicies < ActiveRecord::Migration[7.1]
inboxes.each do |inbox|
next if InboxCapacityLimit.exists?(agent_capacity_policy_id: policy.id, inbox_id: inbox.id)
limit = inbox.auto_assignment_config['max_assignment_limit'].to_i
limit = [inbox.auto_assignment_config['max_assignment_limit'].to_i, INT_MAX].min
InboxCapacityLimit.create!(
agent_capacity_policy: policy,