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

WIP: [CLEANUP beta] Drop legacy helper APIs, some legacy Handlebars usage #11858

Closed
wants to merge 1 commit into from

Conversation

mixonic
Copy link
Sponsor Member

@mixonic mixonic commented Jul 22, 2015

In 2.0 we drop:

  • Bare function helpers (legacy handlebars compat)
  • View helpers (Ember.Handlebars.makeViewHelper, Ember.HTMLBars.makeViewHelper)
  • makeBoundHelper (Ember.Handlebars.makeBoundHelper)
  • Ember.Handlebars.registerHelper
  • Ember.HTMLBars._registerHelper

This PR is very much WIP (my setup is whack), but I wanted to get it out there in case any one else was looking at this part of the 2.0 changes.

This requires and audit of deprecations messages in 1.13 to ensure everything is properly deprecated.

…amespace

Drop:

* Bare function helpers (legacy handlebars compat)
* View helpers
* makeBoundHelper
@rwjblue
Copy link
Member

rwjblue commented Jul 26, 2015

#11900 adds deprecations for all of these older helper styles, in preparation for this PR removing them...

@rwjblue
Copy link
Member

rwjblue commented Aug 2, 2015

Needs work to be mergeable. I'd like to land this before 2.0.0-beta.5 (so that folks have a chance to digest with one last beta before final 2.0.0).

I know that you are fighting with local build issues (I'm sorry), please let me know if you think this is in jeopardy of not being ready in the next day or two and I can try to pick it up...

@mixonic mixonic mentioned this pull request Aug 3, 2015
2 tasks
@mixonic
Copy link
Sponsor Member Author

mixonic commented Aug 3, 2015

replaced by #11965

@mixonic mixonic closed this Aug 3, 2015
@mixonic mixonic deleted the bus-drop-bare-helpers branch August 3, 2015 03:53
Eric-Guo added a commit to Eric-Guo/ember-examples-with-cli that referenced this pull request Aug 7, 2015
Eric-Guo added a commit to Eric-Guo/ember-examples-with-cli that referenced this pull request Aug 7, 2015
Eric-Guo added a commit to Eric-Guo/ember-examples-with-cli that referenced this pull request Aug 9, 2015
Eric-Guo added a commit to Eric-Guo/ember-examples-with-cli that referenced this pull request Aug 11, 2015
Eric-Guo added a commit to Eric-Guo/ember-examples-with-cli that referenced this pull request Aug 13, 2015
Eric-Guo added a commit to Eric-Guo/ember-examples-with-cli that referenced this pull request Aug 14, 2015
Eric-Guo added a commit to Eric-Guo/ember-examples-with-cli that referenced this pull request Aug 21, 2015
Eric-Guo added a commit to Eric-Guo/ember-examples-with-cli that referenced this pull request Aug 23, 2015
Eric-Guo added a commit to Eric-Guo/ember-examples-with-cli that referenced this pull request Sep 12, 2015
Eric-Guo added a commit to Eric-Guo/ember-examples-with-cli that referenced this pull request Sep 12, 2015
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

2 participants