Conversation
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
Adds support for displaying a TheGamesDB (TGDB) link in the platform info drawer when a platform has tgdb_id set, aligning behavior with other metadata providers and addressing issue #3251.
Changes:
- Add a TGDB chip/link to
PlatformInfoDrawer.vuewhencurrentPlatform.tgdb_idis present. - Populate
tgdb_idduring platform scanning using the TGDB metadata handler as an additional fallback.
Reviewed changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| frontend/src/components/Gallery/AppBar/Platform/PlatformInfoDrawer.vue | Adds TGDB external link chip in the platform drawer UI when tgdb_id exists. |
| backend/handler/scan_handler.py | Extends scan_platform to fill tgdb_id from TGDB handler when other sources don’t provide it. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| "ra_id": ra_platform.get("ra_id") or hasheous_platform.get("ra_id") or None, | ||
| "tgdb_id": moby_platform.get("tgdb_id") | ||
| or hasheous_platform.get("tgdb_id") | ||
| or tgdb_platform.get("tgdb_id") | ||
| or None, |
There was a problem hiding this comment.
This change introduces a new fallback for tgdb_id coming directly from the TGDB metadata handler. There is existing test coverage for scan_platform, but it currently doesn't assert tgdb_id; please extend/add a test case to verify tgdb_id is populated when TGDB provides it (e.g., for n64) and other sources don't override it.
☂️ Python Coverage
Overall Coverage
New FilesNo new covered files... Modified Files
|
Description
Explain the changes or enhancements you are proposing with this pull request.
Fixes #3251
Checklist
Please check all that apply.