[Kde-finance-apps] Review of alkquotes

Brian Cappello briancappello at gmail.com
Thu Jul 1 05:36:44 CEST 2010


Hi Alvaro,
Thanks so much for taking the time to review and write out all of these
insightful comments! Now the challenge of fixing them :) I noticed too that
you were accepted to KDE e.V., congratulations!

On Tue, Jun 29, 2010 at 10:46 PM, Alvaro Soliverez <asoliverez at gmail.com>wrote:

> Hello Brian,
> here is a first review of the code:
>
> - For strings that will be displayed to the user, please use i18n, so
> the strings can be translated
> - Create a folder to drop all CMake modules there
> - Add unit tests, at least for all public methods in the backend classes
>



More information about the Kde-finance-apps mailing list