Bugzilla – Bug 13123
Generic app setup for all apps
Last modified: 2009-10-05 16:42:59 UTC
Refactor all apps to generic on-device setup. See Napster for an example. Nice to have: Refactor web code in User::Account::<service> to use the same common API methods.
*** Bug 13108 has been marked as a duplicate of this bug. ***
If we dont do this, what is broken now?
No choice, it has to be done. See http://embargo.wiki.slimdevices.com/index.php?title=App_Gallery_Spec#Screens:_App_Launch.2FSetup
Going to make a running list here of what's left and what's done for this bug: Done: # Napster TODO: # Classical # Facebook # LastFM (switch to 'mobile' version of API auth that does not require browser) # Live365 # Mediafly # MP3tunes # Pandora # Rhapsody # Sirius # Slacker # RadioIO? # DI.fm? # Sky.fm? Won't fix: # Deezer (only MOD feature requires account, and no plans to support this yet)
Facebook done.
== Automatic comment from SVN commit #6837 to Repository: network == https://svn.slimdevices.com/network?view=revision&revision=6837 Bug 13123 - refactor Pandora to use generic setup, generic accounts menu, multiple accounts. Web UI does not support multiple accounts, can wait for new UI == Modified 9 files total == squeezenetwork/branches/qa/docroot/strings.txt squeezenetwork/branches/qa/docroot/user/account/pandora.tt squeezenetwork/branches/qa/lib/Net/Pandora/V3.pm squeezenetwork/branches/qa/lib/SDI/SqueezeNetwork/Controller/API/Pandora/V1/OPML/TrackInfo.pm squeezenetwork/branches/qa/lib/SDI/SqueezeNetwork/Controller/API/Pandora/V1/OPML.pm squeezenetwork/branches/qa/lib/SDI/SqueezeNetwork/Controller/API/Pandora/V1/Playback.pm squeezenetwork/branches/qa/lib/SDI/SqueezeNetwork/Controller/API/Pandora.pm squeezenetwork/branches/qa/lib/SDI/SqueezeNetwork/Controller/User/Account/Pandora.pm squeezenetwork/branches/qa/lib/SDI/Util/Services.pm
== Automatic comment from SVN commit #6839 to Repository: network == https://svn.slimdevices.com/network?view=revision&revision=6839 Bug 13123 - refactor Last.fm for app setup, multiple accounts. Needs more work on the web UI later, and probably fixes for SC pref syncing == Modified 11 files total == squeezenetwork/branches/qa/docroot/strings.txt squeezenetwork/branches/qa/docroot/user/account/lastfm.tt squeezenetwork/branches/qa/lib/Net/LastFM/V2.pm squeezenetwork/branches/qa/lib/SDI/SqueezeNetwork/Controller/API/LastFM/V1/OPML/TrackInfo.pm squeezenetwork/branches/qa/lib/SDI/SqueezeNetwork/Controller/API/LastFM/V1/OPML.pm squeezenetwork/branches/qa/lib/SDI/SqueezeNetwork/Controller/API/LastFM/V1/Playback.pm squeezenetwork/branches/qa/lib/SDI/SqueezeNetwork/Controller/API/LastFM/V1/Scrobbling.pm squeezenetwork/branches/qa/lib/SDI/SqueezeNetwork/Controller/API/LastFM.pm squeezenetwork/branches/qa/lib/SDI/SqueezeNetwork/Controller/User/Account/LastFM.pm squeezenetwork/branches/qa/lib/SDI/SqueezeNetwork/Controller/User/Account/Pandora.pm squeezenetwork/branches/qa/lib/SDI/Util/Services.pm
Grr, dumb checkin script.
== Automatic comment from SVN commit #6854 to Repository: network == https://svn.slimdevices.com/network?view=revision&revision=6854 Bug 13123 - refactor Rhapsody for generic/multiple accounts == Modified 5 files total == squeezenetwork/branches/qa/lib/SDI/SqueezeNetwork/Controller/API/AppSetup/V1/OPML.pm squeezenetwork/branches/qa/lib/SDI/SqueezeNetwork/Controller/API/Rhapsody/V1/OPML/Account.pm squeezenetwork/branches/qa/lib/SDI/SqueezeNetwork/Controller/API/Rhapsody/V1/OPML.pm squeezenetwork/branches/qa/lib/SDI/SqueezeNetwork/Controller/API/Rhapsody.pm squeezenetwork/branches/qa/lib/SDI/Util/Services.pm
== Automatic comment from SVN commit #6855 to Repository: network == https://svn.slimdevices.com/network?view=revision&revision=6855 Bug 13123 - Rhapsody trial stuff == Modified 5 files total == squeezenetwork/branches/qa/lib/SDI/SqueezeNetwork/Controller/API/AppSetup/V1/OPML.pm squeezenetwork/branches/qa/lib/SDI/SqueezeNetwork/Controller/API/Napster.pm squeezenetwork/branches/qa/lib/SDI/SqueezeNetwork/Controller/API/Rhapsody/V1/OPML.pm squeezenetwork/branches/qa/lib/SDI/SqueezeNetwork/Controller/API/Rhapsody.pm squeezenetwork/branches/qa/lib/SDI/SqueezeNetwork/Controller/API.pm
== Automatic comment from SVN commit #6857 to Repository: network == https://svn.slimdevices.com/network?view=revision&revision=6857 Bug 13123 - refactor Sirius for generic signup/multiple accounts. Web UI currently disabled == Modified 12 files total == squeezenetwork/branches/qa/docroot/user/account/sirius.tt squeezenetwork/branches/qa/lib/Net/Sirius/V1.pm squeezenetwork/branches/qa/lib/SDI/SqueezeNetwork/Controller/API/AppSetup/V1/OPML.pm squeezenetwork/branches/qa/lib/SDI/SqueezeNetwork/Controller/API/LastFM.pm squeezenetwork/branches/qa/lib/SDI/SqueezeNetwork/Controller/API/Napster.pm squeezenetwork/branches/qa/lib/SDI/SqueezeNetwork/Controller/API/Pandora.pm squeezenetwork/branches/qa/lib/SDI/SqueezeNetwork/Controller/API/Rhapsody.pm squeezenetwork/branches/qa/lib/SDI/SqueezeNetwork/Controller/API/Sirius/V1/OPML.pm squeezenetwork/branches/qa/lib/SDI/SqueezeNetwork/Controller/API/Sirius/V1/Playback.pm squeezenetwork/branches/qa/lib/SDI/SqueezeNetwork/Controller/API/Sirius.pm squeezenetwork/branches/qa/lib/SDI/SqueezeNetwork/Controller/API.pm squeezenetwork/branches/qa/lib/SDI/Util/Services.pm
== Auto-comment from SVN commit #6878 to the network repo by andy == == https://svn.slimdevices.com/network?view=revision&revision=6878 == Bug 13123 - refactored web settings pages for Rhapsody, Napster, Sirius
== Auto-comment from SVN commit #6880 to the network repo by andy == == https://svn.slimdevices.com/network?view=revision&revision=6880 == Bug 13123 - refactor Slacker
== Auto-comment from SVN commit #6923 to the network repo by andy == == https://svn.slimdevices.com/network?view=revision&revision=6923 == Bug 13123 - refactor Live365
I had to do something a bit different for the MP3tunes free trial. This trial requires an email, password, first name, and last name. I've automated this by using the user's MySB account info, generating a random password, and emailing them the details. I used a template left over from a previous idea where we were going to auto-register people for accounts. I think this will work OK.
== Auto-comment from SVN commit #6925 to the network repo by andy == == https://svn.slimdevices.com/network?view=revision&revision=6925 == Bug 13123 - refactor MP3tunes
FYI, re MP3 Tunes (and all other third-party apps that require an account), while that seems like it will work, I would be fine if we sent users to our partners' sites to do any "registration," whether for a free trial or for full accounts. And then we'd just support entry of the username/password for each respective service. This keeps it consistent across all services and avoids us having to deal with supporting their varying data collection needs (e.g., first name, last name, zip/postal code, age, credit card number, etc.).
Yes, but unfortunately that won't work because the MP3tunes trial is tied to our partner ID and must be created from our site. We'd have to get them to develop something on their site for your idea to work.
== Auto-comment from SVN commit #6930 to the network repo by andy == == https://svn.slimdevices.com/network?view=revision&revision=6930 == Bug 13123 - refactor RadioIO
== Auto-comment from SVN commit #6932 to the network repo by andy == == https://svn.slimdevices.com/network?view=revision&revision=6932 == Bug 13123 - refactor RadioTime
== Auto-comment from SVN commit #6933 to the network repo by andy == == https://svn.slimdevices.com/network?view=revision&revision=6933 == Bug 13123 - refactor Classical
== Auto-comment from SVN commit #6938 to the network repo by andy == == https://svn.slimdevices.com/network?view=revision&revision=6938 == Bug 13123 - refactor Mediafly
== Auto-comment from SVN commit #6946 to the network repo by andy == == https://svn.slimdevices.com/network?view=revision&revision=6946 == Bug 13123 - DI.fm/Sky.fm support, still need to add Sky.fm stream list
This should be done now!
This bug has been fixed in the latest release of MySqueezebox.com (formally known as SqueezeNetwork)! If you are still experiencing this problem, feel free to reopen the bug with your new comments and we'll have another look.