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.

Sign up for GitHub

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

Jump to bottom

trakt.tv integration #27

Closed
nihir opened this issue Mar 2, 2014 · 8 comments
Closed

trakt.tv integration #27

nihir opened this issue Mar 2, 2014 · 8 comments
Assignees
@garfield69
Labels
enhancement

Comments

@nihir
Copy link

nihir commented Mar 2, 2014

It'd be great to include integration with trakt.tv

This would import viewed status' and watched shows.

Thanks!
@SchizoDuckie
Copy link
Owner

There's a couple of things I want to do before this, but should not be a problem to implement.

@rvisharma
Copy link

+1 for Trakt.tv integration!

@thecodingdude
Copy link

I was going to actually make a thread about this myself - trakt.tv integration is a must tbth.

SchizoDuckie added a commit that referenced this issue Mar 17, 2014
@SchizoDuckie
A start on trakt.tv integration for episode listings. Looks promising…
89080a8 
…, and will resolve tvdb / tvrage matching problems. re #27
SchizoDuckie added a commit that referenced this issue Mar 31, 2014
@SchizoDuckie
W.i.p. switching to Trakt.TV as main api. Re #27
3e8cc73 
W.i.p importexport freature to backup settings for upgrading and transferring.
@garfield69
Copy link
Collaborator

additional trakt.tv user suggestions:
http://www.reddit.com/r/duckietv/comments/2617ch/suggestion_better_trakt_integration/

SchizoDuckie added a commit that referenced this issue Jul 5, 2014
@SchizoDuckie
include sha1 service for trakt.tv auth re #27
bef461d
SchizoDuckie added a commit that referenced this issue Jul 5, 2014
@SchizoDuckie
Remove global form so that the subform works re #27
9a2cf3d
SchizoDuckie added a commit that referenced this issue Jul 5, 2014
@SchizoDuckie
add trakt.tv auth form re #27
cdd8bb9
SchizoDuckie added a commit that referenced this issue Jul 5, 2014
@SchizoDuckie
Add basics for trakt.tv settings in settingsservice re #27
3f1f0bd
SchizoDuckie added a commit that referenced this issue Jul 5, 2014
@SchizoDuckie
Trakttv controller that updates the viewvalue of the form element the…
7f2f162 
… proper way re #27
SchizoDuckie added a commit that referenced this issue Jul 6, 2014
@SchizoDuckie
Implemented trakt.tv reading favorite shows and displaying them with …
c53e958 
…fetched episodes as serieHeader directive. re #27

Todo: feed shows to FavoritesService.addFavorite and parse TraktTv.getUserWatched into episodes to mark as watched.
@garfield69 garfield69 mentioned this issue Jul 6, 2014
Merged
garfield69 added a commit that referenced this issue Jul 6, 2014
@garfield69
Merge pull request #190 from garfield69/angular
88e7056 
initial translations for #27
SchizoDuckie added a commit that referenced this issue Jul 6, 2014
@SchizoDuckie
Fix for chrome bug #377860 (download parameter ignored since v0.35 )
98d5a6f 
Moved trakttv controller into it's own tab made trakttv import work and mark episodes at watched. re #27
SchizoDuckie added a commit that referenced this issue Jul 6, 2014
@SchizoDuckie
Fixed seriectrl not loading due to removed trakt.tv dependency re #27
495296d
SchizoDuckie added a commit that referenced this issue Jul 6, 2014
@SchizoDuckie
Added trakttv tab translation re #27
6874b2a
SchizoDuckie added a commit that referenced this issue Jul 6, 2014
@SchizoDuckie
Made trakttv import count and show howmany watched episodes exist on …
1b7a909 
…trakt.tv and hide serieheader titles re #27
SchizoDuckie added a commit that referenced this issue Jul 6, 2014
@SchizoDuckie
Made trakttv import count and show howmany watched episodes exist on …
1820e84 
…trakt.tv and hide serieheader titles re #27
garfield69 added a commit that referenced this issue Jul 6, 2014
@garfield69
Merge pull request #193 from garfield69/angular
352a865 
TRAKTTV tab translations re #27
SchizoDuckie added a commit that referenced this issue Jul 6, 2014
@SchizoDuckie
Handle and refactoring of episode marking watched and not watched wit…
8106d0a 
…h forwarding to trakt.tv re #27
SchizoDuckie added a commit that referenced this issue Jul 6, 2014
@SchizoDuckie
When password hash has been stored, show the trakt.tv autosync enable…
f40258c 
… button that marks episodes as watched on trakt.tv re #27
SchizoDuckie added a commit that referenced this issue Jul 6, 2014
@SchizoDuckie
Fetch trakt.tv settings from SettingsService and forward the markAsWa…
a82a44c 
…tched calls to the entity to improve DRY re #27
SchizoDuckie added a commit that referenced this issue Jul 11, 2014
@SchizoDuckie
W.i.p. adding trakt.tv recommendations based on your takt.tv profile re
287d9d5 
#27
SchizoDuckie added a commit that referenced this issue Jul 13, 2014
@SchizoDuckie
Added pushing all shows and episodes watched to trakt.tv and fixed fe…
14c73ac 
…tching them. 2 way sync now working. (Requires some more UI feedback when pushing, done for the rest i guess) re #27
@SchizoDuckie
Copy link
Owner

I've cleaned up this thread a bit by removing the off-topic discussion on git workflow/clients

As far as I can see it, this is finished, and just needs some textual context in the template/translations.

Perhaps it's cool to ask users on upgrade/first install with a nice popup $dialog if they have a trakt account, and guide them to the settings page then. Totally optional though, I'm happy releasing this as-is as well

@garfield69
Copy link
Collaborator

You need to update en_us.json line 336
"TRAKTTV/sync/desc": "TraktTV description and instructions to be added here.",
Once you've switched the issue label from Translations needed (when finished to Ready for translation then I'll do the translations for the other languages as well as any outstanding for #124.

SchizoDuckie added a commit that referenced this issue Jul 15, 2014
@SchizoDuckie
Finished trakt.tv UI and half of the translations (didn't add new key…
d685dd5 
…s yet) re #27
@SchizoDuckie
Copy link
Owner

That should do it :)

@garfield69
Copy link
Collaborator

translations completed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@garfield69 garfield69

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@SchizoDuckie @nihir @thecodingdude @rvisharma @garfield69