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

[9.x] Add Policies to Model Show Command #45153

Merged
merged 3 commits into from Dec 2, 2022

Conversation

ahinkle
Copy link
Contributor

@ahinkle ahinkle commented Dec 1, 2022

This PR adds registered policies to the artisan model:show command. This can be useful to see what policies are registered on a model.

Screen Shot 2022-11-30 at 10 09 29 PM

@ahinkle ahinkle changed the title Add Policies to Model Show Command [9.x] Add Policies to Model Show Command Dec 1, 2022
@ahinkle
Copy link
Contributor Author

ahinkle commented Dec 1, 2022

/cc @jessarcher

@taylorotwell
Copy link
Member

Since it's very atypical I think to have more than one policy per model (correct me if I'm wrong), should this be something we just put at the top ... just below "Table" maybe?

CleanShot 2022-12-01 at 09 47 14@2x

@taylorotwell taylorotwell merged commit a9bf589 into laravel:9.x Dec 2, 2022
@ahinkle
Copy link
Contributor Author

ahinkle commented Dec 2, 2022

Agreed. Thanks for the code change. 👍

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.

None yet

2 participants