-
Notifications
You must be signed in to change notification settings - Fork 305
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
[script.trakt] 3.3.0 #1206
[script.trakt] 3.3.0 #1206
Conversation
Travis tests were successfulHey @razzeee, |
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.
See comments. Mostly some files that need to be removed from the submission. Apart from that, great work as usual. Thanks!
def setSetting(setting, value): | ||
__addon__.setSetting(setting, str(value)) | ||
|
||
def getSettingAsBool(setting): |
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.
should not be needed in newer api versions
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.
Can I change that in the next version?
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.
sure
if getSettingAsBool('debug'): | ||
try: | ||
xbmc.log(self.format(record), levels[record.levelno]) | ||
except UnicodeEncodeError: |
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.
should not be needed (matrix is py3 only)
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.
Can I change that in the next version?
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.
sure
script.trakt/tests/fixtures/compare_shows_batman_episode_to_add.json
Outdated
Show resolved
Hide resolved
Travis tests were successfulHey @razzeee, |
Travis tests were successfulHey @razzeee, |
Description
Here's the new version for matrix.
Two things happend, I ported to matrix and lunatix did some work on PVR detection.
Diff is here:
trakt/script.trakt@3.2.4...master
Checklist: