Re: Review Request 128608: Add Donate entry to KStandardShortcut
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128608/ --- (Updated Aug. 14, 2016, 12:16 p.m.) Status -- This change has been marked as submitted. Review request for KDE Frameworks and Matthew Dawson. Changes --- Submitted with commit 3434b9efd0a8b77d558e4376af38e17d59ded66c by Elvis Angelaccio to branch master. Repository: kconfig Description --- This patch adds a Donate entry, such that https://git.reviewboard.kde.org/r/128609/ builds. There is no default shortcut set. Diffs - src/gui/kstandardshortcut.h a5a7c28f86094bd346abc8dc17e24a6bd365f5bd src/gui/kstandardshortcut.cpp 6d749455782d55a81ed5448f35eade954c1df615 Diff: https://git.reviewboard.kde.org/r/128608/diff/ Testing --- Thanks, Elvis Angelaccio
Re: Review Request 128608: Add Donate entry to KStandardShortcut
> On Aug. 9, 2016, 11:23 p.m., Matthew Dawson wrote: > > LGTM. I can't speak for the other review. Sorry, one other thing: Please make sure to put a CHANGELOG: entry in the commit message. Thanks! - Matthew --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128608/#review98258 --- On Aug. 8, 2016, 5:05 a.m., Elvis Angelaccio wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/128608/ > --- > > (Updated Aug. 8, 2016, 5:05 a.m.) > > > Review request for KDE Frameworks and Matthew Dawson. > > > Repository: kconfig > > > Description > --- > > This patch adds a Donate entry, such that > https://git.reviewboard.kde.org/r/128609/ builds. > > There is no default shortcut set. > > > Diffs > - > > src/gui/kstandardshortcut.h a5a7c28f86094bd346abc8dc17e24a6bd365f5bd > src/gui/kstandardshortcut.cpp 6d749455782d55a81ed5448f35eade954c1df615 > > Diff: https://git.reviewboard.kde.org/r/128608/diff/ > > > Testing > --- > > > Thanks, > > Elvis Angelaccio > >
Re: Review Request 128608: Add Donate entry to KStandardShortcut
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128608/#review98258 --- Ship it! LGTM. I can't speak for the other review. - Matthew Dawson On Aug. 8, 2016, 5:05 a.m., Elvis Angelaccio wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/128608/ > --- > > (Updated Aug. 8, 2016, 5:05 a.m.) > > > Review request for KDE Frameworks and Matthew Dawson. > > > Repository: kconfig > > > Description > --- > > This patch adds a Donate entry, such that > https://git.reviewboard.kde.org/r/128609/ builds. > > There is no default shortcut set. > > > Diffs > - > > src/gui/kstandardshortcut.h a5a7c28f86094bd346abc8dc17e24a6bd365f5bd > src/gui/kstandardshortcut.cpp 6d749455782d55a81ed5448f35eade954c1df615 > > Diff: https://git.reviewboard.kde.org/r/128608/diff/ > > > Testing > --- > > > Thanks, > > Elvis Angelaccio > >
Re: Review Request 128608: Add Donate entry to KStandardShortcut
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128608/ --- (Updated Aug. 8, 2016, 9:05 a.m.) Review request for KDE Frameworks and Matthew Dawson. Changes --- Bump @since to 5.26 Repository: kconfig Description --- This patch adds a Donate entry, such that https://git.reviewboard.kde.org/r/128609/ builds. There is no default shortcut set. Diffs (updated) - src/gui/kstandardshortcut.h a5a7c28f86094bd346abc8dc17e24a6bd365f5bd src/gui/kstandardshortcut.cpp 6d749455782d55a81ed5448f35eade954c1df615 Diff: https://git.reviewboard.kde.org/r/128608/diff/ Testing --- Thanks, Elvis Angelaccio
Review Request 128608: Add Donate entry to KStandardShortcut
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128608/ --- Review request for KDE Frameworks and Matthew Dawson. Repository: kconfig Description --- This patch adds a Donate entry, such that https://git.reviewboard.kde.org/r/128609/ builds. There is no default shortcut set. Diffs - src/gui/kstandardshortcut.h a5a7c28f86094bd346abc8dc17e24a6bd365f5bd src/gui/kstandardshortcut.cpp 6d749455782d55a81ed5448f35eade954c1df615 Diff: https://git.reviewboard.kde.org/r/128608/diff/ Testing --- Thanks, Elvis Angelaccio ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel