Code review comment for lp:~stolowski/unity-scope-click/local-departments

Revision history for this message
Alejandro J. Cura (alecu) wrote :

Great work pawel!

I know the branch is in progress, so let me give an initial observation: in order for the branch to be more easily reviewable, and to make sure that it does not grow much more, please split it in a few branches.

The first one should only include all the methods that do the queries and the db access, and all the corresponding tests.
The second one should start filling the db from preview.cpp using the branch above.
The third one would query the db to build the combo of departments.

review: Needs Fixing

« Back to merge proposal