Add column filtering support to cards API

app/models/filter.rb

@@ -28,6 +28,7 @@ def cards
       result = result.assigned_to(assignees.ids) if assignees.present?
       result = result.where(creator_id: creators.ids) if creators.present?
       result = result.where(board: boards.ids) if boards.present?
+      result = result.where(column: columns.ids) if columns.present?
       result = result.tagged_with(tags.ids) if tags.present?
       result = result.where(cards: { created_at: creation_window }) if creation_window
       result = result.closed_at_window(closure_window) if closure_window

app/models/filter/params.rb

@@ -12,6 +12,7 @@ module Filter::Params
     creator_ids: [],
     closer_ids: [],
     board_ids: [],
+    column_ids: [],
     tag_ids: [],
     terms: []
   ]
@@ -57,7 +58,8 @@ def as_params
       params[:assignment_status] = assignment_status
       params[:terms]             = terms
       params[:tag_ids]           = tags.ids
-      params[:board_ids]    = boards.ids
+      params[:board_ids]         = boards.ids
+      params[:column_ids]        = columns.ids
       params[:card_ids]          = card_ids
       params[:assignee_ids]      = assignees.ids
       params[:creator_ids]       = creators.ids

app/models/filter/resources.rb

@@ -4,6 +4,7 @@ module Filter::Resources
   included do
     has_and_belongs_to_many :tags
     has_and_belongs_to_many :boards
+    has_and_belongs_to_many :columns
     has_and_belongs_to_many :assignees, class_name: "User", join_table: "assignees_filters", association_foreign_key: "assignee_id"
     has_and_belongs_to_many :creators, class_name: "User", join_table: "creators_filters", association_foreign_key: "creator_id"
     has_and_belongs_to_many :closers, class_name: "User", join_table: "closers_filters", association_foreign_key: "closer_id"

db/migrate/20251220080000_create_columns_filters.rb

@@ -0,0 +1,10 @@
+class CreateColumnsFilters < ActiveRecord::Migration[8.0]
+  def change
+    create_table :columns_filters, id: false do |t|
+      t.uuid :column_id, null: false
+      t.uuid :filter_id, null: false
+      t.index :column_id
+      t.index :filter_id
+    end
+  end
+end

docs/API.md

@@ -499,6 +499,7 @@ __Query Parameters:__
 | Parameter | Description |
 |-----------|-------------|
 | `board_ids[]` | Filter by board ID(s) |
+| `column_ids[]` | Filter by column ID(s) |
 | `tag_ids[]` | Filter by tag ID(s) |
 | `assignee_ids[]` | Filter by assignee user ID(s) |
 | `creator_ids[]` | Filter by card creator ID(s) |

test/fixtures/columns_filters.yml

@@ -0,0 +1 @@
+# empty - populated dynamically in tests

test/models/filter_test.rb

@@ -45,6 +45,16 @@ class FilterTest < ActiveSupport::TestCase
     assert_empty users(:david).filters.new(board_ids: [ boards(:writebook).id ]).boards
   end
 
+  test "filter by column" do
+    filter = users(:david).filters.new column_ids: [ columns(:writebook_in_progress).id ]
+    assert_equal [ cards(:text) ], filter.cards
+
+    filter = users(:david).filters.new column_ids: [ columns(:writebook_triage).id ]
+    assert_includes filter.cards, cards(:logo)
+    assert_includes filter.cards, cards(:layout)
+    assert_not_includes filter.cards, cards(:text)
+  end
+
   test "remembering equivalent filters" do
     assert_difference "Filter.count", +1 do
       filter = users(:david).filters.remember(sorted_by: "latest", assignment_status: "unassigned", tag_ids: [ tags(:mobile).id ])
@@ -63,8 +73,8 @@ class FilterTest < ActiveSupport::TestCase
   end
 
   test "turning into params" do
-    filter = users(:david).filters.new sorted_by: "latest", tag_ids: "", assignee_ids: [ users(:jz).id ], board_ids: [ boards(:writebook).id ]
-    expected = { assignee_ids: [ users(:jz).id ], board_ids: [ boards(:writebook).id ] }
+    filter = users(:david).filters.new sorted_by: "latest", tag_ids: "", assignee_ids: [ users(:jz).id ], board_ids: [ boards(:writebook).id ], column_ids: [ columns(:writebook_in_progress).id ]
+    expected = { assignee_ids: [ users(:jz).id ], board_ids: [ boards(:writebook).id ], column_ids: [ columns(:writebook_in_progress).id ] }
     assert_equal expected, filter.as_params
   end