-
-
Notifications
You must be signed in to change notification settings - Fork 87
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
Add a database connection to exists:
validation
#183
Comments
This seems like a valid issue, but rather rare one and easily fixable by extending our MenuBuilder tool class. Personally feel like this doesn't deserve us bloating our configuration file. Since it's a tool class and not used internally, I recommend you extend it in your own project and call your class with updated methods inside the tools function in NovaServiceProvider class. |
I was wrong about this, we use it internally to, which complicates the solution a bit, but with relative ease you can still override the vendor class in your project. |
https://github.com/outl1ne/nova-menu-builder/blob/main/config/nova-menu.php#L18 |
@KasparRosin sadly that didn't work for us. I'm not sure why not anymore. We've forked your package and added the changes there to make it work for our situation. |
We are using this package for a websites that is multi tenant using the spatie package. We now have an issue where the validation if a menu exists will fail because it will check the landlord database.
The menu's in this case are stored in the tenant database so we need to overrule the database connection in the validation. We will create a pull request to address this issue.
The text was updated successfully, but these errors were encountered: