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

Added EMS (Elastic Map Service) app logo #914

Merged
merged 4 commits into from
Jun 8, 2018
Merged

Conversation

cchaos
Copy link
Contributor

@cchaos cchaos commented Jun 7, 2018

This is a quicky to get it up for the EMS landing page.

screen shot 2018-06-07 at 17 40 51 pm

screen shot 2018-06-07 at 17 44 01 pm

@cchaos cchaos requested a review from snide June 7, 2018 21:46
Copy link
Contributor

@snide snide left a comment

Choose a reason for hiding this comment

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

Looks like you need to update the snapshot. Like the icon!

@cchaos
Copy link
Contributor Author

cchaos commented Jun 8, 2018

Ugh yeah, it seems the tests don't update when you make a change to the icon only when you add the icon... Unless you clean the cache

@cchaos
Copy link
Contributor Author

cchaos commented Jun 8, 2018

jenkins test this

@cchaos cchaos merged commit 65896e3 into elastic:master Jun 8, 2018
@cchaos cchaos deleted the ems-logo branch June 8, 2018 14:10
@cchaos
Copy link
Contributor Author

cchaos commented Jun 8, 2018

@thomasneirynck This is now in 0.0.52

@thomasneirynck
Copy link

thanks @cchaos 🎆 !

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.

3 participants