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

Google Drive and Dropbox resolvers #172

Open
wants to merge 90 commits into
base: master
Choose a base branch
from

Conversation

AltarBeastiful
Copy link

Changes necessary in Tomahawk to implement Google Drive and Dropbox resolvers.

To compile :

Features :

  • Resolvers can register to signals of their config.ui dialog.

Usage : Add a connections field in the settings of the resolver containing the signals of the current widget.

{
    fields: [{
                name: "associateButton",
                widget: "associateButton",      
                connections : [{ 
                                  signal: "clicked()", 
                                  javascriptCallback: "resolver.associateClicked();" 
                              }]  
            }]
}
  • Parse ID3 tag from stream asynchronously. JS resolvers can call readCloudFile to read the tags from a distant music file.
  • Resolver can request a web page to be opened to the user (used for Oauth).
  • Asynch and Sync post request in tomahawk.js

Review on Reviewable

AltarBeastiful and others added 30 commits February 10, 2013 17:23
… config.ui.

Remove custom Dropbox account factory from QtResolver.
…thode in QtScriptResolver to parse ID3Tags from a stream ; modify cmake configuration to use taglib 1.8 at least
teo and others added 28 commits March 28, 2013 15:04
Conflicts:
	src/libtomahawk/resolvers/QtScriptResolver.cpp
Conflicts:
	src/libtomahawk/resolvers/QtScriptResolver.cpp
Conflicts:
	src/AudioControls.cpp
@loclamor
Copy link

loclamor commented Jun 7, 2013

Hi,

what about our work ?

@dschmidt
Copy link
Member

better ping on irc than here, will talk to @muesli tomorrow.. :)

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.

8 participants