Skip to content

Create add:table command #7

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

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Create add:table command #7

wants to merge 7 commits into from

Conversation

lpheller
Copy link
Member

@lpheller lpheller commented Feb 2, 2022

#1

@lpheller lpheller requested a review from cbl February 2, 2022 08:11
@lpheller
Copy link
Member Author

lpheller commented Feb 2, 2022

Actually, shouldn't these go directly into a components folder?

@cbl
Copy link
Contributor

cbl commented Feb 2, 2022

@lpheller

The table needs to be published in the corresponding page. So a UserTable for the Page Users must be published in resources/admin/js/Pages/Users/components/UserTable.vue.

Maybe the command should take 2 arguments, the page and the Tablename:

php artisan add:table Users UserTable

@lpheller
Copy link
Member Author

lpheller commented Feb 8, 2022

Updated to take 2 arguments into account. However, i made the second one optional. So you may use either of the following options to create the corresponding Vue components in Pages\Users\components:

php artisan add:table Users UserTable
// or simply
php artisan add:table Users

Providing the table name however enables to create differently named table components:

php artisan add:table Users OtherTable

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants