----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/109283/#review28757 -----------------------------------------------------------
Hi, when I try to apply the patch, git says "fatal: corrupt patch at line 118", reviewboard also complaing. Please ensure you have the full and correct patch and reupload, perhaps rebasing on top of the current master first. I also noticed that when the user doesn't want a track to be scrobbled, he probably doesn't want it to show up under Last.fm "now playing" so please add the check of isToBeScrobbled() to ScrobblerAdapter::updateNowPlaying(), too. Also please update toolTip of the config dialog option to mention that (no need to update the option test itself). - Matěj Laitl On March 6, 2013, 9:20 p.m., Vedant Agarwala wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/109283/ > ----------------------------------------------------------- > > (Updated March 6, 2013, 9:20 p.m.) > > > Review request for Amarok. > > > Description > ------- > > Added a check-box to disable scobbling for tracks with a particular label. > The label can be selected from a drop-down list or entered manually. Any > track that contains this label is skipped from being submitted to last.fm for > being scrobbled. > > > Diffs > ----- > > src/services/lastfm/LastFmConfigWidget.ui 5c5e51b > src/services/lastfm/LastFmServiceConfig.h 3f33b72 > src/services/lastfm/LastFmServiceSettings.h 41b2ead > src/services/lastfm/LastFmServiceSettings.cpp f6e1564 > src/services/lastfm/ScrobblerAdapter.h ea74196 > src/services/lastfm/ScrobblerAdapter.cpp b1a09f8 > src/statsyncing/Process.cpp c42fdc4 > src/statsyncing/ScrobblingService.h 971edd7 > > Diff: http://git.reviewboard.kde.org/r/109283/diff/ > > > Testing > ------- > > Builds and runs successfully. > > > File Attachments > ---------------- > > screen shot of the last fm service settings > > http://git.reviewboard.kde.org/media/uploaded/files/2013/03/06/modifiedLastFm.png > updated settings widget > > http://git.reviewboard.kde.org/media/uploaded/files/2013/03/06/modifiedLastFm1.png > > > Thanks, > > Vedant Agarwala > >
_______________________________________________ Amarok-devel mailing list Amarok-devel@kde.org https://mail.kde.org/mailman/listinfo/amarok-devel
