No notification pop-up appears with Venom

ShaVerZ commented 4 years ago

FireTV Box (gen.2) 4k Kodi 18.5 Up Next 1.0.7 (Latest)

No notification popup and it seems to have stopped going on to the next episode. The playlist seems to be created within the addon, when viewing the playlist manager.

I couldnt find the exact information you required but the .log link is;

https://pastebin.com/GPvBYptu

I hope this helps and that I've provided what you need.

dagwieers commented 4 years ago

Ok, so it appears we did not get any information from the add-on. I would have expected something like this in the logs:

2019-12-10 13:42:44.090 T:1655604096  NOTICE: [service.upnext] Api -> addon_data_received called with data {u'play_info': {u'video_id': u'vid-e16b0168-5ab7-455d-a00f-18985bb3d4a7'}, u'next_episode': {u'rating': None, u'tvshowid': u'622902452527', u'episode': 2, u'title': u'Haar klasgenoten nemen Caro niet serieus', u'season': 1, u'plot': u'[COLOR blue][B]8 more months[/B] available[/COLOR]\n\nEen aantal weken zijn verstreken en Caro heeft langzaam maar zeker de weg op haar nieuwe school gevonden. Yemi en Kyona zijn twee goeie vrienden geworden maar er is iets dat ze desondanks nog steeds voor iedereen achterhoudt.\n\xa0\nAan de hand van haar po\xebzieopdracht voor de les Nederlands probeert ze haar klas op de hoogte te brengen van de situatie thuis. Helaas wordt haar gedicht niet door iedereen serieus genomen en zet Caro het op een lopen.\n\n[COLOR yellow]vrtnu.be/p.6wAOQBler[/COLOR]', u'episodeid': u'622899164527', u'firstaired': u'2019-08-01', u'art': {u'tvshow.fanart': u'https://images.vrt.be/orig/2019/01/07/cd9c3144-12a5-11e9-abcc-02b7b76bf47f.jpg', u'thumb': u'https://images.vrt.be/orig/2019/01/15/2ea66322-18a4-11e9-abcc-02b7b76bf47f.jpg', u'tvshow.clearart': None, u'tvshow.landscape': u'https://images.vrt.be/orig/2019/01/15/2ea66322-18a4-11e9-abcc-02b7b76bf47f.jpg', u'tvshow.poster': u'https://images.vrt.be/orig/2019/01/15/2ea66322-18a4-11e9-abcc-02b7b76bf47f.jpg', u'tvshow.clearlogo': None}, u'playcount': None, u'runtime': 1560, u'showtitle': u'#LikeMe'}, 'id': u'plugin.video.vrt.nu_play_action', u'current_episode': {u'rating': 0.0, u'tvshowid': u'622902452527', u'episode': 1, u'title': u'Caroline moet naar een nieuwe school', u'season': 1, u'plot': u"[COLOR blue][B]8 more months[/B] available[/COLOR]\n\n1 september, de aanvang van een nieuw schooljaar. En dat is het zeker voor nieuweling Caroline Timmers die haar eerste dag start op een, voor haar, compleet onbekende school. \n\xa0\nDoor de ziekte van Caro's moeder Kristel moest haar gezin noodgedwongen verhuizen van hun woonplaats in de Ardennen naar Antwerpen. Een serieuze verandering voor de jonge meid die letterlijk alles heeft moeten achterlaten. \n\xa0\nDat het de overgang niet van een leien dakje zal lopen, wordt al gauw duidelijk.\n\n[COLOR yellow]vrtnu.be/p.9q1okYWGJ[/COLOR]", u'episodeid': u'622899157527', u'firstaired': u'2019-08-01', u'art': {u'tvshow.fanart': u'https://images.vrt.be/orig/2019/01/07/cd9c3144-12a5-11e9-abcc-02b7b76bf47f.jpg', u'thumb': u'https://images.vrt.be/orig/2019/01/07/fbc7b11b-12a6-11e9-abcc-02b7b76bf47f.jpg', u'tvshow.clearart': None, u'tvshow.landscape': u'https://images.vrt.be/orig/2019/01/07/fbc7b11b-12a6-11e9-abcc-02b7b76bf47f.jpg', u'tvshow.poster': u'https://images.vrt.be/orig/2019/01/07/fbc7b11b-12a6-11e9-abcc-02b7b76bf47f.jpg', u'tvshow.clearlogo': None}, u'playcount': 9, u'runtime': 1560.0001220703125, u'showtitle': u'#LikeMe'}}

If UpNext doesn't get any information from the add-on, it will not show a pop-up.

This issue has been reported now a few times, and it was reported it worked with v1.0.1, but not with v1.0.2 or newer. Given that the only changes to v1.0.2 are related to:

JSONRPC abstraction (low impact)
Playlist handling (high impact)
Add endtime property (low impact)

So I would need to know what the add-on is sending, and why we aren't receiving it. cc @Venom123

PS Mind you that a few times now it was suggested this was related to UpNext no longer using Addon Signals, but that is incorrect. Even v1.0.1 is not using Addon Signals, it was removed earlier (39d3661af00e28bfc7613b6d62264ae0caa149cf) in v.0.0.9. The issue related to Addon Signals switching from hex to base64 encoding has been dealt with in the v1.0.7 release. (older versions of UpNext should work fine with older versions of Addon Signals though)

ShaVerZ commented 4 years ago

I've tried other versions and just gone back to 1.0.1 and still no popup and not moving to next episode.

v1.0.1 log;

https://pastebin.com/AZ1uxBm7

dagwieers commented 4 years ago

So what ought to work

Up Next v1.0.6 and older with Addon Signals v0.0.3 and older
Up Next v1.0.7 (any Addon Signals version)

But what will definitely not work is Up Next v1.0.6 or older together with Addon Signals v0.0.4 (latest).

ShaVerZ commented 4 years ago

Not quite sure what you mean but 1.0.7 doesn't work with any version of Addon Signals (latest).

I'm probably not understanding what you mean.

dagwieers commented 4 years ago

Addon Signals v0.0.3 uses hex encoding. Addon Signals v0.0.4 uses base64 encoding.

Up Next v1.06 and older uses hex encoding. Up Next v1.0.7 and newer supports both base64 and hex encoding.

So if you test anything but v1.0.7, you have to use Addon Signals v.0.0.3 otherwise it is guaranteed to not work and we haven't established anything but that.

In other words, the tests you did before using older versions of Up Next don't prove anything unless you also used an older version of Addon Signals.

ShaVerZ commented 4 years ago

I was using 1.0.7 when I reported this.....

Thanks for your help though, much appreciated. Probably best I leave it.....

dagwieers commented 4 years ago

Not quite sure what you mean but 1.0.7 doesn't work with any version of Addon Signals (latest).

Up Next v1.0.7 works with e.g. the Netflix add-on which is using Addon Signals v0.0.4, so it does work fine (and I am using it daily). We also have integration tests testing Addon Signals v0.0.4 with Up Next to ensure they are compatible. So no issues there. The reported problem here seems to be specific to the implementation of Venom and Seren.

ShaVerZ commented 4 years ago

Ah right, I understand. I thought you meant I should try 1.0.7 ensuring Addon Signals 0.0.4, that's where the confusion lay. I thought you meant it should work with that combination whilst using Seren/Venom.

It does seem to work with old versions of Kodi though.....

Thanks again.

dagwieers commented 4 years ago

@ShaVerZ Well, it should work for Venom and Seren, but it clearly doesn't. We haven't established the reason for this. I was told it stopped working even with v1.0.2 up to v1.0.6, which would indicate an issue with any of these changes: https://github.com/im85288/service.upnext/compare/1.0.1..1.0.2

But I am not sure if the tests were executed with the correct Addon Signals v0.0.3, so not clear if we can make that conclusion.

ShaVerZ commented 4 years ago

If theres anything you want me to try, let me know, I have a few spare FireTV boxes which are not in use. I'm happy to test certain setups if you wish.....

ShaVerZ commented 4 years ago

Well, it's working now. I had to drop back to Kodi 18.2 though, no idea why it seems fine with 18.2 but nothing higher..... thought it was worth mentioning.

dagwieers commented 4 years ago

So using Kodi 18.2 and what version of UpNext and AddonSignals?

ShaVerZ commented 4 years ago

I'm not currently home but I could only get 0.1.8 working. v1.0.1 and up didnt. Im assuming it is addon Signals v0.0.3 but I'll need to check when I'm home.

dagwieers commented 4 years ago

Looking at the Venom code I see various reasons why it would fail: https://github.com/123Venom/plugin.video.venom/blob/84edfdb37ecd785b5f0dac472f859c45d0c94fa2/resources/lib/modules/player.py#L347-L370

Apart from the fact that control.playlist.getposition() and self.mediatype may interfere with normal operation, the broad try-blocks may cause things not completing as expected.

Proper python code should not be using broad excepts, and I would suggest that Venom uses tox and pylint for testing for common python issues. Something that worked before could suddenly fail because of changes in behaviour that was not accounted for. Especially Python is susceptible for this. cc @123Venom

That said, I don't know if this is the reason for Up Next not working. Adding more log-statements that would give information on what is going on would be helpful as well. Now I don't see any logging from Venom in the logs. Did it ever send out the notification? I wouldn't know.

ShaVerZ commented 4 years ago

I got back earlier and noticed it wasnt working, I've tried a combination of installs and none seem to work, I can only assume that Venom has received an update at some point and it has stopped up next working. It doesnt matter which combination you use re: Add on Signals.

I could go back and try older versions of Venom but tbh, I cant be assed today ;)

dagwieers commented 4 years ago

IMO it is better to wait for the developer @123Venom to troubleshoot Venom.

123Venom commented 4 years ago

ok I've been looking at this and I'm stumped how I can run v1.0.1 of UpNext without a single issue but both Seren and Venom suffer the same exact problem beyond v1.0.1 with no UpNext notification showing up. I just tried v1.0.7 and logged this 2019-12-12 03:22:30.045 T:7332 NOTICE: [service.upnext] Api -> addon_data_received called with data {u'play_info': {u'item_id': 2244839}, u'next_episode': {u'rating': u'7.5', u'tvshowid': u'80379', u'art': {u'tvshow.fanart': u'https://thetvdb.com/banners/fanart/original/80379-34.jpg', u'thumb': u'https://thetvdb.com/banners/episodes/80379/5714255.jpg', u'tvshow.clearart': u'https://assets.fanart.tv/fanart/tv/80379/hdclearart/the-big-bang-theory-519818ab2d8a9.png', u'tvshow.landscape': u'https://thetvdb.com/banners/fanart/original/80379-34.jpg', u'tvshow.poster': u'https://thetvdb.com/banners/posters/80379-18.jpg', u'tvshow.clearlogo': u'https://assets.fanart.tv/fanart/tv/80379/hdtvlogo/the-big-bang-theory-50477a642f672.png'}, u'plot': u'The Air Force shows interest in the guidance system and Penny lets it slip at work that Bernadette is expecting.', u'season': u'10', u'title': u'The Military Miniaturization', u'episodeid': 2267810, u'tvshowimdb': u'tt0898266', u'firstaired': u'2007', u'year': u'2016', u'episode': u'2', u'playcount': 0, u'showtitle': u'The Big Bang Theory'}, u'notification_time': 60, 'id': u'plugin.video.venom_play_action', u'current_episode': {u'rating': u'7.6', u'tvshowid': u'80379', u'art': {u'tvshow.fanart': u'https://thetvdb.com/banners/fanart/original/80379-34.jpg', u'thumb': u'https://thetvdb.com/banners/episodes/80379/5654968.jpg', u'tvshow.clearart': u'https://assets.fanart.tv/fanart/tv/80379/hdclearart/the-big-bang-theory-519818ab2d8a9.png', u'tvshow.landscape': u'https://thetvdb.com/banners/fanart/original/80379-34.jpg', u'tvshow.poster': u'https://thetvdb.com/banners/posters/80379-18.jpg', u'tvshow.clearlogo': u'https://assets.fanart.tv/fanart/tv/80379/hdtvlogo/the-big-bang-theory-50477a642f672.png'}, u'plot': u"After Leonard's father, Alfred and Sheldon's mother, Mary share an evening together, everyone deals with an awkward morning the following day. Also, Penny's family arrives for the ceremony, including her anxiety-ridden mother, Susan, and her drug dealing brother, Randall.", u'season': u'10', u'title': u'The Conjugal Conjecture', u'episodeid': 2244839, u'firstaired': u'2007', u'episode': u'1', u'playcount': 0, u'showtitle': u'The Big Bang Theory'}} 2019-12-12 03:22:35.055 T:10360 WARNING: CRenderManager::WaitForBuffer - timeout waiting for buffer 2019-12-12 03:23:35.179 T:7332 WARNING: Previous line repeats 1 times. 2019-12-12 03:23:35.179 T:7332 NOTICE: [service.upnext] Monitor -> Calling autoplayback totaltime - playtime is 48.3489990234 2019-12-12 03:23:35.195 T:7332 NOTICE: [service.upnext] Api -> Got details of next playlist item {u'jsonrpc': u'2.0', u'id': 1, u'result': {u'items': [{u'plot': u'The Air Force shows interest in the guidance system and Penny lets it slip at work that Bernadette is expecting.', u'tvshowid': -1, u'dateadded': u'', u'episode': 2, u'rating': 7.5, u'season': 10, u'lastplayed': u'', u'resume': {u'position': 0.0, u'total': 0.0}, u'label': u'10x02 . The Military Miniaturization', u'writer': [u'Chuck Lorre', u'Eric Kaplan', u'Jim Reynolds', u'Maria Ferrari', u'Steve Holland', u'Steven Molaro'], u'firstaired': u'', u'title': u'The Military Miniaturization', u'file': u'plugin://plugin.video.venom/?action=play&title=The+Military+Miniaturization&year=2007&imdb=tt0898266&tvdb=80379&season=10&episode=2&tvshowtitle=The+Big+Bang+Theory&premiered=2016-09-26&meta=%7B%22rating%22%3A+%227.5%22%2C+%22poster2%22%3A+%22https%3A%2F%2Fassets.fanart.tv%2Ffanart%2Ftv%2F80379%2Ftvposter%2Fthe-big-bang-theory-5214251a5b671.jpg%22%2C+%22imdb%22%3A+%22tt0898266%22%2C+%22year%22%3A+%222016%22%2C+%22duration%22%3A+%221500%22%2C+%22landscape%22%3A+%22https%3A%2F%2Fassets.fanart.tv%2Ffanart%2Ftv%2F80379%2Ftvthumb%2FT_80379+%280%29.jpg%22%2C+%22plot%22%3A+%22The+Air+Force+shows+interest+in+the+guidance+system+and+Penny+lets+it+slip+at+work+that+Bernadette+is+expecting.%22%2C+%22votes%22%3A+%2241886%22%2C+%22castandart%22%3A+%5B%7B%22role%22%3A+%22Sheldon+Cooper%22%2C+%22name%22%3A+%22Jim+Parsons%22%2C+%22thumbnail%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Factors%2F325467.jpg%22%7D%2C+%7B%22role%22%3A+%22Leonard+Hofstadter%22%2C+%22name%22%3A+%22Johnny+Galecki%22%2C+%22thumbnail%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Factors%2F325466.jpg%22%7D%2C+%7B%22role%22%3A+%22Penny%22%2C+%22name%22%3A+%22Kaley+Cuoco%22%2C+%22thumbnail%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Factors%2F325465.jpg%22%7D%2C+%7B%22role%22%3A+%22Rajesh+Koothrappali%22%2C+%22name%22%3A+%22Kunal+Nayyar%22%2C+%22thumbnail%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Factors%2F325463.jpg%22%7D%2C+%7B%22role%22%3A+%22Howard+Wolowitz%22%2C+%22name%22%3A+%22Simon+Helberg%22%2C+%22thumbnail%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Factors%2F325464.jpg%22%7D%2C+%7B%22role%22%3A+%22Bernadette+Rostenkowski%22%2C+%22name%22%3A+%22Melissa+Rauch%22%2C+%22thumbnail%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Factors%2F325462.jpg%22%7D%2C+%7B%22role%22%3A+%22Amy+Farrah+Fowler%22%2C+%22name%22%3A+%22Mayim+Bialik%22%2C+%22thumbnail%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Factors%2F325461.jpg%22%7D%2C+%7B%22role%22%3A+%22Emily+Sweeney%22%2C+%22name%22%3A+%22Laura+Spencer%22%2C+%22thumbnail%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Factors%2F358188.jpg%22%7D%2C+%7B%22role%22%3A+%22Leslie+Winkle%22%2C+%22name%22%3A+%22Sara+Gilbert%22%2C+%22thumbnail%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Factors%2F378343.jpg%22%7D%2C+%7B%22role%22%3A+%22Stuart+Bloom%22%2C+%22name%22%3A+%22Kevin+Sussman%22%2C+%22thumbnail%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Factors%2F378344.jpg%22%7D%2C+%7B%22role%22%3A+%22Barry+Kripke%22%2C+%22name%22%3A+%22John+Ross+Bowie%22%2C+%22thumbnail%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Factors%2F382566.jpg%22%7D%2C+%7B%22role%22%3A+%22Debbie+Wolowitz%22%2C+%22name%22%3A+%22Carol+Ann+Susi%22%2C+%22thumbnail%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Factors%2F403186.jpg%22%7D%5D%2C+%22title%22%3A+%22The+Military+Miniaturization%22%2C+%22tvdb%22%3A+%2280379%22%2C+%22mpaa%22%3A+%22TV-14%22%2C+%22writer%22%3A+%22Chuck+Lorre+%2F+Eric+Kaplan+%2F+Jim+Reynolds+%2F+Maria+Ferrari+%2F+Steve+Holland+%2F+Steven+Molaro%22%2C+%22label%22%3A+%22The+Military+Miniaturization%22%2C+%22episodeIDS%22%3A+%7B%22tvdb%22%3A+5714255%2C+%22tmdb%22%3A+1221932%2C+%22imdb%22%3A+%22tt5973910%22%2C+%22tvrage%22%3A+0%2C+%22trakt%22%3A+2267810%7D%2C+%22season%22%3A+%2210%22%2C+%22banner2%22%3A+%22https%3A%2F%2Fassets.fanart.tv%2Ffanart%2Ftv%2F80379%2Ftvbanner%2Fthe-big-bang-theory-52db1f74e2bee.jpg%22%2C+%22tvshowyear%22%3A+%222007%22%2C+%22fanart2%22%3A+%22https%3A%2F%2Fassets.fanart.tv%2Ffanart%2Ftv%2F80379%2Fshowbackground%2Fthe-big-bang-theory-561e3c3dbc1ea.jpg%22%2C+%22status%22%3A+%22Ended%22%2C+%22extended%22%3A+true%2C+%22poster%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Fposters%2F80379-18.jpg%22%2C+%22tvshowtitle%22%3A+%22The+Big+Bang+Theory%22%2C+%22mediatype%22%3A+%22episode%22%2C+%22studio%22%3A+%22CBS%22%2C+%22genre%22%3A+%22Comedy%22%2C+%22banner%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Fgraphical%2F80379-g13.jpg%22%2C+%22clearart%22%3A+%22https%3A%2F%2Fassets.fanart.tv%2Ffanart%2Ftv%2F80379%2Fhdclearart%2Fthe-big-bang-theory-519818ab2d8a9.png%22%2C+%22thumb%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Fepisodes%2F80379%2F5714255.jpg%22%2C+%22episode%22%3A+%222%22%2C+%22metacache%22%3A+true%2C+%22overlay%22%3A+6%2C+%22premiered%22%3A+%222016-09-26%22%2C+%22clearlogo%22%3A+%22https%3A%2F%2Fassets.fanart.tv%2Ffanart%2Ftv%2F80379%2Fhdtvlogo%2Fthe-big-bang-theory-50477a642f672.png%22%2C+%22fanart%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Ffanart%2Foriginal%2F80379-34.jpg%22%2C+%22season_poster%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Fposters%2F80379-18.jpg%22%2C+%22playcount%22%3A+0%2C+%22seasoncount%22%3A+24%2C+%22trailer%22%3A+%22plugin%3A%2F%2Fplugin.video.venom%2F%3Faction%3Dtrailer%26name%3DThe%2BBig%2BBang%2BTheory%26imdb%3Dtt0898266%22%7D&t=20191212032118292000', u'art': {u'season.poster': u'image://https%3a%2f%2fthetvdb.com%2fbanners%2fposters%2f80379-18.jpg/', u'fanart': u'image://https%3a%2f%2fassets.fanart.tv%2ffanart%2ftv%2f80379%2fshowbackground%2fthe-big-bang-theory-561e3c3dbc1ea.jpg/', u'poster': u'image://https%3a%2f%2fthetvdb.com%2fbanners%2fposters%2f80379-18.jpg/', u'clearlogo': u'image://https%3a%2f%2fassets.fanart.tv%2ffanart%2ftv%2f80379%2fhdtvlogo%2fthe-big-bang-theory-50477a642f672.png/', u'clearart': u'image://https%3a%2f%2fassets.fanart.tv%2ffanart%2ftv%2f80379%2fhdclearart%2fthe-big-bang-theory-519818ab2d8a9.png/', u'tvshow.poster': u'image://https%3a%2f%2fassets.fanart.tv%2ffanart%2ftv%2f80379%2ftvposter%2fthe-big-bang-theory-5214251a5b671.jpg/', u'banner': u'image://https%3a%2f%2fassets.fanart.tv%2ffanart%2ftv%2f80379%2ftvbanner%2fthe-big-bang-theory-52db1f74e2bee.jpg/', u'landscape': u'image://https%3a%2f%2fassets.fanart.tv%2ffanart%2ftv%2f80379%2ftvthumb%2fT_80379%20(0).jpg/', u'thumb': u'image://https%3a%2f%2fthetvdb.com%2fbanners%2fepisodes%2f80379%2f5714255.jpg/'}, u'playcount': 0, u'runtime': 1500, u'type': u'episode', u'showtitle': u'The Big Bang Theory', u'streamdetails': {u'video': [{u'language': u'', u'stereomode': u'', u'width': 0, u'codec': u'h264', u'aspect': 0.0, u'duration': 0, u'height': 0}], u'audio': [], u'subtitle': []}}], u'limits': {u'start': 1, u'total': 24, u'end': 2}}} 2019-12-12 03:23:35.195 T:7332 NOTICE: [service.upnext] PlaybackManager -> episode details {u'plot': u'The Air Force shows interest in the guidance system and Penny lets it slip at work that Bernadette is expecting.', u'tvshowid': -1, u'dateadded': u'', u'episode': 2, u'rating': 7.5, u'season': 10, u'lastplayed': u'', u'episodeid': None, u'resume': {u'position': 0.0, u'total': 0.0}, u'label': u'10x02 . The Military Miniaturization', u'writer': [u'Chuck Lorre', u'Eric Kaplan', u'Jim Reynolds', u'Maria Ferrari', u'Steve Holland', u'Steven Molaro'], u'firstaired': u'', u'title': u'The Military Miniaturization', u'file': u'plugin://plugin.video.venom/?action=play&title=The+Military+Miniaturization&year=2007&imdb=tt0898266&tvdb=80379&season=10&episode=2&tvshowtitle=The+Big+Bang+Theory&premiered=2016-09-26&meta=%7B%22rating%22%3A+%227.5%22%2C+%22poster2%22%3A+%22https%3A%2F%2Fassets.fanart.tv%2Ffanart%2Ftv%2F80379%2Ftvposter%2Fthe-big-bang-theory-5214251a5b671.jpg%22%2C+%22imdb%22%3A+%22tt0898266%22%2C+%22year%22%3A+%222016%22%2C+%22duration%22%3A+%221500%22%2C+%22landscape%22%3A+%22https%3A%2F%2Fassets.fanart.tv%2Ffanart%2Ftv%2F80379%2Ftvthumb%2FT_80379+%280%29.jpg%22%2C+%22plot%22%3A+%22The+Air+Force+shows+interest+in+the+guidance+system+and+Penny+lets+it+slip+at+work+that+Bernadette+is+expecting.%22%2C+%22votes%22%3A+%2241886%22%2C+%22castandart%22%3A+%5B%7B%22role%22%3A+%22Sheldon+Cooper%22%2C+%22name%22%3A+%22Jim+Parsons%22%2C+%22thumbnail%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Factors%2F325467.jpg%22%7D%2C+%7B%22role%22%3A+%22Leonard+Hofstadter%22%2C+%22name%22%3A+%22Johnny+Galecki%22%2C+%22thumbnail%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Factors%2F325466.jpg%22%7D%2C+%7B%22role%22%3A+%22Penny%22%2C+%22name%22%3A+%22Kaley+Cuoco%22%2C+%22thumbnail%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Factors%2F325465.jpg%22%7D%2C+%7B%22role%22%3A+%22Rajesh+Koothrappali%22%2C+%22name%22%3A+%22Kunal+Nayyar%22%2C+%22thumbnail%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Factors%2F325463.jpg%22%7D%2C+%7B%22role%22%3A+%22Howard+Wolowitz%22%2C+%22name%22%3A+%22Simon+Helberg%22%2C+%22thumbnail%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Factors%2F325464.jpg%22%7D%2C+%7B%22role%22%3A+%22Bernadette+Rostenkowski%22%2C+%22name%22%3A+%22Melissa+Rauch%22%2C+%22thumbnail%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Factors%2F325462.jpg%22%7D%2C+%7B%22role%22%3A+%22Amy+Farrah+Fowler%22%2C+%22name%22%3A+%22Mayim+Bialik%22%2C+%22thumbnail%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Factors%2F325461.jpg%22%7D%2C+%7B%22role%22%3A+%22Emily+Sweeney%22%2C+%22name%22%3A+%22Laura+Spencer%22%2C+%22thumbnail%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Factors%2F358188.jpg%22%7D%2C+%7B%22role%22%3A+%22Leslie+Winkle%22%2C+%22name%22%3A+%22Sara+Gilbert%22%2C+%22thumbnail%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Factors%2F378343.jpg%22%7D%2C+%7B%22role%22%3A+%22Stuart+Bloom%22%2C+%22name%22%3A+%22Kevin+Sussman%22%2C+%22thumbnail%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Factors%2F378344.jpg%22%7D%2C+%7B%22role%22%3A+%22Barry+Kripke%22%2C+%22name%22%3A+%22John+Ross+Bowie%22%2C+%22thumbnail%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Factors%2F382566.jpg%22%7D%2C+%7B%22role%22%3A+%22Debbie+Wolowitz%22%2C+%22name%22%3A+%22Carol+Ann+Susi%22%2C+%22thumbnail%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Factors%2F403186.jpg%22%7D%5D%2C+%22title%22%3A+%22The+Military+Miniaturization%22%2C+%22tvdb%22%3A+%2280379%22%2C+%22mpaa%22%3A+%22TV-14%22%2C+%22writer%22%3A+%22Chuck+Lorre+%2F+Eric+Kaplan+%2F+Jim+Reynolds+%2F+Maria+Ferrari+%2F+Steve+Holland+%2F+Steven+Molaro%22%2C+%22label%22%3A+%22The+Military+Miniaturization%22%2C+%22episodeIDS%22%3A+%7B%22tvdb%22%3A+5714255%2C+%22tmdb%22%3A+1221932%2C+%22imdb%22%3A+%22tt5973910%22%2C+%22tvrage%22%3A+0%2C+%22trakt%22%3A+2267810%7D%2C+%22season%22%3A+%2210%22%2C+%22banner2%22%3A+%22https%3A%2F%2Fassets.fanart.tv%2Ffanart%2Ftv%2F80379%2Ftvbanner%2Fthe-big-bang-theory-52db1f74e2bee.jpg%22%2C+%22tvshowyear%22%3A+%222007%22%2C+%22fanart2%22%3A+%22https%3A%2F%2Fassets.fanart.tv%2Ffanart%2Ftv%2F80379%2Fshowbackground%2Fthe-big-bang-theory-561e3c3dbc1ea.jpg%22%2C+%22status%22%3A+%22Ended%22%2C+%22extended%22%3A+true%2C+%22poster%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Fposters%2F80379-18.jpg%22%2C+%22tvshowtitle%22%3A+%22The+Big+Bang+Theory%22%2C+%22mediatype%22%3A+%22episode%22%2C+%22studio%22%3A+%22CBS%22%2C+%22genre%22%3A+%22Comedy%22%2C+%22banner%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Fgraphical%2F80379-g13.jpg%22%2C+%22clearart%22%3A+%22https%3A%2F%2Fassets.fanart.tv%2Ffanart%2Ftv%2F80379%2Fhdclearart%2Fthe-big-bang-theory-519818ab2d8a9.png%22%2C+%22thumb%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Fepisodes%2F80379%2F5714255.jpg%22%2C+%22episode%22%3A+%222%22%2C+%22metacache%22%3A+true%2C+%22overlay%22%3A+6%2C+%22premiered%22%3A+%222016-09-26%22%2C+%22clearlogo%22%3A+%22https%3A%2F%2Fassets.fanart.tv%2Ffanart%2Ftv%2F80379%2Fhdtvlogo%2Fthe-big-bang-theory-50477a642f672.png%22%2C+%22fanart%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Ffanart%2Foriginal%2F80379-34.jpg%22%2C+%22season_poster%22%3A+%22https%3A%2F%2Fthetvdb.com%2Fbanners%2Fposters%2F80379-18.jpg%22%2C+%22playcount%22%3A+0%2C+%22seasoncount%22%3A+24%2C+%22trailer%22%3A+%22plugin%3A%2F%2Fplugin.video.venom%2F%3Faction%3Dtrailer%26name%3DThe%2BBig%2BBang%2BTheory%26imdb%3Dtt0898266%22%7D&t=20191212032118292000', u'art': {u'season.poster': u'image://https%3a%2f%2fthetvdb.com%2fbanners%2fposters%2f80379-18.jpg/', u'fanart': u'image://https%3a%2f%2fassets.fanart.tv%2ffanart%2ftv%2f80379%2fshowbackground%2fthe-big-bang-theory-561e3c3dbc1ea.jpg/', u'poster': u'image://https%3a%2f%2fthetvdb.com%2fbanners%2fposters%2f80379-18.jpg/', u'clearlogo': u'image://https%3a%2f%2fassets.fanart.tv%2ffanart%2ftv%2f80379%2fhdtvlogo%2fthe-big-bang-theory-50477a642f672.png/', u'clearart': u'image://https%3a%2f%2fassets.fanart.tv%2ffanart%2ftv%2f80379%2fhdclearart%2fthe-big-bang-theory-519818ab2d8a9.png/', u'tvshow.poster': u'image://https%3a%2f%2fassets.fanart.tv%2ffanart%2ftv%2f80379%2ftvposter%2fthe-big-bang-theory-5214251a5b671.jpg/', u'banner': u'image://https%3a%2f%2fassets.fanart.tv%2ffanart%2ftv%2f80379%2ftvbanner%2fthe-big-bang-theory-52db1f74e2bee.jpg/', u'landscape': u'image://https%3a%2f%2fassets.fanart.tv%2ffanart%2ftv%2f80379%2ftvthumb%2fT_80379%20(0).jpg/', u'thumb': u'image://https%3a%2f%2fthetvdb.com%2fbanners%2fepisodes%2f80379%2f5714255.jpg/'}, u'playcount': 0, u'runtime': 1500, u'type': u'episode', u'showtitle': u'The Big Bang Theory', u'streamdetails': {u'video': [{u'language': u'', u'stereomode': u'', u'width': 0, u'codec': u'h264', u'aspect': 0.0, u'duration': 0, u'height': 0}], u'audio': [], u'subtitle': []}} 2019-12-12 03:23:35.195 T:7332 NOTICE: [service.upnext] Monitor -> Up Next style autoplay succeeded. but not notification. I'm seeing jasonrpc so I have to wonder if UpNext receives the proper data why it's looking in the local library? It never did that before unless tvshowid was not received but it is. I've made zero changes to the data sent and received to UpNext since implementing support with v1.0.1 yet anything beyond 1.0.1 and it no longer works in Seren or Venom. I'm going to revert back to working v1.0.1 and compare logged data

ShaVerZ commented 4 years ago

Which version of Kodi have you tested @123Venom and which version of Addon Signals?

I cant get any version to work on FireTV Box (gen2).....

123Venom commented 4 years ago

v1.0.1 log...works perfectly! I see what's not transmitted but why idk 2019-12-12 04:09:35.438 T:9520 NOTICE: UP NEXT Monitor -> log : Calling autoplayback totaltime - playtime is 59.3669433594 2019-12-12 04:09:35.441 T:9520 NOTICE: UP NEXT Api -> log : handle_addon_lookup_of_next_episode episode returning data {"rating": "7.7", "tvshowid": "80379", "art": {"tvshow.fanart": "https://thetvdb.com/banners/fanart/original/80379-34.jpg", "thumb": "https://thetvdb.com/banners/episodes/80379/4413988.jpg", "tvshow.clearart": "https://assets.fanart.tv/fanart/tv/80379/hdclearart/the-big-bang-theory-519818ab2d8a9.png", "tvshow.landscape": "https://thetvdb.com/banners/fanart/original/80379-34.jpg", "tvshow.poster": "https://thetvdb.com/banners/posters/80379-18.jpg", "tvshow.clearlogo": "https://assets.fanart.tv/fanart/tv/80379/hdtvlogo/the-big-bang-theory-50477a642f672.png"}, "plot": "A game threatens Sheldon's relationship with Stephen Hawking; Penny secretly signs up for a college class.", "season": "6", "title": "The Extract Obliteration", "episodeid": 75485, "tvshowimdb": "tt0898266", "firstaired": "2007", "year": "2012", "episode": "6", "playcount": 0, "showtitle": "The Big Bang Theory"} 2019-12-12 04:09:35.443 T:9520 NOTICE: UP NEXT Api -> log : handle_addon_lookup_of_current episode returning data {"rating": "7.7", "tvshowid": "80379", "art": {"tvshow.fanart": "https://thetvdb.com/banners/fanart/original/80379-34.jpg", "thumb": "https://thetvdb.com/banners/episodes/80379/4408722.jpg", "tvshow.clearart": "https://assets.fanart.tv/fanart/tv/80379/hdclearart/the-big-bang-theory-519818ab2d8a9.png", "tvshow.landscape": "https://thetvdb.com/banners/fanart/original/80379-34.jpg", "tvshow.poster": "https://thetvdb.com/banners/posters/80379-18.jpg", "tvshow.clearlogo": "https://assets.fanart.tv/fanart/tv/80379/hdtvlogo/the-big-bang-theory-50477a642f672.png"}, "plot": "The gang heads to Stuart's comic book store to celebrate Halloween; Leonard uses science to seduce Penny.", "season": "6", "title": "The Holographic Excitation", "episodeid": 75484, "firstaired": "2007", "episode": "5", "playcount": 0, "showtitle": "The Big Bang Theory"} 2019-12-12 04:09:35.446 T:9520 NOTICE: UP NEXT PlaybackManager -> log : episode details {"rating": "7.7", "tvshowid": "80379", "art": {"tvshow.fanart": "https://thetvdb.com/banners/fanart/original/80379-34.jpg", "thumb": "https://thetvdb.com/banners/episodes/80379/4413988.jpg", "tvshow.clearart": "https://assets.fanart.tv/fanart/tv/80379/hdclearart/the-big-bang-theory-519818ab2d8a9.png", "tvshow.landscape": "https://thetvdb.com/banners/fanart/original/80379-34.jpg", "tvshow.poster": "https://thetvdb.com/banners/posters/80379-18.jpg", "tvshow.clearlogo": "https://assets.fanart.tv/fanart/tv/80379/hdtvlogo/the-big-bang-theory-50477a642f672.png"}, "plot": "A game threatens Sheldon's relationship with Stephen Hawking; Penny secretly signs up for a college class.", "season": "6", "title": "The Extract Obliteration", "episodeid": 75485, "tvshowimdb": "tt0898266", "firstaired": "2007", "year": "2012", "episode": "6", "playcount": 0, "showtitle": "The Big Bang Theory"} 2019-12-12 04:09:35.454 T:9520 NOTICE: UP NEXT PlaybackManager -> log : played in a row settings "3" 2019-12-12 04:09:35.457 T:9520 NOTICE: UP NEXT PlaybackManager -> log : played in a row 1 2019-12-12 04:09:35.459 T:9520 NOTICE: UP NEXT PlaybackManager -> log : showing next up page as played in a row is 1 2019-12-12 04:10:26.147 T:14156 WARNING: CGUIWindowManager - CGUIWindowManager::HandleAction - ignoring action 107, because topmost modal dialog closing animation is running 2019-12-12 04:10:26.166 T:9520 WARNING: Previous line repeats 1 times. 2019-12-12 04:10:26.166 T:9520 NOTICE: UP NEXT PlaybackManager -> log : playing media episode 2019-12-12 04:10:26.168 T:9520 NOTICE: UP NEXT Api -> log : sending data to addon to play: {"item_id": 75484} 2019-12-12 04:10:26.181 T:14156 WARNING: CGUIWindowManager - CGUIWindowManager::HandleAction - ignoring action 107, because topmost modal dialog closing animation is running 2019-12-12 04:10:26.183 T:9520 NOTICE: UP NEXT Monitor -> log : Up Next style autoplay succeeded.

https://postimg.cc/rDnpsQ5W

123Venom commented 4 years ago

Which version of Kodi have you tested @123Venom and which version of Addon Signals?

I cant get any version to work on FireTV Box (gen2).....

When I implemented UpNext support I was on 18.1...currently on 18.5 and just tested on win pc and Shield and both behave the same way. Anything after v1.0.1 and no notification. 1.0.1 works flawlessly and I've changed no code since nor had reason to till perhaps now. I'd love to get to the bottom of this. And Addon Signals I'm still on 0.0.3 with auto updates off because when that updated to 0.0.4 it also broke but going back to 0.0.3 and all was fine.

dagwieers commented 4 years ago

@123Venom So you confirm with 100% certainty that v1.0.1 works fine, and v1.0.2 does not?

Can you please post full Kodi debug logs rather than fragments?

Also previous reports did not include the addon_data_received called with data section, which indicated there was no information send. So it is absolutely essential that we can interpret the information we get correctly, and if you only provide us with fragments, we may be looking at the wrong data and draw wrong conclusions.

ShaVerZ commented 4 years ago

SEREN has had an update and Up Next Works, I still can't get it to work with Venom.

However, I'm using v1.0.1 with 0.0.0.3 Addon Signals.

dagwieers commented 4 years ago

@ShaVerZ Sigh, so you are now confirming that Venom does not work with Up Next v1.0.1 using Venom with Addon Signals v0.0.3 ? This would go against all previous reports...

ShaVerZ commented 4 years ago

When forwarding to the end of a programme, to see if Up Next works, should you allow a minute or two to the end or longer.

I used Seren last night and watched a programme completely and it worked. Perhaps I'm not leaving enough time at the end.....

dagwieers commented 4 years ago

According to the Seren changelog, Seren v1.6.0 removed Up Next support

Removed support for UpNext addon

dagwieers commented 4 years ago

@ShaVerZ Up Next by default is configured to show the pop-up 30 seconds before the end. The master branch now has some heuristics that make this time depend on the length of the video. from 15 seconds before the end to 50 seconds (but configurable to anything you prefer).

Update: Some add-ons provide Up Next with their own notification time, I noticed Seren has its own configuration option which also defaults to 30 seconds (but is a slider from 10 to 60).

ShaVerZ commented 4 years ago

Well, its 100% working with Seren and I have some settings which resemble Up Next settings.

ShaVerZ commented 4 years ago

20191213_123159 20191213_123218

ShaVerZ commented 4 years ago

I just tested Venom and I can't get it working with v1.0.1 and 0.0.3.

Perhaps Seren is using it's own notifications instead of Up Next.

dagwieers commented 4 years ago

Well, the Seren code at https://github.com/nixgates/plugin.video.seren is incomplete. The code refers to files that do not exist in the tree. But it is clear they moved to their own implementation rather than use Up Next.

dagwieers commented 4 years ago

Perhaps Seren is using it's own notifications instead of Up Next.

That is what I have been saying.

ShaVerZ commented 4 years ago

Sorry @dagwieers , I didnt see your post.

I can't get it working with Venom, even with the setup users apparently say works.

123Venom commented 4 years ago

Sorry @dagwieers , I didnt see your post.

I can't get it working with Venom, even with the setup users apparently say works.

Yes Seren no longer uses UpNext as of v1.51 I believe is when he went in another direction.

ShaVerZ commented 4 years ago

I can't get it working with Venom, even with the setup users apparently say works.

I was talking about Venom.....

123Venom commented 4 years ago

@123Venom So you confirm with 100% certainty that v1.0.1 works fine, and v1.0.2 does not?

Can you please post full Kodi debug logs rather than fragments?

_Also previous reports did not include the addon_data_received called with data section, which indicated there was no information send. So it is absolutely essential that we can interpret the information we get correctly, and if you only provide us with fragments, we may be looking at the wrong data and draw wrong conclusions._

Sorry but I tried to post the relevant data communication between UpNext and Venom to limit all the other Kodi riff raff...perhaps that's just a preference of my own when getting log data and not wanting to read a whole pile of bogus data not related to issue I'm tracking down...lol.

Anyway here's the log data with v1.0.1 install and UpNext notification works fine. https://ufile.io/1zyq5ihg

Here's a log with v1.0.7 installed and UpNext notification stopped working. https://ufile.io/nlysazqs

123Venom commented 4 years ago

Sorry @dagwieers , I didnt see your post.

I can't get it working with Venom, even with the setup users apparently say works.

are you using the "Play from Here" kodi context menu option?

123Venom commented 4 years ago

@dagwieers here's what I was actually trying to show in those initial shortened posts of log data, sorry I didn't point it out directly. What I never see in v1.0.7 that I see in v1.0.1 is this.

NOTICE: UP NEXT Api -> log : sending data to addon to play: {"item_id": 3189430}

Seems I never get anything sent back that I only do in v1.0.1...less I missed it due to a terminology change in v1.0.7, but I did scan it pretty good for relevant info and found nothing transmitted back to addon in v1.0.7.

123Venom commented 4 years ago

@dagwieers Also I stated "terminology changes because I see this v1.0.1 NOTICE: UP NEXT Api -> log

v1.0.7 NOTICE: [service.upnext] Api ->

Petty I know but but when scanning a log for data transmissions to/off you can miss it because of those trivial changes...lol

123Venom commented 4 years ago

Looking at the Venom code I see various reasons why it would fail: https://github.com/123Venom/plugin.video.venom/blob/84edfdb37ecd785b5f0dac472f859c45d0c94fa2/resources/lib/modules/player.py#L347-L370

Apart from the fact that control.playlist.getposition() and self.mediatype may interfere with normal operation, the broad try-blocks may cause things not completing as expected.<

How would getting the kodi playlist position, to acquire what is the current and next item in the "kodi playlist", interfere with normal operations? current_position = control.playlist.getposition() next_url = control.playlist[current_position + 1].getPath()

Gets the next item url in kodi playlist and from my control module is this.... playlist = xbmc.PlayList(xbmc.PLAYLIST_VIDEO) ..wrong how?

Worked fine in v1.0.1 but not anymore?

and the fact of checking my "self.mediatype" is for inquiring on the metadata passed to the player . If the "mediatype" is a movie what's then the point of UpNext acting upon it? It avoids the function next_info() that is for episode playback ONLY and assignment of variables that are required to pass to UpNext, log says transmit is fine no?, but no data received after v1.0.1 per log? I pass this next_info() when self.mediatype="episode". Nothing seems to be received back after v1.0.1...not sure how that could even interfere as you state, that code is long before the function to get and send the next_info(). Sorry I just do not agree with those first two statements because to me if I'm wrong it would never have worked to begin with.

If I can log those kodi playlist positions, current and next, it's wrong how exactly? Or even "interfere with normal operations" for that matter.

Seren and Venom shared that same code for the last 6 months and I know this because I worked with the Seren dev to implement UpNext integration based on the posted addon integration guidelines...he's since chosen another route in his latest release. I also got a ton of help from @angelblue05 back then that helped get this all working and much respect for doing so and her time!!! Now it's broken after v1.0.1 and by the sounds of it fingers pointed at me when no code changes took place my end, or Seren for that matter, but did in UpNext so you tell me what that looks like?

ShaVerZ commented 4 years ago

Sorry @dagwieers , I didnt see your post. I can't get it working with Venom, even with the setup users apparently say works.

are you using the "Play from Here" kodi context menu option?

No, should I? If so, this is probably the issue 😮

im85288 commented 4 years ago

@123Venom Version 1.0.1 was my initial release and I was fully familiar with everything involved in the addon at that time. Since then it’s been contributions from others that have moved the addon on, the main change I can think off added after that was playlist support. If things have stopped working on the integration side since that version that’s most likely the cause. I guess we are going to have to try and pinpoint what went into subsequent versions that changed how it works..but my hunch is the playlist support.

123Venom commented 4 years ago

Sorry @dagwieers , I didnt see your post. I can't get it working with Venom, even with the setup users apparently say works.

are you using the "Play from Here" kodi context menu option?

No, should I? If so, this is probably the issue open_mouth

not trying to be rude but when the integration took place back around May/.June I had a very good explanation of it's workings written up in the changelog and even commented to see the "Newsinfo" for a more detail expanation. There's a wealth of info in there for you to read and is accessible from the main menu. I believe it's even still in there. Yes you need to use "Play from here". Now that's been settled test again with v1.0.1 and let me know if it works there(UpNext v1.01 and Addon Signals 0.0.3 combo). Does not seem to work with any version of UpNext after that. Looks to me like nothing transmitted back to addon.

ShaVerZ commented 4 years ago

Sorry @dagwieers , I didnt see your post. I can't get it working with Venom, even with the setup users apparently say works.

are you using the "Play from Here" kodi context menu option?

No, should I? If so, this is probably the issue open_mouth

not trying to be rude but when the integration took place back around May/.June I had a very good explanation of it's workings written up in the changelog and even commented to see the "Newsinfo" for a more detail expanation. There's a wealth of info in there for you to read and is accessible from the main menu. I believe it's even still in there. Yes you need to use "Play from here". Now that's been settled test again with v1.0.1 and let me know if it works there(UpNext v1.01 and Addon Signals 0.0.3 combo). Does not seem to work with any version of UpNext after that. Looks to me like nothing transmitted back to addon.

👍My bad, that's that solved.

123Venom commented 4 years ago

@123Venom Version 1.0.1 was my initial release and I was fully familiar with everything involved in the addon at that time. Since then it’s been contributions from others that have moved the addon on, the main change I can think off added after that was playlist support. If things have stopped working on the integration side since that version that’s most likely the cause. I guess we are going to have to try and pinpoint what went into subsequent versions that changed how it works..but my hunch is the playlist support.

Actually I thought 0.98 was your initial release from memory. Not that that's trustworthy at my age...lol Not sure if you recall back mid summer when I was working on UpNext integration I had some random communication issues and when I reached out you had mentioned 1.0.1 addressed those issues and it sure did so away I went. That's where my old memory says there was an initial release prior to 1.0.1..but that version imo was the best!

123Venom commented 4 years ago

Sorry @dagwieers , I didnt see your post. I can't get it working with Venom, even with the setup users apparently say works.

are you using the "Play from Here" kodi context menu option?

No, should I? If so, this is probably the issue open_mouth

not trying to be rude but when the integration took place back around May/.June I had a very good explanation of it's workings written up in the changelog and even commented to see the "Newsinfo" for a more detail expanation. There's a wealth of info in there for you to read and is accessible from the main menu. I believe it's even still in there. Yes you need to use "Play from here". Now that's been settled test again with v1.0.1 and let me know if it works there(UpNext v1.01 and Addon Signals 0.0.3 combo). Does not seem to work with any version of UpNext after that. Looks to me like nothing transmitted back to addon.

+1My bad, that's that solved.

No worries...now that we got that squared away test again and tell me if you can duplicate what I'm seeing, and about 6 other users who've reached out to me see over this.

ShaVerZ commented 4 years ago

Sorry @dagwieers , I didnt see your post. I can't get it working with Venom, even with the setup users apparently say works.

are you using the "Play from Here" kodi context menu option?

No, should I? If so, this is probably the issue open_mouth

not trying to be rude but when the integration took place back around May/.June I had a very good explanation of it's workings written up in the changelog and even commented to see the "Newsinfo" for a more detail expanation. There's a wealth of info in there for you to read and is accessible from the main menu. I believe it's even still in there. Yes you need to use "Play from here". Now that's been settled test again with v1.0.1 and let me know if it works there(UpNext v1.01 and Addon Signals 0.0.3 combo). Does not seem to work with any version of UpNext after that. Looks to me like nothing transmitted back to addon.

+1My bad, that's that solved.

No worries...now that we got that squared away test again and tell me if you can duplicate what I'm seeing, and about 6 other users who've reached out to me see over this.

I've been a little busy today, tomorrow I'll update to 1.0.2 and see what happens. I'll also try the latest version with Addon Signals 0.0.4 and update you. I'll capture logs and post.

Apologies for not being able to do this today, I appreciate the work you put in to this and Venom. I'm a new user to Venom and only switched a few weeks ago.....

ShaVerZ commented 4 years ago

Up Next v1.0.2 with Addon Signals v0.0.3 No Notification (Pop-up) https://pastebin.com/MGvf3Q5t

123Venom commented 4 years ago

Up Next v1.0.2 with Addon Signals v0.0.3 No Notification (Pop-up) https://pastebin.com/MGvf3Q5t

Right but what I wanted you to also test is UpNext 1.0.1 where it's working fine.

ShaVerZ commented 4 years ago

v1.0.1 is working with addon signals v0.0.3, did you want me to provide the log or just see if it is working?

123Venom commented 4 years ago

v1.0.1 is working with addon signals v0.0.3, did you want me to provide the log or just see if it is working?

I don't need it, it's not something fixable on my end.

im85288 / service.upnext

No notification pop-up appears with Venom #134