Skip to content
This repository has been archived by the owner on Mar 31, 2023. It is now read-only.

Fix Emulsify Icons #259

Merged
merged 5 commits into from
Jun 22, 2018
Merged

Fix Emulsify Icons #259

merged 5 commits into from
Jun 22, 2018

Conversation

ModulesUnraveled
Copy link
Contributor

@ModulesUnraveled ModulesUnraveled commented Jun 22, 2018

This PR updates the lock files so that yarn users are getting the latest requirements.

Steps to review:

  • Clone this repo git@github.com:fourkitchens/emulsify.git
  • Checkout this branch git checkout icons-fix
  • composer install
  • yarn install
  • yarn start
  • Visit the icons pattern, and verify it shows the bars

@shaal
Copy link
Contributor

shaal commented Jun 22, 2018

icons still not working, only after running yarn upgrade

@ModulesUnraveled
Copy link
Contributor Author

I wonder if you pulled this down before I pushed the updated yarn.lock file. Because pulling this now, and running yarn upgrade doesn't change anything for me

@ModulesUnraveled
Copy link
Contributor Author

@shaal This should be ready for re-review. Would you mind?

Copy link
Contributor

@evanmwillhite evanmwillhite left a comment

Choose a reason for hiding this comment

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

Worked for me. Ship it and let's do a release

@shaal
Copy link
Contributor

shaal commented Jun 22, 2018 via email

@ModulesUnraveled ModulesUnraveled merged commit b7630ce into develop Jun 22, 2018
@ModulesUnraveled ModulesUnraveled deleted the icons-fix branch June 22, 2018 21:27
@shaal
Copy link
Contributor

shaal commented Jun 22, 2018

It works great! thank you

@ModulesUnraveled
Copy link
Contributor Author

Awesome! We just tagged a new release v2.6. Thanks for reporting, and helping debug!

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

Successfully merging this pull request may close these issues.

None yet

3 participants