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

Do not merge! Site change #91

Closed
wants to merge 49 commits into from
Closed

Conversation

linqcan
Copy link
Collaborator

@linqcan linqcan commented Feb 17, 2014

Do not merge!
Branch will be squashed when new site goes live!

PR created for reference.

Updated against: http://beta.svtplay.se/

Before going live

  • Change BASE_URL in the svt module
  • Update background image
  • Update README and docs
  • Decide if BestOfSvt.se should be included or not ("Populärt" and "Sista chansen" are "native" featues now)
  • Create settings module for accessing settings via constants
  • Move logic to find video URL to SVT module
  • Add images to categories Images on Categories #92
  • Add category "Sändningar"
  • Convert SVT date format to XBMC date format for proper "aired" support
  • Add unit tests for the SVT module
  • Add playlist feature
  • Add better support for live broadcasts (only show ongoing?)
  • Add support for "Kanaler"

@sefsh
Copy link

sefsh commented Mar 5, 2014

Thanks for maintaining this addon. 💓

@linqcan
Copy link
Collaborator Author

linqcan commented Mar 6, 2014

Pleasure! 😄

@linqcan
Copy link
Collaborator Author

linqcan commented May 5, 2014

Launch is approaching: http://kundo.se/org/svt-play-beta/d/chromecast-5/#c679102

@nilzen
Copy link
Collaborator

nilzen commented May 5, 2014

I guess we should merge this in to master soon then and push it to the add on repo for Gotham, after they launch the new site we can just change the url from beta...

@linqcan
Copy link
Collaborator Author

linqcan commented May 5, 2014

Maybe we should wait until launch, they have changed the naming conventions on some elements back and forth. Guessing this might change until the last minute. Trying to avoid quick fixes due to their last minute change of naming...

Will hopefully be able to clean up the playlist feature and merge/include it into this PR this week.

@linqcan
Copy link
Collaborator Author

linqcan commented May 12, 2014

Added basic playlist support. Adding items from the playlist is done by using the context menu ("c" on a keyboard and "hold menu" on ATV2 (?)). Removing an item from the playlist is done by clicking the context menu button while the item is selected in the mini-gui for the playlist ("playlist manager"). This needs to be added to README and maybe integrate a guide somehow in the plugin.

@linqcan
Copy link
Collaborator Author

linqcan commented Jun 3, 2014

Preparing for release. Fixing against last-minutes changes from SVT.

@linqcan
Copy link
Collaborator Author

linqcan commented Jun 4, 2014

Closing this PR. Merge against gotham instead.

@linqcan linqcan closed this Jun 4, 2014
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.

None yet

3 participants