Code review comment for lp:~thisfred/ubuntuone-client/lp-702055

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

Great branch!

Two small issues:
 * I think that new_share_available in StatusFrontend should also pass _callback to messaging.show_message, just like new_udf_available
 * I think that StatusFrontend._callback should be named more descriptively.

Otherwise great work :-)

review: Needs Fixing

« Back to merge proposal