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

K dispozici je jen poslední díl #32

Open
marosrojis opened this issue Dec 1, 2019 · 5 comments
Open

K dispozici je jen poslední díl #32

marosrojis opened this issue Dec 1, 2019 · 5 comments

Comments

@marosrojis
Copy link

Issue #30 řešilo problém, že byl k dispozici jen poslední díl. Nyní se tento problém ve verzi 1.0.19 vyskytuje znova.

Zkusil jsem použít verzi 1.0.18, která minulý týden nefungovala. Nyní začala fungovat (https://github.com/kodi-czsk/plugin.video.dmd-czech.novaplus/tree/e7fc766de5aa0b278e35f64baccad8af4b563eb3).

Navrhuji provést revert posledního commitu.

Děkuji za pomoc :)

@elring
Copy link
Contributor

elring commented Dec 1, 2019

Super. Nějak se nemůžou rozhodnout, jak stránky budou vypadat :-)
Revert není správná cesta, protože tam byly další opravy (změnili formát zápisu rozlišení apod.).

Opravil jsem to tak, že jsou (teoreticky) funkční obě varianty dalších dílů. Ke stažení je testovací verze opět zde.
Jako bonus jsou opraveny úvodní kategorie - nově tam jsou (mimo Seriály a pořady a Televizních novin) jen Top pořady, Poslední díly a Bonusy. Ostatní jsou na Nova Plus zrušeny. Uvidíme, jak to dlouho takto vydrží.

@marosrojis
Copy link
Author

Máš pravdu, tohle je lepší varianta než revert :)

Funguje, díky moc za rychlou opravu.

@wombat01
Copy link

wombat01 commented Dec 2, 2019

Nedomluvíme se, abychom to sjednotili? Na "novém" repu je fork z tohoto, není zájem si každý jet na svém písečku, pokud je doplněk živý. Problém ovšem je, že tam je změněné addonid z důvodu sjednocení názvů.

https://github.com/xbmc-kodi-cz/plugin.video.novaplus.cz/

@elring
Copy link
Contributor

elring commented Dec 2, 2019

Ahoj. Je to trošku offtopic, ale nenašel jsem kontakt na Tebe, tak odpovím tady :-) Na repo jsem už i koukal (a i připravil opravy, co jsem dělal tady, ale ještě nevymyslel, jestli commitovat přímo nebo do branche, ale...). Tady to už nějaký pátek udržuji, i když to už vlastně moc nepoužívám. Ale baví mě to (učit se Python) a už jsem zvažoval požádat adminy o commit access...

Z principu mi přijde škoda dělat forky. Už máme fork O2TV, Prima a určitě spoustu dalších... Takže to pro každého uživatele znamená si ručně doplňky aktualizovat. Když jsou součástí kodi-czsk, tak stačí přidat repozitář a aktualizují se samy. To vidím jako velkou výhodu...

Takže pokud chceš udržovat funkčnější verzi doplňku tady, než jsou moje jen opravy funkcí, tak směle do toho, vůbec netrvám na tom, že to musím tvořit já :-)

@wombat01
Copy link

wombat01 commented Dec 2, 2019

Forky na novém repu se udělaly právě z důvodu těchto rychlých aktualizací, kdy může příspívat kdokoliv nemusí se čekat až to někdo(a jestli vůbec) mergne. Na novém repu u živých doplňků se všechno děje automaticky přes pull bota, takže je fork identický (a toto je v ideálním případě i cíl), ale nejde to tak i vždy. Proto pokud to jakkoliv jde, je lepší dělat změny v hlavním repozitáři.

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

No branches or pull requests

3 participants