Code review comment for ~ltrager/maas:lp1733015

Revision history for this message
Mike Pontillo (mpontillo) wrote :

I'm +1 on this branch since it makes things correct and consistent.

But I think the final rendered string is a bit large. We might consider cutting out the days and just use the number of hours, to be more concise. Example:

https://usercontent.irccloud-cdn.com/file/ESWpb82l/getting-a-bit-too-big.png

review: Approve

« Back to merge proposal