Implement or remove the is_trade parameter from GET /api/v3/transactions #27

New Issue

cal · 2026-02-20T06:50:14Z

cal commented

2026-02-20 06:50:14 +00:00

app/routers_v3/transactions.py:77-78 — The is_trade parameter is documented in OpenAPI but raises 501. Either implement it or remove it from the API signature.

Priority: low | Labels: todo, tech-debt

`app/routers_v3/transactions.py:77-78` — The `is_trade` parameter is documented in OpenAPI but raises 501. Either implement it or remove it from the API signature. **Priority**: low | **Labels**: todo, tech-debt

cal added the

ai-working

label 2026-03-06 05:01:01 +00:00

cal referenced this issue from a commit

2026-03-06 05:02:44 +00:00

fix: remove unimplemented is_trade parameter from GET /api/v3/transactions (#27)

cal referenced a pull request that will close this issue

2026-03-06 05:02:53 +00:00

fix: remove unimplemented is_trade parameter from GET /api/v3/transactions (#27) #57

cal removed the

ai-working

label 2026-03-06 05:03:00 +00:00

cal commented

2026-03-06 05:03:02 +00:00

Opened PR #57: #57

The Transaction model has no is_trade field, so implementation would require a schema migration. Removed the parameter from the function signature and the 501 stub block instead.

Opened PR #57: https://git.manticorum.com/cal/major-domo-database/pulls/57 The `Transaction` model has no `is_trade` field, so implementation would require a schema migration. Removed the parameter from the function signature and the 501 stub block instead.

cal added the

ai-pr-opened

label 2026-03-06 05:03:08 +00:00

Sign in to join this conversation.