Skip to content

Fix card list filters to use correct API parameter names#10

Merged
robzolkos merged 1 commit intomasterfrom
fix-card-list-filter-params
Dec 11, 2025
Merged

Fix card list filters to use correct API parameter names#10
robzolkos merged 1 commit intomasterfrom
fix-card-list-filter-params

Conversation

@robzolkos
Copy link
Copy Markdown
Collaborator

Summary

  • Fixed board_idboard_ids[]
  • Fixed tag_idtag_ids[]
  • Fixed assignee_idassignee_ids[]
  • Removed --column filter (not supported by API)

The Fizzy API expects plural array params but the CLI was sending singular params, which the API silently ignored - causing all filter options to return unfiltered results.

Test plan

  • bundle exec rake test - all 103 tests pass
  • Real API test: fizzy card list --board=<id> returns only cards from that board
  • Real API test: fizzy card list --tag=<id> returns only cards with that tag
  • Real API test: fizzy card list --assignee=<id> returns only assigned cards
  • Real API test: Combined filters work correctly

The Fizzy API expects plural array params (board_ids[], tag_ids[],
assignee_ids[]) but we were sending singular params (board_id, tag_id,
assignee_id) which the API silently ignored.

Changes:
- board_id → board_ids[]
- tag_id → tag_ids[]
- assignee_id → assignee_ids[]
- Removed column filter (not supported by API)

Tested against real API - all filters now work correctly.
@robzolkos robzolkos merged commit 46a643a into master Dec 11, 2025
1 check passed
@robzolkos robzolkos deleted the fix-card-list-filter-params branch December 11, 2025 22:11
waynemsmith added a commit to Concurrent-Systems/fizzy-cli that referenced this pull request Mar 28, 2026
Parse @name patterns and replace with Fizzy ActionText mention
attachments so mentions trigger notifications. Fetches user sgids
from /prompts/users endpoint. Supports @firstname and @"Full Name"
syntax, with ambiguity warnings and email-safe matching.

Closes basecamp#10

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant