Code review comment for lp:~tvec623/ubuntu-desktop-accomplishments/20120814_accomp_importshowell

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

Great accomplishment, Tony! Thanks for submitting it. I have tested it, and I have two notes for you:

 * In the 'steps' field, step 4 is unclear ("Click the icon"). While I know that you meant 'click the icon you are presented with in dash search results' (though it took me some time to guess), I'd suggest to merge this step with the previous one, because it makes an impression that one needs to click some icon /after/ they launch Shotwell. And, after all, the point of this accomplishment is not explaining how to launch an application via dash, but how to get shotwell to manage one's images :-)
 * For some reason I am not awarded with this accomplishment even though I have imported photos to Shotwell. The script still exits with code 1... I have done a simple debugging, and it seems that ~/.shotwell directory is not present on my system. Can it be it stores my images DB in some other location? It would be great if you managed to fix this. If you need me to assist you by trying some scripts on my system which misses the ~/.shotwell directory, feel free to get in touch with me at #ubuntu-accomplishments on freenode :-)

review: Needs Fixing

« Back to merge proposal