{"action":"created","issue":{"url":"https://api.github.com/repos/ahmadsoe/ember-highcharts/issues/42","labels_url":"https://api.github.com/repos/ahmadsoe/ember-highcharts/issues/42/labels{/name}","comments_url":"https://api.github.com/repos/ahmadsoe/ember-highcharts/issues/42/comments","events_url":"https://api.github.com/repos/ahmadsoe/ember-highcharts/issues/42/events","html_url":"https://github.com/ahmadsoe/ember-highcharts/pull/42","id":111282954,"number":42,"title":"Update initializer to work with ember 2.1.x","user":{"login":"gregjopa","id":534034,"avatar_url":"https://avatars.githubusercontent.com/u/534034?v=3","gravatar_id":"","url":"https://api.github.com/users/gregjopa","html_url":"https://github.com/gregjopa","followers_url":"https://api.github.com/users/gregjopa/followers","following_url":"https://api.github.com/users/gregjopa/following{/other_user}","gists_url":"https://api.github.com/users/gregjopa/gists{/gist_id}","starred_url":"https://api.github.com/users/gregjopa/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gregjopa/subscriptions","organizations_url":"https://api.github.com/users/gregjopa/orgs","repos_url":"https://api.github.com/users/gregjopa/repos","events_url":"https://api.github.com/users/gregjopa/events{/privacy}","received_events_url":"https://api.github.com/users/gregjopa/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2015-10-13T22:15:50Z","updated_at":"2015-10-14T20:55:35Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/ahmadsoe/ember-highcharts/pulls/42","html_url":"https://github.com/ahmadsoe/ember-highcharts/pull/42","diff_url":"https://github.com/ahmadsoe/ember-highcharts/pull/42.diff","patch_url":"https://github.com/ahmadsoe/ember-highcharts/pull/42.patch"},"body":"This change adds support for the new 2.1.x container api. I've tested this change in 2.1.0, 2.0.0, and 1.13.7.\r\n\r\nI'm not sure this is the best way to handle this api change. Let me know if you have any suggestions on how to better do this and I will update the PR.\r\n\r\nThis initializer code was originally added by @Dhaulagiri."},"comment":{"url":"https://api.github.com/repos/ahmadsoe/ember-highcharts/issues/comments/148197576","html_url":"https://github.com/ahmadsoe/ember-highcharts/pull/42#issuecomment-148197576","issue_url":"https://api.github.com/repos/ahmadsoe/ember-highcharts/issues/42","id":148197576,"user":{"login":"ahmadsoe","id":1957737,"avatar_url":"https://avatars.githubusercontent.com/u/1957737?v=3","gravatar_id":"","url":"https://api.github.com/users/ahmadsoe","html_url":"https://github.com/ahmadsoe","followers_url":"https://api.github.com/users/ahmadsoe/followers","following_url":"https://api.github.com/users/ahmadsoe/following{/other_user}","gists_url":"https://api.github.com/users/ahmadsoe/gists{/gist_id}","starred_url":"https://api.github.com/users/ahmadsoe/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ahmadsoe/subscriptions","organizations_url":"https://api.github.com/users/ahmadsoe/orgs","repos_url":"https://api.github.com/users/ahmadsoe/repos","events_url":"https://api.github.com/users/ahmadsoe/events{/privacy}","received_events_url":"https://api.github.com/users/ahmadsoe/received_events","type":"User","site_admin":false},"created_at":"2015-10-14T20:55:35Z","updated_at":"2015-10-14T20:55:35Z","body":"I can't think another way how to load default chart option. `ember-simple-auth`,  `ember-cli-mirage` or `ember-pop-over` inject their configs into the container. \r\n\r\nHave you tried to change the lookup method here https://github.com/ahmadsoe/ember-highcharts/blob/master/addon/utils/option-loader.js#L6\r\nbecause the default chart config is just a simple function, not an Ember class, it requires `container.lookupFactory` instead of `container.lookup`."}}