use "min" for syncing

[rectifyer]

Came across your plugin and looks pretty sweet! One small suggestion, you can send the "min" parameter for the user/library calls to retrieve about half the data. Might as well optimize as much as you can :)

Also, it would be cool to fork the official trakt plugin and merge in this functionality. I've been wanting to expand the official plugin with support for collection sync first, then watch status sync after that.

[N3MIS15]

I will add min param asap. Funny enough, I was going to contact you to see if this would be of any interest for the official plugin.

There is not a whole lot of error handling at the moment, but i will clean it up a bit and get a PR together.

[N3MIS15]

started adding sync to your addon: https://github.com/N3MIS15/script.trakt/tree/sync
only movies for now, episodes will be next

[dpons039]

Hey,

N3MIS15, I wrongly open it under the other place. Will be added soon the TV shows?

Also there's another issue with the Movies, it always mark 1 as not sync.

The pastebin error is: http://pastebin.com/qpB2K5Zj

KR,
David

[N3MIS15]

I know the issue with the movies section, it has something to do with incorrect metadata. I think some more logging is needed so the user can identify the movie(s) that are giving the issue so they can be at least identified. I had this happen to be in my testing and fixed it by rescraping the problem movie on xbmc.

The tvshows issue seems to be a missed exception on my part.
I will try to address these issues over the weekend when i have time.

[dpons039]

Okay, thank you very much :D