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
[PVR][JSON-RPC] Expose EPG tag's series number as 'seasonnum' property. #20003
Conversation
@ksooo this is a non-breaking interface change so bump JSON minor version e.g. 12.4.0 It is best practice (results can be parsed positionally?) as a non-breaking change that the added field needs to go at the end of the type definition and field enum rather than inserted (in the obvious place ) as you have done. Yes to backporting this as a fixm since we still have v19 and v20 inline with each other. |
ce70818
to
7551b73
Compare
@DaveTBlake thanks, requested changes are now in. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Great, thx @ksooo If you provide a current windows build, i'm ready to test it. But just for my interest: what exactly episodepart is intended for? Is there an equivalent info label for that? |
Honestly, I have no idea what episodepart is good for. It was introduced before my Kodi time. |
For details see #19999 (comment)
@phunkyfish when you find some time for a code review...
@DaveTBlake is the JSON-RPC change okay?
BTW, I wouldlike to backport this as I consider it a fix.
@louisbraun fyi