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 ember-modifier to add support for 3.22 modifier capabilities #494

Merged
merged 2 commits into from
Nov 19, 2021
Merged

Update ember-modifier to add support for 3.22 modifier capabilities #494

merged 2 commits into from
Nov 19, 2021

Conversation

SergeAstapov
Copy link
Contributor

This fixes deprecations in recent version of Ember and add support for Ember v4

@lukemelia
Copy link
Collaborator

@SergeAstapov do you mind rebasing when you have a chance?

@SergeAstapov
Copy link
Contributor Author

@lukemelia rebased! However 3.12 tests are failing (which is weird enough considering 3.8 pass). Will look into root cause

@SergeAstapov
Copy link
Contributor Author

SergeAstapov commented Nov 17, 2021

@lukemelia had to install ember-modifier-manager-polyfill to make CI happy for 3.12
Appeared there was a typo in Ember introduced in 3.8 and fixed in 3.14

Some details can be seen in emberjs/ember-render-modifiers#14

@lukemelia I think this may be included in either v6 or v7.
If it does not make sense to release it as v6, then we simply can drop support for Ember < 3.13, remove scenarios from ember-try config and get rid of ember-modifier-manager-polyfill altogether

@lukemelia lukemelia merged commit a11d47e into adopted-ember-addons:master Nov 19, 2021
@SergeAstapov SergeAstapov deleted the update-ember-modifier branch November 19, 2021 02:34
leoeuclids pushed a commit to leoeuclids/ember-keyboard that referenced this pull request Dec 11, 2023
This moves the `ember-a11y-testing` config into the dummy app's environment file. This prevents the config from being merged with the consuming app's config.
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