Code review comment for lp:~dobey/unity-scope-click/payments-button

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

The branch looks ok, but I think it would be safer to keep this feature behind an environ variable, so it does not break when the index starts having prices.
Please add such a check in the "if (details.package.price > double(0.00)) {".

review: Needs Fixing

« Back to merge proposal