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

feat(Tidal): format textual copyright symbols as © or ℗ #53

Merged
merged 1 commit into from
Jul 9, 2024

Conversation

phw
Copy link
Collaborator

@phw phw commented Jul 9, 2024

On Tidal it s very common to have copyright symbols in textual form, e.g. "(p)" or "(C)", instead of the dedicated symbols ℗ or ©. The symbols are also used, but it is not mandatory.

Add a utility function formatCopyrightSymbols to handle this. Also use this function in the existing Spotify implementation for copyright cleanup.

This PR partially but not completely addresses #23. I wanted to fix the most prominent issue where Tidal shows "(p) 2024 Some Band", but Spotify and iTunes give "℗ 2024 Some Band". Now there is a single consistent copyright entry.

Previously for https://tidal.com/album/64944658:
grafik

Now:
grafik

What this PR does not address:

  • Handling copyright entries on Tidal without any symbol at all (textual or not). This needs more consideration, see my comment Normalize and merge copyright lines #23 (comment)
  • Considering Tidal copyright entries on track level
  • Considering any other provider (but only ones with copyright right now are iTunes and Spotify, and those seem to be fine)

Add a utility function formatCopyrightSymbols to handle this and use
it also in the Spotify implementation.
@phw phw added feature New feature or request Tidal labels Jul 9, 2024
Copy link
Owner

@kellnerd kellnerd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you, LGTM

@kellnerd kellnerd merged commit 0102dcf into kellnerd:main Jul 9, 2024
2 checks passed
@phw phw deleted the tidal-clean-copyright branch July 9, 2024 08:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request Tidal
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants