Initial Upgrade Essentials: Make it work with Filament v4#87
Initial Upgrade Essentials: Make it work with Filament v4#87coolsam726 wants to merge 1 commit intomokhosh:mainfrom
Conversation
coolsam726
commented
Aug 29, 2025
coolsam726
commented
- Changed dependencies, classes and tailwind styling to ensure the basic configuration works on Filament v4
- Removed tailwind config as this is not needed in tailwind v4
- Removed postcss
- Fixed the KanbanBoard class to be compatible with v4
- Changed dependencies, classes and tailwind styling to ensure the basic configuration works on Filament v4 - Removed tailwind config as this is not needed in tailwind v4 - Removed postcss - Fixed the KanbanBoard class to be compatible with v4
|
Damn! Bro moves fast haha |
|
@mokhosh We may need your approval :) |
|
@jalexmelendez Thanks! I have tested the changes and they work fine on Filament v4. I couldn't get the tests to work though. I don't know why. That's why I had left the PR as a draft. I understand there is another similar PR awaiting approval, we may need to coordinate efforts. |
|
Hey there! So, it seems there is something related to the Laravel 10 tests. As Filament4 has a minimum requirement of Laravel v11.28+, I'd say we could remove the tests that are failing, as they are not relevant. |
You're correct. @coolsam726 I created a PR on your PR to remove these lines. https://github.com/coolsam726/filament-kanban/blob/7054529a2d2f6ff967872bf5e95e97573ee6346c/.github/workflows/run-tests.yml#L20C1-L23C23 (I'm not sure if that's the preferred way of contributing to things like this so apologies in advance). |
5 participants
