Remove automatic injections about ember-useragent HOT 6 CLOSED

with custom macros, that raise a deprecation, when accessed. This allows users to discover and migrate all the callsites accidentally relying on these implicit injections bit by bit

I think we already did this for users in ember-source >= 3.26

emberjs/ember.js#19358

from ember-useragent.

hi! Ping on this.

specifically, ember v4.0 will not work with this pattern.

emberjs/ember.js#19680

from ember-useragent.

It is unquestionable, that these implicit injections will have to be removed eventually due to emberjs/ember.js#19680. Thus I recommend that anyone wanting to or currently using this addon, overrides https://github.com/willviles/ember-useragent/blob/master/app/initializers/user-agent.js in their own app with a no-op, like:

// app/initializers/user-agent.js
export function initialize() {}

This way the implicit injections are already disabled and you can't accidentally make use of them. Please note, that if you're making this change in an app that already uses ember-useragent, you should regression test it to ensure, you haven't already used these implicit injections accidentally.

from ember-useragent.

As a way forward for ember-useragent itself, I would recommend to replace the application.inject(...) usages with custom macros, that raise a deprecation, when accessed. This allows users to discover and migrate all the callsites accidentally relying on these implicit injections bit by bit.

Optionally configuration could be included, to completely disable the injection, which should be recommended to be enabled by all new users. This way the breaking release can be delayed indefinitely, all the while being ready for Ember 4.0.

from ember-useragent.

@snewcomer perfect, thanks! I wasn't sure any more and didn't re-read.

from ember-useragent.

Released in 0.11.0. Thanks for all contributions!

from ember-useragent.