Make auto-subscriptions wait until there is a logged in user

[jankapunkt]

This kind of a question / feature request.

Currently the package's client code automatically subscribes to the roles and immediately causes data being loaded.

I would like to defer this step until my users are actually logged in reducing the initial data load when starting the app. I know this is just a tiny performance improvement but many of these small things add up in the end.

Proposed solution:

Tracker.autorun(function () {
  if (!Meteor.userId()) return
  Roles.subscription = Meteor.subscribe('_roles')
})

I assume, that the information about user roles on the client makes no sense until there is not a logged in user (correct me if there is a usecase, that contradicts this assumption). Therefore this should not have any negative impact on current setups.

I could provide a PR for this

[SimonSimCity]

I'd say this is a good candidate for the next major version 😎 #295

[drone1]

Roles.subscription got deleted? I don't see any info about this in the v4 notes. Actually I don't see any v4 notes :) My code waits for Roles.subscription.ready() – please advise! Thank you.

[drone1]

I see this was removed in c46b2e6 and monitoring a subscription is no longer necessary.