-
Couldn't load subscription status.
- Fork 4
Feat / Adding tenant menu switcher #26
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
Feat / Adding tenant menu switcher #26
Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #26 +/- ##
============================================
+ Coverage 72.36% 72.41% +0.05%
- Complexity 303 322 +19
============================================
Files 56 57 +1
Lines 1914 1947 +33
============================================
+ Hits 1385 1410 +25
- Misses 529 537 +8 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tenant switching works, but the frontend link is not being shown. See comments below.
resources/views/filament/components/brand-with-tenant-switcher.blade.php
Outdated
Show resolved
Hide resolved
resources/views/filament/components/brand-with-tenant-switcher.blade.php
Outdated
Show resolved
Hide resolved
|
@SlimDeluxe This is ready for review |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The frontend link issue remains, please fix.
|
@SlimDeluxe I had to register FrontendAdminPanel in core. Ideally this should happen automatically but for some reason its not working. But registering the panel fixed (as you said above) the frontend link not showing issue. Maybe there is a better solution for this. For now its working. |
|
@SlimDeluxe Now it should be fixed could you give it a try |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please check below. Otherwise ok.
|
@SlimDeluxe This is ready for review |
Ticket: EC-204