-
-
Notifications
You must be signed in to change notification settings - Fork 97
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
GetPlanUrl::__invoke Argument #3 ($host) must be of type string, null given #10
Comments
Create a query param called host and use |
@karenirenecano Thanks for replying! Which file do I append the host exactly? |
For my case it's on the pricing page. I append it on the url like this |
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. |
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? |
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. |
For bug reporting only! If you're posting a feature request or discussion, please ignore.
Expected Behavior
After changing plan it should redirect back successfully using billing route.
Current Behavior
When redirecting back from the billing route, I encountered this error
Failure Information
I believe this is due to the necessary $host parameter in the new updates. But when i check the url path, host parameter dont seems to exists to begin with.
Steps to Reproduce
Please provide detailed steps for reproducing the issue.
Open app from the sidebar
![SCR-20221213-gxv](https://user-images.githubusercontent.com/1124731/207224822-4e059079-c5c5-4cba-a38e-579c2e3d17f2.png)
Changing pricing plan using billing route
Encounter issue.
Context
Please provide any relevant information about your setup. This is important in case the issue is not reproducible except for under certain conditions.
Failure Logs
Please include any relevant log snippets or files here.
Please let me know how can I further help.
The text was updated successfully, but these errors were encountered: