Code review comment for lp:~pete-woods/unity-scope-soundcloud/native-rewrite

Revision history for this message
Pete Woods (pete-woods) wrote :

> Could we make this a proper module instead of yet another daemon? I know the
> tutorial now shows how to do a daemon, but modules are the way forward.

I only had a single day available to work on this, so I don't know if I'm supposed to make any more changes, but never mind that.

Have you got a good example of a module I could refer to?

Also, I couldn't get the track preview working with this. It just appears as a generic preview, with just a button (action?) that loads the SoundCloud website, there's no "play" button or anything like that.

Do you have any suggestions about what I've done wrong with the preview?

« Back to merge proposal