Code review comment for lp:~knitzsche/ubuntu-rest-scopes/fix-10-day-forecast-url

Revision history for this message
Facundo Batista (facundo) wrote :

Ah, it's a good catch that the URL needs a 5 digit precision (even if the numbers are completed with zeros).

A detail in the formatting: the 2 as width doesn't make sense, as if you request 5 digits for precision, it always will be greater than 2, so please remove it.

Also, please include this formatting change in the ENGLISH_URL_TEMPLATE itself, no need to prebuild other string.

Thanks!!

review: Needs Fixing

« Back to merge proposal