Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update colour for MHCLG #1319

Merged
merged 1 commit into from
May 13, 2019
Merged

Update colour for MHCLG #1319

merged 1 commit into from
May 13, 2019

Conversation

thomasleese
Copy link
Contributor

Closes #1301.

Trello Card

@NickColley NickColley requested a review from dashouse May 7, 2019 12:40
Copy link
Contributor

@36degrees 36degrees left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, @thomasleese 👍

Copy link
Contributor

@NickColley NickColley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is looking great @thomasleese, I think if you add a CHANGELOG entry we'll be good to go.

@thomasleese
Copy link
Contributor Author

@NickColley Thanks, I've updated the CHANGELOG now.

@@ -22,11 +22,11 @@

🔧 Fixes:

- Pull Request Title goes here
- Update colour for MHCLG
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If possible could you leave the template behind so that future people updating the CHANGELOG can use it? I can do this for you if you'd like, lemme know :)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No problem, I've amended the commit and also put in a ## Template header, if that's helpful.

Fixes the brand colour for MHCLG to their correct corporate "green" brand.

Closes #1301.

[Trello Card](https://trello.com/c/wqKT3TpE/621-change-mhclgs-brand-colour)
@thomasleese
Copy link
Contributor Author

It doesn't look like I have permission to merge this.

@bevanloon
Copy link
Contributor

@NickColley are you happy with the PR now? If so, would you do the honours and merge please? Neither Tom nor I have merge permissions

@NickColley NickColley merged commit fd47405 into alphagov:master May 13, 2019
@NickColley
Copy link
Contributor

Thanks both :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

MHLCG brand colour needs to be updated
4 participants