Prefer/Must use Facade notice in documentation about laravel-firebase HOT 3 CLOSED

Thank you for letting me know, the documentation really is not clear about that (the SDK docs only refer to the SDK, not its integration).

Until I figure out better documentation on the main package, would perhaps a note in the Laravel package's README help? I was thinking about something like this in the Usage section:

You don't need and should not use the new Factory() pattern described in the SDK documentation, this is already done through the Service Provider of this package. Always use Dependency Injection, the Facades or the app() helper, otherwise the package configuration has no effect.

What do you think?

from laravel-firebase.

Thank you for letting me know, the documentation really is not clear about that (the SDK docs only refer to the SDK, not its integration).

Until I figure out better documentation on the main package, would perhaps a note in the Laravel package's README help? I was thinking about something like this in the Usage section:

You don't need and should not use the new Factory() pattern described in the SDK documentation, this is already done through the Service Provider of this package. Always use Dependency Injection, the Facades or the app() helper, otherwise the package configuration has no effect.

What do you think?

Brilliant! That makes more sense to devs. Thanks. 👍

from laravel-firebase.

Added with bd90a20, thanks again! 🌺

from laravel-firebase.