-
-
Notifications
You must be signed in to change notification settings - Fork 263
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
Does not properly load its API inside Laravel Nova #225
Comments
Hey @dm-pf can you share how you've added a link to the Log Viewer? I'm not a Nova user, but I think what you're looking for is this: use Laravel\Nova\Menu\MenuItem;
MenuItem::externalLink('Log Viewer', route('log-viewer.index')) This is taken from Nova v4 docs btw: https://nova.laravel.com/docs/4.0/customization/menus.html#menu-items |
Hi @arukompas - using MenuSection::make('Log Viewer')
->path('/log-viewer')
->icon('lightning-bolt'), Because that way I don't need to "get out" of my Dashboard and I can just load external tools in a dialog window. E.g. Laravel Horizon works fine this way. Even though my use case is a bit particular (I doubt many people are using Nova) - I thought that reporting this would be a good idea because the issue itself could be triggered in other similar situations if not handled properly. Thanks. |
@arukompas Something has been changed i think. Depending where you are in nova the url of the menu item is for correct for example `https://dashboard.test/log-viewer' //NovaServiceProvider.php
MenuItem::link(__('Logs'), '/log-viewer'), |
Hi, thanks for this awesome package ❤️ .
I've tried using it inside my Laravel Nova project - and the way Nova loads external-to-nova URIs is by loading them in a dialog window through an <iframe>. When that happens I get this window:
It seems that because the /log-viewer is loaded through an iframe - it loses track of the correct URI path to load its
api/folders
.The text was updated successfully, but these errors were encountered: