Code review comment for lp:~jcsackett/launchpad/unknown-blueprints-service-597738

Revision history for this message
Guilherme Salgado (salgado) wrote :

Hi Jon,

This looks good, and I only have a couple suggestions.

s/use// @ "[...] does not track use feature planning or documentation."

And for bonus points you could change the 'Configure blueprints' text according to the state the context is in. For example, in the UNKNOWN state, it could be changed to "Tell us how <project> tracks feature planning and documentation". Similarly, for the EXTERNAL (and maybe NOT_APPLICABLE as well?) state, it could be "Use Launchpad to track feature planning and documentation", although that's a bit of a lie as there are other things you can do by following that link (but then it should be a worthy tradeoff if our goal is to get people to use blueprints).

review: Approve (ui*)

« Back to merge proposal