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

Fixed: NPO Recent items. #1750

Merged
merged 2 commits into from
Dec 20, 2023
Merged

Fixed: NPO Recent items. #1750

merged 2 commits into from
Dec 20, 2023

Conversation

basrieter
Copy link
Collaborator

Fixing NPO recent list that broken.

@basrieter
Copy link
Collaborator Author

Graag de fix proberen via deze zip: https://codeload.github.com/retrospect-addon/plugin.video.retrospect/zip/refs/heads/npo-recent

Copy link

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@ErikBrown2
Copy link

Bas, dank je wel. Recent werkt inderdaad weer.

@basrieter
Copy link
Collaborator Author

Bas, dank je wel. Recent werkt inderdaad weer.

Er is wel weer wat veranderd, want ik moet nu echt weer alles voor een bepaalde datum opvragen. Dus de discussie over +5 uur is niet meer relevant.

@sjamaan
Copy link

sjamaan commented Dec 20, 2023

Kan bevestigen, met deze fix werkt het weer. Veel dank! 👍

@basrieter
Copy link
Collaborator Author

Zal vanavond proberen een release te doen.

@basrieter basrieter mentioned this pull request Dec 20, 2023
7 tasks
@PhyxionNL
Copy link
Contributor

Werkt prima.

@chakubie
Copy link

Idem, dank!

@Heleen-Heleen
Copy link

Heleen-Heleen commented Dec 20, 2023

Bedankt Bas, recent werkt weer perfect

@basrieter basrieter merged commit 5622883 into master Dec 20, 2023
13 checks passed
@basrieter basrieter deleted the npo-recent branch January 1, 2024 12:50
@Heleen-Heleen
Copy link

Heleen-Heleen commented Jan 3, 2024

de laatste dagen
heb ik NPO vergeleken met Retrospect
Opvallend is dat films totaal niet in recent staan
maar ook films van Zappbios niet, echter sommige weer wel

Retrospect.zip

Ze staan nu ongefilterd zoals je zelf graag wilde

@basrieter
Copy link
Collaborator Author

Thanks, deze voorbeelden zijn me helder. Ik ga er naar kijken.

@Heleen-Heleen
Copy link

Heleen-Heleen commented Jan 3, 2024

Thanks, deze voorbeelden zijn me helder. Ik ga er naar kijken.

Het is me overigens ook opgevallen dat in sommige gevallen enkele uitzendingen met vrijwel dezelfde naam opvolgend op elkaar niet in Retrospect worden herkend.
Voorbeeld: ABBA (31-12 zie zip) maar ik heb dit ook al eerder gezien met NOS Studio Sport
verleden week

@basrieter
Copy link
Collaborator Author

Zou je deze ZIP eens kunnen proberen: https://codeload.github.com/retrospect-addon/plugin.video.retrospect/zip/refs/heads/master

Volgens mij zijn de issues dan verholpen.

@Heleen-Heleen
Copy link

Heleen-Heleen commented Jan 3, 2024

Wauw, alles werkt weer, enorm bedankt, Bas

@basrieter
Copy link
Collaborator Author

Wauw, alles werkt weer, enorm bedankt, Bas

Mede dankzij de info die je aangeleverd hebt. Die laatste was echt super duidelijk. Er komt dit weekend een officiële update aan met o.a. Videoland erin. Ik zal deze fix meenemen daarin.

@Heleen-Heleen
Copy link

Heleen-Heleen commented Jan 4, 2024

4 januari 24.zip

zou het ook mogelijk zijn, je bent toch met een update bezig,
de teksten bij de afleveringen te plaatsen bij recent?
bij de meeste staat geen tekst of heel erg summier.
bij alle films staat helemaal geen tekst

@basrieter
Copy link
Collaborator Author

Dat wordt lastig: die info zit niet in de API van de EPG:

image

@ErikBrown2
Copy link

Omdat de programma beschrijvingen niet meer te zien zijn bij Recent gebruik ik nu graag 'Media info' in plaats van 'List' als het type van de programma lijst. Dan krijg je wel de beschrijving van het programma te zien. Maar deze setting wordt blijkbaar niet opgeslagen want ik moet telkens deze setting weer kiezen. Wat is verantwoordelijk voor het opslaan van deze setting? Is het Retrospect, Kodi of de skin?

@basrieter
Copy link
Collaborator Author

Dat is Kodi, maar die baseert het op de url en die hangt veranderd steeds. Zeker bij Recent.

@Heleen-Heleen
Copy link

er zijn toch wel 22 beschrijvingen op 4 januari
de meeste staan op NPO 3

NPO 1 met tekst.zip
NPO 2 met tekst.zip
NPO 3 met tekst.zip

Ik heb ze nog niet in een tijdlijn geplaatst omdat dat zeer tijdrovend is en als het niet nodig is dan doe ik dat liever niet

@basrieter
Copy link
Collaborator Author

er zijn toch wel 22 beschrijvingen op 4 januari de meeste staan op NPO 3

NPO 1 met tekst.zip NPO 2 met tekst.zip NPO 3 met tekst.zip

Ik heb ze nog niet in een tijdlijn geplaatst omdat dat zeer tijdrovend is en als het niet nodig is dan doe ik dat liever niet

Sommige plaatjes bij afleveringen hebben een extra omschrijving, die toon ik dan. Maar lang niet alle plaatjes. Dus ik toon wat er is (zoals je bij sommige dus ziet) en als er dus niks staat, dan is er niks aanwezig in de API. Daar kan ik echt niks aan doen.

@Heleen-Heleen
Copy link

Heleen-Heleen commented Jan 5, 2024

Jammer dat het niet gaat, maar in ieder geval werkt recent weer goed

sinds 1 december 23 werkt de epg van de optie tv ook niet meer
heeft dat ook hiermee te maken met onderstaand?

deze vond ik heel erg fijn, prettiger dan live tv op zich

epg

@basrieter
Copy link
Collaborator Author

Jammer dat het niet gaat, maar in ieder geval werkt recent weer goed

sinds 1 december 23 werkt de epg van de optie tv ook niet meer heeft dat ook hiermee te maken met onderstaand?

deze vond ik heel erg fijn, prettiger dan live tv op zich

epg

Dat kan goed. Die heeft @landgenoot toegevoegd en heb ik nog weinig mee gedaan nadat alles is veranderd bij NPO. Ik heb echt heel weinig tijd gehad en ben blij dat ik de basics werkend kan houden. Hopelijk kan @landgenoot een fix aanleveren?

@landgenoot
Copy link
Contributor

Ik zal kijken of het binnenkort kan fixen.

@Heleen-Heleen
Copy link

dat zou super zijn

@ErikBrown2
Copy link

Dat is Kodi, maar die baseert het op de url en die hangt veranderd steeds. Zeker bij Recent.

@basrieter, Is het niet mogelijk om een configuratie setting hiervoor in Retrospect te maken waar de gebruiker kan instellen wat het gewenste formaat van de programma lijst is in Retrospect? Dus als de gebruiker bijvoorbeeld 'media info' wil gebruiken als de standaard programma lijst formaat, dat dit onafhankelijk van Kodi, door de gebruiker ingesteld kan worden in Retrospect. Dat zou een mooie oplossing zijn voor de missende programma omschrijvingen bij NPO start.

@basrieter
Copy link
Collaborator Author

Dat is Kodi, maar die baseert het op de url en die hangt veranderd steeds. Zeker bij Recent.

@basrieter, Is het niet mogelijk om een configuratie setting hiervoor in Retrospect te maken waar de gebruiker kan instellen wat het gewenste formaat van de programma lijst is in Retrospect? Dus als de gebruiker bijvoorbeeld 'media info' wil gebruiken als de standaard programma lijst formaat, dat dit onafhankelijk van Kodi, door de gebruiker ingesteld kan worden in Retrospect. Dat zou een mooie oplossing zijn voor de missende programma omschrijvingen bij NPO start.

In Retrospect 5.7.5 heb ik iets veranderd, dat zou wellicht kunnen helpen. Maar het blijft gewoon een feit dat ik vanuit een add-on geen controle heb over wel views er worden gebruikt. Dat verschilt zelfs per skin. En omdat er URL echt steeds veranderd (andere datum bijv), kan ik er hier niks aan doen.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants