Skip to content
New issue

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

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

Already on GitHub? # to your account

Import Episode, Media & Verify #19

Merged
merged 20 commits into from
May 30, 2024
Merged

Import Episode, Media & Verify #19

merged 20 commits into from
May 30, 2024

Conversation

eteubert
Copy link
Member

@eteubert eteubert commented May 21, 2024

Status

  • findet oder erstellt eine neue Episode basierend auf der guid
  • schreibt einige Episoden Metadaten (siehe Beispiel Payload)
  • lädt enclosure herunter und importiert sie in WordPress
  • verifiziert Audio-URL (und aktiviert das Asset bei Erfolg)

Es fehlt noch einiges an Daten, die importiert werden können, schlage aber vor, das in diesem Zustand zu mergen und den Rest in kleineren PRs nachzuliefern.

Beispiel Payload

{
  "guid": "foobar-123",
  "title": "LOV001 Lorem Ipsum",
  "subtitle": "Talking about endangered species.",
  "summary": "The most exciting episode on the planet. And the universe.",
  "number": "1",
  "explicit": "false",
  "slug": "lov001-lorem-ipsum",
  "duration": "00:00:05.108",
  "type": "full",
  "enclosure": "https://eric.co.de/u/pling.mp3"
}

Notizen

  • dev API Endpunkt vorübergehend auf Port 4005 gelegt, damit es sich nicht mit Local beißt
  • File Upload funktioniert nur richtig, wenn das "WordPress Upload" Modul an und der Pfad konfiguriert ist. Muss auch automatisiert passieren. bevor der Episoden Import startet.

@eteubert eteubert changed the title Draft/WIP: Import Episode Import Episode, Media & Verify May 24, 2024
@eteubert eteubert requested review from alexander-heimbuch and dnkbln and removed request for alexander-heimbuch and dnkbln May 24, 2024 10:35
Copy link
Contributor

@dnkbln dnkbln left a comment

Choose a reason for hiding this comment

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

Für mich sehen die Änderungen gut aus.

Was mir beim Test auffiel, dass der Media-Upload nach uploads/2024/05/27 erfolgte und dadurch die Mediafile in den Episoden nicht gefunden wurden. Der Upload und das Anlegen der Episoden war aber erfolgreich.

@eteubert
Copy link
Member Author

@dnkbln Ist dein Publisher aktuell? Der Uploadpfad funktioniert nur korrekt seit diesem commit: podlove/podlove-publisher@505312d

(und das WordPress File Upload Modul muss überhaupt an sein)

@dnkbln
Copy link
Contributor

dnkbln commented May 28, 2024

@eteubert Ich hatte zwar die aktuelle beta, aber nicht den Branch change/episode-guid-generation. Mit diesem Branch kann ich mit Postman Episoden durch den Server erzeugen lassen und die Media-Files werden auch richtig im Publisher referenziert

@eteubert
Copy link
Member Author

Oh entschuldige @dnkbln , mein Fehler @dnkbln 🙈

@dnkbln dnkbln merged commit d070716 into main May 30, 2024
3 checks passed
@eteubert eteubert deleted the feat/import-episode branch June 1, 2024 12:59
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants