GetPlanUrl::__invoke Argument #3 ($host) must be of type string, null given about laravel-shopify HOT 6 CLOSED

Hi @jasontxf

If you are using the blade templates, then yes you need to include the host for every request as you can't store it in the same way that an SPA does. As your page loads multiple times it means you need to always pass it for every page load so you can load in appbridge every time.

from laravel-shopify.

Create a query param called host and use base64_encode(Auth::user()->name) append it on the link. I had the same issue and this was my fix.

from laravel-shopify.

@karenirenecano Thanks for replying! Which file do I append the host exactly?

from laravel-shopify.

For my case it's on the pricing page. I append it on the url like this billing/1?host=the_base_encoded_shop_name

from laravel-shopify.

ah! I see what you are saying now! I'll give it a try! Thanks alot for the suggestion!

Will update here again after trying.

from laravel-shopify.

Hello,

Here's an updates, your suggestion works very well! Thanks alot!

I created a helper function to include host in every routes to ensure host parameter is included.

function sroute($path, $params) {
    return route($path, [...$params, 'host' => base64_encode(Auth::user()->name)]);
}

That way I dont need to change my current implementation

sroute('billing', ['plan' => 2, 'shop' => Auth::user()->name]) }}"

Though @Kyon147 are we supposed to manually include host in all our links?
Thanks alot for all your time!

from laravel-shopify.