fix: remove unimplemented is_trade parameter from GET /api/v3/transactions (#27) #57
Open
cal
wants to merge 1 commits from
ai/major-domo-database-27 into next-release
pull from: ai/major-domo-database-27
merge into: cal:next-release
cal:main
cal:issue/101-fieldingstats-get-totalstats-total-count-overwritt
cal:issue/71-refactor-manual-db-close-calls-to-middleware-based
cal:fix/remove-hardcoded-webhook
cal:issue/81-add-migration-tracking-system
cal:issue/74-add-missing-indexes-on-foreign-key-columns-in-high
cal:issue/69-boolean-fields-compared-as-integers-sqlite-pattern
cal:issue/75-fix-n-1-query-pattern-in-standings-recalculation
cal:issue/77-replace-row-by-row-delete-with-bulk-delete-in-care
cal:issue/82-align-python-version-between-docker-image-3-11-and
cal:issue/76-replace-deprecated-pydantic-dict-with-model-dump
cal:issue/70-remove-sqlite-fallback-code-from-db-engine-py
cal:issue/78-fix-discord-id-type-mismatch-between-model-charfie
cal:issue/80-disable-autoconnect-and-set-pool-timeout-on-pooled
cal:issue/73-add-type-annotations-to-untyped-query-parameters
cal:issue/79-stop-logging-raw-auth-tokens-in-warning-messages
cal:ai/major-domo-database-66
cal:ai/major-domo-database-65
cal:next-release
cal:ai/major-domo-database-38
cal:ai/major-domo-database-19
cal:ai/major-domo-database-20
cal:ai/major-domo-database-22
cal:ai/major-domo-database-26
cal:ai/major-domo-database-31
cal:ai/major-domo-database-35
cal:feat/sbaplayer-id-plays-filter
cal:chore/optimize-claude-md
1 Commits
| Author | SHA1 | Message | Date | |
|---|---|---|---|---|
Cal Corum
|
1e8ab88690 |
fix: remove unimplemented is_trade parameter from GET /api/v3/transactions (#27)
All checks were successful
Build Docker Image / build (pull_request) Successful in 2m9s
The Transaction model has no is_trade field, so filtering on it cannot be implemented without a schema migration. Removed the parameter from the function signature and the 501 stub block. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> |