gambit/gambit
1
0
Code Issues 534 Packages Projects Wiki

TCGplayer link + market data point at the wrong printing for the selected variant (item detail, trade sheet) #619

New Issue
Open
opened 2026-09-05 01:17:10 +00:00 by gambit-admin · 0 comments
gambit-admin commented 2026-09-05 01:17:10 +00:00
Owner
Copy Link
Copy Source

Merged from #344, #345 (@rudy_tajalle, 2026-08-12, "FIX ASAP URGENT") and #377 (@rudy_tajalle, 2026-08-16). All three were claimed by Kris.

When a specific printing/variant is selected, the TCGplayer link and the market data shown point at a different card and price — on the item detail (#345), on the trade sheet (#377), and reported bare as "market price is incorrect/off" the same day (#344).

Likely the same class as #545 (fixed for inventory display in PR #369) and #548: card_cache is one row per card priced by its headline printing, and any surface that decorates a printing-grain item from the card-grain row shows the headline's link and price. Trade sheet and the TCGplayer link builder need the same printing-aware enrichment (_printing_variant_row in inventory_items.py is reusable).

Merged from #344, #345 (@rudy_tajalle, 2026-08-12, "FIX ASAP URGENT") and #377 (@rudy_tajalle, 2026-08-16). All three were claimed by Kris. When a specific printing/variant is selected, the TCGplayer link and the market data shown point at a different card and price — on the item detail (#345), on the trade sheet (#377), and reported bare as "market price is incorrect/off" the same day (#344). Likely the same class as #545 (fixed for inventory display in PR #369) and #548: card_cache is one row per card priced by its headline printing, and any surface that decorates a printing-grain item from the card-grain row shows the headline's link and price. Trade sheet and the TCGplayer link builder need the same printing-aware enrichment (`_printing_variant_row` in `inventory_items.py` is reusable).
gambit-admin added the bugapi03 · pricing & value labels 2026-09-05 01:17:10 +00:00
gambit-admin added the claimed:kris label 2026-09-05 01:18:04 +00:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
01 · scan & recognition
User-flow fix order step 01 (beginning->end)
 02 · card detail sheet
User-flow fix order step 02 (beginning->end)
 03 · pricing & value
User-flow fix order step 03 (beginning->end)
 05 · inventory list
User-flow fix order step 05 (beginning->end)
 06 · inventory detail / rcn
User-flow fix order step 06 (beginning->end)
 07 · home / dashboard
User-flow fix order step 07 (beginning->end)
 08 · binders
User-flow fix order step 08 (beginning->end)
 09 · trade sheet
User-flow fix order step 09 (beginning->end)
 11 · sell / pos / marketplace
User-flow fix order step 11 (beginning->end)
 12 · analytics / trends
User-flow fix order step 12 (beginning->end)
 13 · cross-cutting
User-flow fix order step 13 (beginning->end)
 api
bug
claimed:Kelly
claimed:josh
claimed:kris
claimed:lucas
claimed:root
claimed:rudy
claimed:rudy2
crash
documentation
enhancement
export
feature
hardware
in progress (design)
Actively being worked on the design app (run.rtcg.app), not yet on production
 performance
security
someday
Parked wishlist — not scheduled, revisit when the roadmap has room
 ui
No Label 03 · pricing & value api bug claimed:kris
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
gambit-admin
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: gambit/gambit#619
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?