Code review comment for lp:~sinzui/launchpad/disable-gmaps-0

Revision history for this message
Robert Collins (lifeless) wrote :

439
440 +
441 + <tal:gmap2 condition="view/gmap2_enabled">

Looks like 2 lines of VWS there which isn't really needed - one would be plenty in the template.

You have my ok to merge & CP; ec2 land will bitch about this, so I'm going to not mark this as 'production-change', and request that you self-tag that, to make the automation work.

review: Approve

« Back to merge proposal