Skip to content

feat(playmatch): move integration to API v2#3689

Merged
gantoine merged 3 commits into
rommapp:masterfrom
DevYukine:feat/playmatch-api-v2
Jul 7, 2026
Merged

feat(playmatch): move integration to API v2#3689
gantoine merged 3 commits into
rommapp:masterfrom
DevYukine:feat/playmatch-api-v2

Conversation

@DevYukine

Copy link
Copy Markdown
Contributor

Description
Moves the Playmatch integration to the new API v2 endpoints, since v2 is the version going forward. Older clients (pre 5.0) stay supported on v1, they just won't receive new features.

While at it, this also sends the CRC32 hash as an additional identify lookup key, which v2 supports.

Disclosure
I am not a Python developer and am not very familiar with the RomM codebase. I used Claude to help with the endpoint switch and to validate the change against the Playmatch v2 OpenAPI spec. I reviewed the result before submitting.

Checklist
Please check all that apply.

  • I've tested the changes locally
  • I've updated relevant comments
  • I've assigned reviewers for this PR
  • I've added unit tests that cover the changes (existing cassette updated to the v2 URL)

Screenshots (if applicable)

Backend change only, no screenshots needed

@gantoine gantoine added the on-hold Pending further research or blocked by another issue label Jul 6, 2026
@gantoine gantoine removed the on-hold Pending further research or blocked by another issue label Jul 7, 2026
@gantoine
gantoine merged commit 30274cb into rommapp:master Jul 7, 2026
6 checks passed
@gantoine

gantoine commented Jul 7, 2026

Copy link
Copy Markdown
Member

changed my mind, this is making the cut lol

@DevYukine
DevYukine deleted the feat/playmatch-api-v2 branch July 9, 2026 02:00
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.

2 participants