Skip to content
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

Add missing sync history methods #125

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

defhead
Copy link

@defhead defhead commented Mar 26, 2021

No description provided.

@UweTrottmann
Copy link
Owner

Thanks!

This duplicates the methods already available from the Users service, no? E.g. you can just use me as username.

@defhead
Copy link
Author

defhead commented Mar 31, 2021

Maybe it does, I just thought that since it's available on Trakt API, it would be interesting to give developers a choice (personally I prefer going throught sync services).
Also, I saw that collections API were also in /sync and /users services, so I didn't think it would be an issue.

(BTW, I don't know why there are failing tests. I didn't touch those ones)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants