Code review comment for lp://qastaging/~verzegnassi-stefano/ubuntu-docviewer-app/document-hub2

Revision history for this message
Stefano Verzegnassi (verzegnassi-stefano) wrote :

I reverted to 'QCoreApplication::setOrganizationDomain".

Here's the reason of my choice: the code has been on testing for a few weeks, and even if the line was wrong, it used to work with no issue both on desktop and on devices.
Since I have no UT device at the moment - my brand new (a.k.a. "refurbished") hammerhead has been shipped today - I can't properly test if any further change will cause some regression.

Content-hub used to work, as all the rest of the application did.
Because of this, I assume that everything is ok for a release.

I should get my new Nexus 5 in max. 2-3 days: when it will happen, I'll be able to fix this part of the code.

« Back to merge proposal