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

Remove Mixin, refactor to glimmer component #513

Conversation

LuisAverhoff
Copy link
Contributor

Removed mixin and changed the Ember Flatpickr from a classic component to a glimmer component.

…t to a glimmer component. Used Ember-cli-update to update ember-cli to 3.20.0 and ran all the codemods that it had.
Copy link
Owner

@RobbieTheWagner RobbieTheWagner 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 good so far, thanks so much for the PR! We use yarn on this project, so would you mind removing package-lock.json and running yarn instead please?

package.json Outdated Show resolved Hide resolved
addon/components/ember-flatpickr.js Show resolved Hide resolved
addon/components/ember-flatpickr.hbs Outdated Show resolved Hide resolved
@LuisAverhoff LuisAverhoff force-pushed the refactor/ember-flatpickr-glimmer branch from 39d3f2f to 6accbc4 Compare August 12, 2020 18:30
Copy link
Owner

@RobbieTheWagner RobbieTheWagner left a comment

Choose a reason for hiding this comment

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

Overall, this looks amazing, great work! Just requested a few documentation changes and had some questions.

addon/components/ember-flatpickr.js Outdated Show resolved Hide resolved
addon/components/ember-flatpickr.js Outdated Show resolved Hide resolved
addon/components/ember-flatpickr.js Outdated Show resolved Hide resolved
addon/components/ember-flatpickr.js Outdated Show resolved Hide resolved
addon/components/ember-flatpickr.js Outdated Show resolved Hide resolved
addon/components/ember-flatpickr.js Outdated Show resolved Hide resolved
addon/components/ember-flatpickr.js Outdated Show resolved Hide resolved
tests/dummy/app/templates/docs/usage.md Show resolved Hide resolved
yarn.lock Show resolved Hide resolved
.eslintrc.js Outdated Show resolved Hide resolved
@LuisAverhoff LuisAverhoff force-pushed the refactor/ember-flatpickr-glimmer branch from c4392a2 to 33647cc Compare August 12, 2020 19:46
@LuisAverhoff LuisAverhoff force-pushed the refactor/ember-flatpickr-glimmer branch from 33647cc to 0f0640c Compare August 12, 2020 19:50
@RobbieTheWagner
Copy link
Owner

Thanks so much for putting in the work on this PR!

@RobbieTheWagner RobbieTheWagner changed the title Removed Mixin and refactored to glimmer component. Remove Mixin, refactor to glimmer component Aug 12, 2020
@RobbieTheWagner RobbieTheWagner merged commit a33a32a into RobbieTheWagner:master Aug 12, 2020
@LuisAverhoff LuisAverhoff deleted the refactor/ember-flatpickr-glimmer branch August 12, 2020 20:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants