Skip to content

Deprecate Ember.DeferredMixin and Ember.Deferred.#5215

Merged
rwjblue merged 4 commits intoemberjs:masterfrom
rwjblue:deprecate-deferred
Jul 24, 2014
Merged

Deprecate Ember.DeferredMixin and Ember.Deferred.#5215
rwjblue merged 4 commits intoemberjs:masterfrom
rwjblue:deprecate-deferred

Conversation

@rwjblue
Copy link
Member

@rwjblue rwjblue commented Jul 23, 2014

No description provided.

rwjblue added 3 commits July 22, 2014 06:51
Ember.DeferredMixin will ultimately be deprecated and removed, but this
configuration (that allows test friendly promises) is not related to
Ember.DeferredMixin anyways.

This just moves that code out of the DeferredMixin.
@stefanpenner
Copy link
Member

although i would love to drop this, think we need to deprecate (with existing functionality) for 1 release, before we can cut it entirely. If someone can convince me i am wrong I will be incredibly happy :)

@rwjblue
Copy link
Member Author

rwjblue commented Jul 24, 2014

@stefanpenner - Updated to deprecate app.then for now (should be completely backwards compat at this point). This can be pulled into beta for 1.7.0, and then we can remove it on Canary.

rwjblue added a commit that referenced this pull request Jul 24, 2014
Deprecate Ember.DeferredMixin and Ember.Deferred.
@rwjblue rwjblue merged commit 08a2e4f into emberjs:master Jul 24, 2014
@rwjblue rwjblue deleted the deprecate-deferred branch July 24, 2014 03:06
praxxis pushed a commit to praxxis/ember-model that referenced this pull request Jul 24, 2014
`Ember.Deferred` has been deprecated as of Ember 1.7.0-beta.3 (emberjs/ember.js#5215).
ef4 pushed a commit to ef4/ember-model that referenced this pull request Aug 24, 2015
`Ember.Deferred` has been deprecated as of Ember 1.7.0-beta.3 (emberjs/ember.js#5215).

Conflicts:
	bower.json
	packages/ember-model/tests/model_test.js
ltapia-dev pushed a commit to ltapia-dev/ember-dev that referenced this pull request Jul 19, 2024
`Ember.Deferred` has been deprecated as of Ember 1.7.0-beta.3 (emberjs/ember.js#5215).
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.

2 participants