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 initializer to be an instance initializer #587

Closed
jnhuynh opened this issue Jul 22, 2015 · 2 comments
Closed

Update initializer to be an instance initializer #587

jnhuynh opened this issue Jul 22, 2015 · 2 comments

Comments

@jnhuynh
Copy link

jnhuynh commented Jul 22, 2015

Ember has deprecated accessing the container instance insider of initializers. Instead there is this notion of instanceInitializers. We should update to mirror that.

We will need to update setup.js to work different or break it up into two separate methods. One that gets called by an initializer that has access to the registry and application and another to be called by instanceInitialzier that has access to the instance.container.

@jcope2013
Copy link
Contributor

#517
#564
#560

@jnhuynh
Copy link
Author

jnhuynh commented Jul 22, 2015

Sorry about that, should have checked more thoroughly.

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

No branches or pull requests

3 participants