Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(ecau): copy MusicBrainz complex artwork types #716

Merged
merged 1 commit into from
Dec 2, 2023

Conversation

ROpdebee
Copy link
Owner

@ROpdebee ROpdebee commented Dec 1, 2023

"Matrix/Runout" and "Raw/Unedited" could not be copied when a MB release's images were copied because the enum used the "Matrix" and "Raw" keys only. Instead, we'll use the full artwork type name as keys. This also requires changes in VGMdb's caption to artwork type mappings and related tests. Also add test case with a real release.

Fixes #708

@ROpdebee ROpdebee added bug Something isn't working mb_enhanced_cover_art_uploads labels Dec 1, 2023
@ROpdebee ROpdebee force-pushed the ecau-fix-mb-raw-type branch from ca1e9f5 to c1fdc04 Compare December 1, 2023 15:02
@ROpdebee
Copy link
Owner Author

ROpdebee commented Dec 1, 2023

/deploy-preview

Copy link

github-actions bot commented Dec 1, 2023

This PR changes 1 built userscript(s):

See all changes

github-actions bot added a commit that referenced this pull request Dec 1, 2023
fix(ecau): copy MusicBrainz complex artwork types (#716)
"Matrix/Runout" and "Raw/Unedited" could not be copied when a MB
release's images were copied because the enum used the "Matrix" and
"Raw" keys only. Instead, we'll use the full artwork type name as
keys. This also requires changes in VGMdb's caption to artwork type
mappings and related tests. Also add test case with a real release.
@ROpdebee ROpdebee force-pushed the ecau-fix-mb-raw-type branch from c1fdc04 to ed07226 Compare December 2, 2023 10:16
@ROpdebee ROpdebee merged commit d9c8120 into main Dec 2, 2023
@ROpdebee ROpdebee deleted the ecau-fix-mb-raw-type branch December 2, 2023 10:18
Copy link

github-actions bot commented Dec 2, 2023

🚀 Released 1 new userscript version(s):

  • mb_enhanced_cover_art_uploads 2023.12.2 in ff95a07

github-actions bot added a commit that referenced this pull request Dec 2, 2023
fix(ecau): copy MusicBrainz complex artwork types (#716)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

MusicBrainz Provider drops the Raw/Unedited image type
1 participant