Update GOVUK logo to new brand (behind a flag) #1243
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Adds the ability to turn on the new GOV.UK branding in emails we send with the GOV.UK branding1
https://trello.com/c/tqglnJkB/1285-update-email-template-with-rebranded-govuk-logo
Notes for reviewers
Reviewers should try sending an email with the
rebrand
keyword arg set toTrue
andFalse
in the API code (see this branch on API). They should also check the email previews you can see when you change email branding in settings on Admin (see this branch on Admin). Please check the version of utils with these changes works:rebrand
arg, set to False, in API and Admin (should change nothing)rebrand
arg, set to True, in API and Admin (should change to the new GOV.UK brand)Please also read each commit and focus on the logic of the coding changes rather than checking specific rendering differences between email clients or how it is with assistive tech. A lot of work has already been done on those (see comments on the trello card and these visual testing results). You can if you want of course :)
Footnotes
this will also affect emails with a GOV.UK + department logo branding (see https://github.com/alphagov/notifications-manuals/wiki/Email-template-documentation for details ↩