Skip to content

TCGPlayer market ref, population grading, price history fixes#52

Merged
Pyronewbic merged 1 commit into
mainfrom
dev
May 12, 2026
Merged

TCGPlayer market ref, population grading, price history fixes#52
Pyronewbic merged 1 commit into
mainfrom
dev

Conversation

@Pyronewbic
Copy link
Copy Markdown
Owner

Summary

  • TCGPlayer market price reference in price chart section (with price sanity check)
  • Population-aware expected outcome in ROI card (likely PSA grade + scarcity)
  • TCGPlayer search: full query first, fallback to simplified, wrong-card filter
  • Demo price history: sold data spans 30+ days with realistic dates
  • Price chart: hidden when < 3 points, redraws on Prices tab activation (fixes blank canvas)
  • Removed redundant JP Market section (duplicated arbitrage)
  • Updated PR template with breaking changes + demo data checks

Breaking changes

None

Demo data

  • Sample data still works (?demo=true)
  • No new fields missing from demo cards

Manual verification

  • Greninja: Grade tab default, switch to Prices — chart draws correctly
  • Umbreon: ROI card shows expected outcome with scarcity

TCGPlayer market price shown as a reference in the price chart section
with market price, median, and printing type. Population-aware expected
outcome in ROI card: maps AI pre-grade to likely PSA grade, shows
population count and scarcity indicator (Scarce/Moderate/Common).
Updated PR template with breaking changes and demo data checks.
@Pyronewbic Pyronewbic merged commit 3481cb1 into main May 12, 2026
2 checks passed
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