-
-
Notifications
You must be signed in to change notification settings - Fork 89
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
Add support for Cluster #458
Conversation
so instead of using the plugin configuration, as a work around we can use the config only, and its work as expected but its not customizable per panel. I left the code in the plugin class as comments to get back to it later. |
Do you happen to have a demo repo I can try all this out on? Honestly, I haven't even used the cluster feature so I don't have anything setup to test against. |
ya I was procrastinating creating a reproduce repo to report to filament 😅 but here it is: and I am using this fork: you'll see the error "Call to a member function getPlugin() on null", so you can disable |
Awesome. Thanks. Just didn't want you to think I was ignoring this or didn't care. 😅 |
Is this not ready or a bug in filament? I couldn't get it to work. |
this PR is using the config file only, so its like half way there, a temp solution. reported the bug here: if this fixed then I'll update this PR to use the plugin class based configuration, so it will be customizable per panel. |
Ok. Can you change it back to draft until it's ready? |
hey @awcodes, so it seems (cluster configuration from the panel) is not possible, as Dan explain:
this PR is ready for a cluster configuration from the panel config file. so if you want to merge it or not totally fine :) 🍻 |
So how does this work? Lol. |
😂
what I was hopping was to customize it per panel: CuratorPlugin::make()
->cluster(\App\Filament\Clusters\Cms::class) but got greedy, will have to convince the client its "better" this way lol |
So, how is this different than over ridding the resource which is already possible? |
less files to copy to the app :) |
still draft... ... one moment plz.the issue