Code review comment for lp:~tvec623/ubuntu-desktop-accomplishments/20120827-sudoku-accomplishment

Revision history for this message
Rafał Cieślak (rafalcieslak256) wrote :

Cool accomplishments! We were definitely lacking these ones. Great thanks for your work on them.

There are two small things that will need fixing before I merge this branch:

* Somehow the importshotwell accomplishment is included in this branch too. I guess that's accidental, as you have proposed a separate branch with them.
* The category your Sudoku accomplishments use is "Gaming". That's correct, but it would be cool if we grouped them to a sub-category withing gaming, named "Sudoku". To do this, you need to set their category to "Gaming:Sudoku".

review: Needs Fixing

« Back to merge proposal