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

[5.5] Adds apiResource option to make:controller command #21865

Closed
wants to merge 1 commit into from

Conversation

DanielDarrenJones
Copy link

@DanielDarrenJones DanielDarrenJones commented Oct 29, 2017

This PR adds the --apiResource (-a) option to the php artisan make:controller command, this functionality is very similar to the --resource command however it only created the index, store, show, update and destroy method stubs.

This PR was created to match the Route::apiResource() method which was added in Laravel 5.4 #19347.

@DanielDarrenJones DanielDarrenJones changed the title Adds apiResource option to make:controller command [5.5] Adds apiResource option to make:controller command Oct 29, 2017
@taylorotwell
Copy link
Member

Formatting does not follow the rest of the framework. CamelCased CLI options, etc.

@DanielDarrenJones
Copy link
Author

@taylorotwell just noticed the format should be --api-resource right?

If I fix would you be willing to pull?

What was the etc? Only other thing I wasn't sure about was the short hand -a should this be -ar?

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