Recordings 2: add independent mobile recording browser - #89
Merged
Conversation
Owner
Author
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Ziel
Führt einen vollständig neuen, mobilen Aufnahmebrowser unter Recordings 2 ein. Der bestehende Aufnahmebrowser unter Aufnahmen bleibt unverändert und dient weiterhin als Referenz.
Ursache des bisherigen Platzhalters
Der vorhandene Tab fiel in den generischen Modul-Platzhalter der Frontend-Shell. Für
recordings2gab es weder einen eigenen Runtime-Einstieg noch Datenhaltung, Navigation oder Rendering. Dadurch erschienListe aus undefined rendern.Umsetzung
web/frontend/recordings2.jswindow.VdrSuiteRecordingBrowser/api/vdr/recordings/folderBewusst nicht geändert
web/frontend/modules/recordings.jsweb/frontend/recording-browser.jsGezielte Prüfungen
Erfolgreich ausgeführt, ohne auf GitHub Actions zu warten:
node --check web/frontend/recordings2.jsnode --check web/frontend/platform/deferred-runtime-loader.jsnode web/frontend/tests/test_recordings2_runtime.jspython3 tools/check_recordings2_runtime_wiring.pytest-recordings2-runtimetest-recordings2-install-stagingPersonen-/Galerie-Nachweis
Die Besetzungs- und Bilderimplementierung aus PR #87 ist im Ausgangsstand vorhanden und wird vom Installationsbundle weiterhin ausgeliefert. Der konkrete verbleibende Integrationsbruch liegt im kanonischen Channels 2: dessen privater Detailrenderer ruft
VdrSuiteEpgMetadataDetail.enhance(...)nicht auf. Der bestehende Hook erweitert derzeit nurcreateEpgEventDetailCardaus der EPG-Timeline.Dieser Folgefix wird bewusst nicht als oberflächlicher DOM-Hack mit Recordings 2 vermischt. Der Branch basiert direkt auf
mainbeic31fa1anach PR #87 und enthält keine Änderungen aus dem ungemergten PR #88.