-
-
Notifications
You must be signed in to change notification settings - Fork 46
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 new command to generate board page for a specific Filament resource #62
Conversation
Thanks for making this PR. However:
Once you fix these issues I can begin to pull your PR and test it and review it. |
Hi @mokhosh Thanks for your feedback. I'll fix those issues. Can you just explain what you mean by unrelated changes ? Thanks |
@@ -30,7 +30,7 @@ php artisan filament-kanban:install | |||
|
|||
## Before You Start | |||
|
|||
> [!IMPORTANT] |
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.
This line for example doesn't have to change for this PR.
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.
This line for example doesn't have to change for this PR.
Oh, I understand now. However, it's my code editor that automatically trims additional spaces. If that's a problem I'll try disabling it for the MD file edition.
@@ -270,7 +291,7 @@ Are you a visual learner? I have created some Youtube videos to get you started | |||
> [!WARNING] | |||
> These videos are recorded with version 1.x of the package. | |||
> It is now much simpler to use the package, and requires much less code from you. | |||
> | |||
> |
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.
Or this one.
@@ -2,16 +2,17 @@ | |||
|
|||
namespace Mokhosh\FilamentKanban; | |||
|
|||
use Filament\Support\Assets\Asset; |
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.
Or the order of imports here.
Filament\Resources\Pages\Page
instead ofFilament\Pages\Page