Use this.import instead of app.import

Cropster / ember-l10n

A GNU gettext based localization workflow for Ember

MIT License

15 stars 7 forks source link

Use this.import instead of app.import #5

Closed mydea closed 8 years ago

mydea commented 8 years ago

After upgrading to ember-cli 2.7, the app.import fix stopped working. So i migrated to use this.import, which should work now with an arbitrary depth of nested addons.

arm1n commented 8 years ago

Hey Francesco, I would like to merge your pull request, but the tests seem to fail? Can you please have a look into, thanks!

mydea commented 8 years ago

Apparently, ember-cli@2.7 (or https://github.com/ember-cli/ember-cli-import-polyfill) is necessary for this to work. I'm not quite sure how this affects the host apps - if it affects them too, we could try to use var import = this.import || app.import; I guess?