DEV: Use Ember.Service over manually registering a factory. (PR #9634)

Follow up to 9e827eb420c7aefd166fba1e7724eced81aaffe7

GitHub

Is there a reason we can’t add service to ALL_TARGETS?

We could but I figured I would just avoid injecting everything everywhere when we don’t have to.

Thank you for reviewing @davidtaylorhq and bringing the use of a service up :slight_smile:

Oh yeah that’s much shorter :slight_smile:

Thanks for doing this. I think it’s a great improvement.

import { inject as service } from "@ember/service";

Just a small proposed change regarding service injection imports. :slightly_smiling_face: (inject => service)

  presenceManager: service(),
import { inject as service } from "@ember/service";
  presenceManager: service(),