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

Ember 3.20 #508

Merged
merged 2 commits into from
Aug 12, 2020
Merged

Ember 3.20 #508

merged 2 commits into from
Aug 12, 2020

Conversation

RobbieTheWagner
Copy link
Owner

No description provided.

@RobbieTheWagner RobbieTheWagner merged commit a327950 into master Aug 12, 2020
@RobbieTheWagner RobbieTheWagner deleted the ember-3.20 branch August 12, 2020 01:42
"ember-cli-node-assets": "^0.2.2",
"ember-diff-attrs": "^0.2.2",
"fastboot-transform": "^0.1.3",
"flatpickr": "^4.6.3"
"flatpickr": "4.6.3"
Copy link

Choose a reason for hiding this comment

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

why did you pin the version? seems unrelated to the Ember update, no?

Copy link
Owner Author

Choose a reason for hiding this comment

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

Pinned until I have time to debug why latest makes things fail

Copy link

Choose a reason for hiding this comment

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

I've been using 4.6.5 and it seems to work fine for me 🤔

Copy link
Owner Author

Choose a reason for hiding this comment

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

4.6.6 was causing failures both here and in Ember Inspector. Flatpickr is notorious for releasing a patch version and changing the entire internal structure of the classes, mousedown vs click etc handlers, but only releasing a patch because the external API did not change. When we write tests against it though, they all start failing and we have to figure out what the new thing is.

Copy link

Choose a reason for hiding this comment

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

I see, thanks. might be worth mentioning in the changelog though 😉

Copy link
Owner Author

Choose a reason for hiding this comment

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

Yes, for sure. The hope is to get a full refactor away from mixins in shortly, update flatpickr to latest, and get a new major release out. Apologies for the temporary lack of detail on the change, but stay tuned 😃

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

Successfully merging this pull request may close these issues.

2 participants