Skip to content
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 vendor publish as well as register make:fm-model command #48

Merged
merged 7 commits into from
Oct 11, 2024

Conversation

likeadeckofcards
Copy link
Member

This PR creates 2 option vendor:publish options. The default will publish a new model in the stubs folder that will be used by the make:fm-model command. The second option will publish the fm.model to override the default model.stub so you can use make:model intended if you plan to only use this project with FileMaker backed models. Also, created a new command make:fm-model that will create model similar to Laravel's native make:model command. The readme file has been updated to include these new features.

@Smef
Copy link
Member

Smef commented Oct 11, 2024

I like the updated artisan command, but I don't think we need to do the stub stuff in here. It's part of the native Laravel feature set, isn't it? Users could update the stub on their own if they wanted to. I'm not sure what it's adding to have in here.

@Smef Smef merged commit a5401d6 into 2.x Oct 11, 2024
1 check passed
@Smef Smef deleted the feature/add-model-stub branch October 11, 2024 18:31
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