Re: Review Request: Mark podcast episodes to keep

lPodcastProvider.cpp 874738996d02970761d0701c0cd8634302659ed2 src/core/podcasts/PodcastMeta.h c2b5b2fe73295837c5e14fc3deb397208fecce1f src/playlistmanager/SyncedPodcast.h 5c270b1b2ca29a2226d79c4835c4a8c30a0d5fa3 Diff: http://git.reviewboard.kde.org/r/100998/diff/ Testing ---

Re: Review Request: GPodder Improvements Patch Rev2

onal. - Lucas --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104335/#review17525 --- On March 25, 2012, 8:34 p.m., Lu

Re: Review Request: Some changes to make Amarok appearance more pretty

On March 22, 2012, 10:04 p.m., Lucas Gomes wrote: > > Could you please explain what are the benefits (direct, current - not > > hypothetical or philosofical) of the TreeItem -> NormalTreeItem, > > MergedTreeItem split? Where exactly is avoids code duplication? > >

Re: Review Request: Some changes to make Amarok appearance more pretty

On March 22, 2012, 10:04 p.m., Lucas Gomes wrote: > > Could you please explain what are the benefits (direct, current - not > > hypothetical or philosofical) of the TreeItem -> NormalTreeItem, > > MergedTreeItem split? Where exactly is avoids code duplication? Ok, first

Re: Review Request: GPodder Improvements Patch Rev2

/GpodderSortFilterProxyModel.cpp ed1e8301f6f415e86e5367b7827817fff45feb98 Diff: http://git.reviewboard.kde.org/r/104335/diff/ Testing --- This patch should build. Everything is working as expected and there aren't any known issues. Thanks, Lucas Gomes ___ Amarok-

Re: Review Request: GPodder Improvements Patch Rev1

is working as expected and there aren't any known issues. Thanks, Lucas Gomes ___ Amarok-devel mailing list Amarok-devel@kde.org https://mail.kde.org/mailman/listinfo/amarok-devel

Re: Review Request: GPodder Improvements Patch Rev1

as expected and there aren't any known issues. Thanks, Lucas Gomes ___ Amarok-devel mailing list Amarok-devel@kde.org https://mail.kde.org/mailman/listinfo/amarok-devel

Review Request: GPodder Service improvements

. Everything is working as expected and there aren't any known issues. Thanks, Lucas Gomes ___ Amarok-devel mailing list Amarok-devel@kde.org https://mail.kde.org/mailman/listinfo/amarok-devel

Re: Review Request: Bug 261062 - JJ: Mark episodes to keep

it.reviewboard.kde.org/r/100998/#review10413 ------- On Feb. 8, 2012, 11:15 p.m., Lucas Gomes wrote: > > --- > This is an automatically generated e-mail. To re

Re: Review Request: Bug 261062 - JJ: Mark episodes to keep

vider.h c3d5e56 src/core-impl/podcasts/sql/SqlPodcastProvider.cpp 183005f src/core/podcasts/PodcastMeta.h 679f7ac src/playlistmanager/SyncedPodcast.h c186df2 Diff: http://git.reviewboard.kde.org/r/100998/diff/diff Testing --- Thanks, Lucas Gomes

Re: Review Request: Bug 261062 - JJ: Mark episodes to keep

/podcasts/sql/SqlPodcastProvider.cpp 183005f src/core/podcasts/PodcastMeta.h 679f7ac src/playlistmanager/SyncedPodcast.h c186df2 Diff: http://git.reviewboard.kde.org/r/100998/diff/diff Testing --- Thanks, Lucas Gomes ___ Amarok-devel mailin

Re: Review Request: Bug 261062 - JJ: Mark episodes to keep

/sql/SqlPodcastProvider.h c3d5e56 src/core-impl/podcasts/sql/SqlPodcastProvider.cpp 183005f src/core/podcasts/PodcastMeta.h 679f7ac src/playlistmanager/SyncedPodcast.h c186df2 Diff: http://git.reviewboard.kde.org/r/100998/diff/diff Testing --- Thanks, Lucas Gomes ___

Re: Review Request: Some changes to make Amarok appearance more pretty

------- On Jan. 29, 2012, 6:42 p.m., Lucas Gomes wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/103603/ >

Re: Review Request: Bug 261062 - JJ: Mark episodes to keep

casts/sql/SqlPodcastProvider.h c3d5e56 src/core-impl/podcasts/sql/SqlPodcastProvider.cpp 183005f src/core/podcasts/PodcastMeta.h 679f7ac Diff: http://git.reviewboard.kde.org/r/100998/diff/diff Testing --- Thanks, Lucas Gomes ___ Amarok-

Re: Review Request: Some changes to make Amarok appearance more pretty

diff/diff Testing --- This patch should build. Everything is working as expected and there aren't any known issues. Screenshots (updated) --- CollectionBrowser http://git.reviewboard.kde.org/r/103603/s/420/ PodcastBrowser http://git.reviewboard.kde.org/r/103603/s/423/

Re: Review Request: Some changes to make Amarok appearance more pretty

creenshots (updated) --- CollectionBrowser http://git.reviewboard.kde.org/r/103603/s/420/ PodcastBrowser http://git.reviewboard.kde.org/r/103603/s/421/ Thanks, Lucas Gomes ___ Amarok-devel mailing list Amarok-devel@kde.org https://mail.kde.org/mailman/listinfo/amarok-devel

Re: Review Request: Some changes to make Amarok appearance more pretty

--- PodcastBrowser http://git.reviewboard.kde.org/r/103603/s/402/ CollectionBrowser http://git.reviewboard.kde.org/r/103603/s/401/ Thanks, Lucas Gomes ___ Amarok-devel mailing list Amarok-devel@kde.org https://mail.kde.org/mailman/listinfo/amarok-devel

Re: Review Request: Some changes to make Amarok appearance more pretty

and there aren't any known issues. Screenshots (updated) --- PodcastBrowser http://git.reviewboard.kde.org/r/103603/s/402/ CollectionBrowser http://git.reviewboard.kde.org/r/103603/s/401/ Thanks, Lucas Gomes ___ Amarok-devel maili

Re: Review Request: Some changes to make Amarok appearance more pretty

.org/r/103603/s/390/ Thanks, Lucas Gomes ___ Amarok-devel mailing list Amarok-devel@kde.org https://mail.kde.org/mailman/listinfo/amarok-devel

Re: Review Request: Some changes to make Amarok appearance more pretty

tMeta.cpp b9851f7 src/widgets/TrackSelectWidget.cpp 5bd5059 Diff: http://git.reviewboard.kde.org/r/103603/diff/diff Testing --- This patch should build. Everything is working as expected and there aren't any known issues. Thanks, Lucas Gomes

Review Request: Some changes to make Amarok appearance more pretty

c/widgets/TrackSelectWidget.cpp 5bd5059 Diff: http://git.reviewboard.kde.org/r/103603/diff/diff Testing --- This patch should build. Everything is working as expected and there aren't any known issues. Thanks, Lucas Gomes ___ Amarok-devel mailing lis

Review Request: Some clean-up and sort

dder/GpodderTreeItem.cpp a772e4a Diff: http://git.reviewboard.kde.org/r/103195/diff/diff Testing --- Thanks, Lucas Gomes ___ Amarok-devel mailing list Amarok-devel@kde.org https://mail.kde.org/mailman/listinfo/amarok-devel

Re: Review Request: Important bugfixes to Gpodder Service

Playlist.h 0e94576 src/playlistmanager/SyncedPlaylist.cpp eaa856f src/playlistmanager/SyncedPodcast.h 85c80fc src/playlistmanager/SyncedPodcast.cpp 92dd6f8 src/services/gpodder/GpodderProvider.cpp 2ba0bf3 Diff: http://git.reviewboard.kde.org/r/103122/diff/diff Testing --- Than

Review Request: Important bugfixes to Gpodder Service

istmanager/SyncedPlaylist.cpp eaa856f src/playlistmanager/SyncedPodcast.h 85c80fc src/playlistmanager/SyncedPodcast.cpp 92dd6f8 src/services/gpodder/GpodderProvider.cpp 2ba0bf3 Diff: http://git.reviewboard.kde.org/r/103122/diff/diff Testing --- Thanks, L

Re: Review Request: Done some improvements on Gpodder Service

e.cpp 9e04d38 src/services/gpodder/GpodderServiceConfig.cpp 5ff6dea src/services/gpodder/GpodderServiceModel.cpp ef35c2f src/services/gpodder/GpodderServiceSettings.cpp 51ab598 Diff: http://git.reviewboard.kde.org/r/103034/diff/diff Testing --- Thanks, Lucas

Re: Review Request: Done some improvements on Gpodder Service

On Nov. 3, 2011, 8:34 a.m., Lucas Gomes wrote: > > I'll take a look at the provider not appearing at runtime, probably a > > QtGroupingProxy thing. Another problem is that the provider keep showing on PodcastBrowser even if I disable the gpodder s

Re: Review Request: Done some improvements on Gpodder Service

ice.h 02ff539 src/services/gpodder/GpodderService.cpp 9e04d38 src/services/gpodder/amarok_service_gpodder.desktop 10661b7 Diff: http://git.reviewboard.kde.org/r/103034/diff/diff Testing --- Thanks, Lucas Gomes ___ Amarok-devel mailing list A

Review Request: Done some improvements on Gpodder Service

ngs.h fa53387 src/services/gpodder/GpodderServiceSettings.cpp 51ab598 Diff: http://git.reviewboard.kde.org/r/103034/diff/diff Testing --- Thanks, Lucas Gomes ___ Amarok-devel mailing list Amarok-devel@kde.org https://mail.kde.org/mailman/lis

Review Request: [GSoC] Amarok integration with gpodder.net (GPodder stuff)

hanks, Lucas Gomes ___ Amarok-devel mailing list Amarok-devel@kde.org https://mail.kde.org/mailman/listinfo/amarok-devel

Review Request: [GSoC] Amarok integration with gpodder.net (Playlist synchronisation stuff)

--- This patch is the result of git diff for the range of several commits prior to HEAD, rebased on current master, and should build. Everything is working as expected and there aren't any known issues. Thanks, Lucas Gomes ___ Amarok-

Re: Review Request: [GSoC] Amarok integration with gpodder.net

personal clone repository. http://quickgit.kde.org/?p=clones%2Famarok%2Fgomes%2Fmaskmaster-amarok.git&a=summary - Lucas Gomes On Sept. 24, 2011, 2:25 p.m., Lucas Gomes wrote: > > --- > This is an automatically generated e-m

Re: Review Request: [GSoC] Amarok integration with gpodder.net

--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102600/ --- (Updated Sept. 24, 2011, 2:25 p.m.) Review request for Amarok, Stefan Derk

Review Request: [GSoC] Amarok integration with gpodder.net

--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102600/ --- Review request for Amarok, Stefan Derkits and Bart Cerneels. Summary -

Re: Review Request: Bug 261062 - JJ: Mark episodes to keep

--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/100998/ --- (Updated April 1, 2011, 2:59 p.m.) Review request for Amarok. Summary --

Review Request: Bug 261062 - JJ: Mark episodes to keep

--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/100998/ --- Review request for Amarok. Summary --- Now downloaded episodes are ke