mintsoft / kodi.plugin.ytchannels

kodi.plugin.ytchannels
GNU General Public License v3.0
5 stars 8 forks source link

Changed search maximum to 50, sorted video list by published date #8

Closed yeahme49 closed 3 years ago

yeahme49 commented 3 years ago

Same as matrix PR. Not sure what the best way to handle changes are. Do you want me to submit them to both branches going forward? or just to master and you can rebase matrix? Not sure what the best/easiest way is.

One thing I was testing was using the new settings format on Leia. https://kodi.wiki/view/Add-on_settings_conversion#Preface says Leia can use the new format but recommends against it. I tested and it appears everything worked except the erase database function, it wouldn't run. No errors but I'm assuming its one of the bugs in Leia that the wiki mentions. Erase database could maybe be moved to the context menu instead of settings, then the new settings format could be used with leia and matrix, one less thing to have different between the 2 versions.

mintsoft commented 3 years ago

I'd go for just to master and I'll rebase matrix; if there's anything matrix specific it's probably best to do that in a separate PR because it'll end up conflicted anyway