-
Notifications
You must be signed in to change notification settings - Fork 34
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
Implemented: Remove a playback (progress) item. #54
Conversation
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.
Thanks for the contribution 😄, please see my inline comment.
trakt/interfaces/sync/playback.py
Outdated
@@ -16,3 +16,10 @@ def episodes(self, store=None, **kwargs): | |||
store, | |||
**kwargs | |||
) | |||
|
|||
@authenticated | |||
def delete_progress(self, playbackid, **kwargs): |
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.
Do we need this method? couldn't the delete
method be used directly?
3 similar comments
Thanks for your comment. I guess we don't need it. |
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.
Looks good 👍, preparing new release now.
Released in v2.13.0 |
[recreated #53 from a clean slate]
I noticed that remove a playback item was not yet implemented, so I implemented it myself.
It might not be the best code or way to accomplish that,
but it's working, and that's something.
Usage is:
Trakt['sync/playback'].delete_progress(pbid)
Trakt['sync/playback'].delete(pbid)
pbid
being the playback id you get fromTrakt['sync/playback'].get()
If improvements are needed, the are very welcome.
P.S. I also made a script to test use this..
https://github.com/sharkykh/TraktPlaybackProgressManager