filamentphp / filament

A collection of beautiful full-stack components for Laravel. The perfect starting point for your next app. Using Livewire, Alpine.js and Tailwind CSS.
https://filamentphp.com
MIT License
19.39k stars 2.97k forks source link

Tenant routes without prefix get precedence over app's web routes #14683

Open mrmonat opened 3 weeks ago

mrmonat commented 3 weeks ago

Package

filament/filament

Package Version

v3.2.117

Laravel Version

11.27.2

Livewire Version

No response

PHP Version

PHP 8.3.0

Problem description

When using Multi-Tenancy in Combination with ->path('') (or ->path('/')) and without ->tenantRoutePrefix('xyz') Routes in routes/web.php and other Packages routes (for example /horizon from Laravel Horizon) result in a 404 not found error (or a redirect to /login if not logged in to filament).

// routes/web.php
return [
    Route::get('/test', function () { // Results in 404
        return "OK";
    }
];

Expected behavior

Filament's Routes should be registered after the Routes in web.php and after other Packages Routes.

Visiting /test should work, visiting /horizon should work.

Steps to reproduce

Reproduction repository (issue will be closed if this is not valid)

https://github.com/mrmonat/filament-issue

Relevant log output

No response

Donate 💰 to fund this issue

Fund with Polar