UPDATE: configurable auth provider

[vkislichenko]

I'm sure there is a valid reasoning behind current logic of extending User model with shopify shop trait,
But I really do need them to be separate.

This PR provides one more shopify app config option shop_auth_provider ("users" by default)
Model is then taken from config auth.providers.{$config['shop_auth_provider']}.model

Works if manual migrations are used and user table is replaced with custom one
Models provided by this package will still reference custom model using user_id columns, but it seems like a minor detail related to bigger refactoring.

[gnikyt]

Sounds good to me! Please see the failed lint, adjust, and I'll merge :)

[vkislichenko]

@lucasmichot thanks for the fix,
I was offline on vacation this whole time :)

@osiset if there's anything else wrong with current PR - please let me know.

[vkislichenko]

UPDATE: added config option shop_auth_guard
It's mostly to prevent use of custom default auth guards (like jwt), but it might be also useful in other cases when shop_auth_provider specifies a custom model

[gnikyt]

Couple small issues to correct linting, then can be merged.

[gnikyt]

Checks passed, linting pass. Merging. Thanks @vkislichenko !

[salahy]

What about the migrations ? they still change the user model

[vkislichenko]

@salahy

What about the migrations ? they still change the user model

there's "Manual migrations" in config
and command php artisan vendor:publish --tag=shopify-migrations to push migrations to your app's folder so you're free to modify before migrating.

[nahid]

When this changes will be released? @osiset