filament icon indicating copy to clipboard operation
filament copied to clipboard

feat: support Redirector type in Logout response

Open saade opened this issue 8 months ago • 1 comments

This shouldn't break any existing applications since the docs does not tell users to extend the base class, but instead implement the contract.

saade avatar Apr 25 '25 22:04 saade

Can you please explain the situation that causes this to be returned in a normal Filament app without this class overidden?

danharrin avatar Apr 26 '25 15:04 danharrin

Gonna be real with you — I can’t remember exactly what the issue was 😅 but I do remember that this exception was being thrown, and this fix took care of it.

I haven’t seen it happen again, so I can’t really reproduce it to give you more context.

Either way, the PR is safe (the implementation is the same as the other responses), but totally fine to close it if you feel it’s unnecessary.

Thanks a ton! 🙌

saade avatar May 09 '25 14:05 saade

Thanks

danharrin avatar May 10 '25 14:05 danharrin