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

[CLEANUP canary] Remove Ember.Select #13144

Merged
merged 1 commit into from Mar 21, 2016

Conversation

HeroicEric
Copy link
Sponsor Member

As requested in #13127:

This removes Ember.Select.

Views were removed from the 2.0 API unless using the ember-legacy-views addon, which is intended to end support after Ember v2.4. This includes Ember.select.

http://emberjs.com/deprecations/v1.x/#toc_ember-view

@HeroicEric HeroicEric changed the title [CLEANUP beta] Remove Ember.Select 💀 [CLEANUP beta] Remove Ember.Select Mar 21, 2016
@rwjblue
Copy link
Member

rwjblue commented Mar 21, 2016

@HeroicEric next round is on me ❤️

@mixonic
Copy link
Sponsor Member

mixonic commented Mar 21, 2016

Can we make this Canary? I don't see much of a reason for dropping legacy helpers in 2.5-beta. It is already a beta release, we missed the boat, no need to rush this.

As requested in emberjs#13127:

This removes `Ember.Select`.

Views were removed from the 2.0 API unless using the ember-legacy-views
addon, which is intended to end support after Ember v2.4. This includes
`Ember.select`.

http://emberjs.com/deprecations/v1.x/#toc_ember-view
@rwjblue
Copy link
Member

rwjblue commented Mar 21, 2016

Agreed. Canary seems good.

@jonnii
Copy link

jonnii commented Mar 21, 2016

good night sweet prince

@HeroicEric HeroicEric changed the title [CLEANUP beta] Remove Ember.Select [CLEANUP canary] Remove Ember.Select Mar 21, 2016
@HeroicEric
Copy link
Sponsor Member Author

Updated the commit message for canary

@mixonic
Copy link
Sponsor Member

mixonic commented Mar 21, 2016

Thanks @HeroicEric 🔥 🔥 🔥 🔥 🔥 🔥 🔥

@mixonic
Copy link
Sponsor Member

mixonic commented Mar 21, 2016

I'm merging this after some discussion by a subset of core. We continue to make progress on landing the glimmer refactor into Ember master (see epic burndown). It is not expected that this work makes into into canary in time for Ember 2.6, however good progress is being made.

In several recent blog posts and issues core has raised that Ember 2.4 will likely be the final release supporting the legacy view and controller APIs. It looks like that may actually be 2.5, but as 2.4 is an LTS support will live on in 2.4 for far longer than 2.5 regardless.

So that landing the glimmer refactor work is not a complete rush, we're opening the current canary (2.6) to removals of APIs incompatible with the glimmer refactor. Additionally this will be an opportunity to cull a good amount of code from Ember that has only existed to support the legacy addons via intimate feature flags. These are already deprecated in 2.4 per the LTS guidelines.

Please see the deprecation guide for further details about migrating away from Ember's select solution, or try out the fantastic ember-power-select.

mixonic added a commit that referenced this pull request Mar 21, 2016
[CLEANUP canary] Remove Ember.Select
@mixonic mixonic merged commit bab4447 into emberjs:master Mar 21, 2016
@HeroicEric HeroicEric deleted the remove-view-select branch March 21, 2016 03:06
@wycats
Copy link
Member

wycats commented Mar 21, 2016

@mixonic I would probably be more circumspect about exact versions 😉

The high order bit is we want to ship Glimmer 2 as soon as we can, but no sooner. In the meantime, we'll have the six-week release cycle to keep us warm at night.

(I should add that what you said sounds plausible, but by no means guaranteed. Promising exact dates, even hedged, is a good way to feel pressured into shipping too early.)

andrewle added a commit to andrewle/ember-engines that referenced this pull request Apr 15, 2016
`Ember.Select` was deprecated and then removed as of
emberjs/ember.js#13144. This removes ember-engine's reference to
`Ember.Select` to bring engines up to compatibility with Ember 2.4+.
andrewle added a commit to andrewle/ember-engines that referenced this pull request Apr 15, 2016
`Ember.Select` was deprecated and then removed as of
emberjs/ember.js#13144. This removes ember-engine's reference to
`Ember.Select`.
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.

None yet

5 participants