feat(config): auto register pubsub storage configurable

[stephenlautier]

Features

config: configurable auto register pubsub

Mainly this was added as I had a teething issue due to it, basically, storage grain such as redis/dynamodb weren't working due to this, and the only option had to move UseSignalR further down the chain otherwise it throws a weird exception.

To be honest, I think the auto-register PubSub might not be a good idea since then configuring PubSub afterwards seems to be causing issues

[stephenlautier]

@galvesribeiro any updates on this?

[galvesribeiro]

Sorry, missed this.

So, the point on that is to only register the pubsub store if none was provided.

I've recently working on a stream provider which fall on the same situation...

We decided to not register it at all and let the user do that for by thenselves and throw a meanful exception in case one wasn't registered.

Add that property actually goes against what DI would stands for by making a dependency become a configuration option...

What do you think?

IMHO, not registering anything and explicitly throw is better if we mention on the docs the user has to explicitly register a PubSubStore...

Thoughts?

[stephenlautier]

Basically, that's what I wanted to do, to not register anything and let the consumer do it. The only reason I did it this way is so its not breaking, but if you're ok with making it breaking its good for me.

What I'm not sure is how you would throw if its not registered, as in how would you check?
Feel free to close this PR and do that as it might be faster to do so.

[galvesribeiro]

Ok cool, check #88 please.

Thanks for the PR!