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

YouTube Music-IDs für Folgen hinzugefügt #79

Merged
merged 4 commits into from
Oct 14, 2024
Merged

Conversation

stehefan
Copy link
Contributor

Habe die IDs für alle Folgen hinzugefügt und Spot-Checks gemacht um die Richtigkeit zu überprüfen.

Die weiteren Informationen (Übersicht zu Schema etc) habe ich nicht weiter angepasst, da du dafür sicher einen somewhat-automatischen Workflow hast. Ansonsten reiche ich das noch nach :-)

@stehefan stehefan mentioned this pull request Oct 13, 2024
11 tasks
@YourMJK YourMJK self-assigned this Oct 14, 2024
@YourMJK YourMJK added Erweiterung Neues Feature oder Wunsch good first issue Good for newcomers labels Oct 14, 2024
@YourMJK
Copy link
Owner

YourMJK commented Oct 14, 2024

Vielen herzlichen Dank für den Beitrag, sieht sehr gut aus!

Ich denk mal, du hast das über den Titel gematcht, die doppelte ID bei den beiden gleichnamigen "und die schwarze Katze" hab' ich mal korrigiert ;)

Das Exportieren der JSON, TSV und Website ist natürlich automatisch.
Das Anpassen des Codes, der Diagramme etc., wenn ich das Modell ändere, ist dagegen leider mehr manuell als automatisch …
Aber nachdem das in letzter Zeit doch häufiger als gedacht passiert ist, habe ich darin zumindest Routine :D

@YourMJK YourMJK merged commit 5115c9d into YourMJK:master Oct 14, 2024
@stehefan
Copy link
Contributor Author

Ich denk mal, du hast das über den Titel gematcht, die doppelte ID bei den beiden gleichnamigen "und die schwarze Katze" hab' ich mal korrigiert ;)

Jap, hab ich 🙈 Ist dir das schon mehrfach passiert oder warum direkt raugeschaut? :-D Danke für den Merge, helfe gerne noch wenn es was braucht ✌️

@YourMJK
Copy link
Owner

YourMJK commented Oct 14, 2024

Ist dir das schon mehrfach passiert oder warum direkt raugeschaut? :-D

Im SQL sind die IDs zwar (noch) nicht als UNIQUE definiert, aber mein Code warnt mich beim Exportieren wenn es im Index eine ID bereits existiert.
Aber ich konnte es mir schon gleich denken, wer der Übeltäter ist. Ist mir tatsächlich schon mehrfach passiert :D

helfe gerne noch wenn es was braucht

Klar, ein paar Issues habe ich eh schon gelabelt mit "Hilfe gesucht"!
Ansonsten gäbe es noch die MusicBrainz- und Discogs-IDs in #28. Das ist aber komplizierter, da bräuchte ich nämlich auch die richtige Releases (die von/nach 2005, ohne "Europa Logo!").
Das hängt auch mit #34 zusammen, die Katalognummern und restlichen DiscIDs bekommt man nämlich auch gut von MusicBrainz.

@stehefan
Copy link
Contributor Author

Nice, schaue ich mir die Tage mal an und pinge dich im issue ✌️

@YourMJK YourMJK changed the title YouTube Music IDs für Folgen hinzugefuegt YouTube Music-IDs für Folgen hinzugefügt Oct 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Erweiterung Neues Feature oder Wunsch good first issue Good for newcomers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants