adminarchitect / core

AdminArchitect - Active Admin for Laravel
http://adminarchitect.com
MIT License
211 stars 66 forks source link

add try..catch to delete action #72

Closed zoryamba closed 5 years ago

zoryamba commented 5 years ago

Could you please add try..catch block in delete action of ScaffoldController to handle deleting errors? It could be useful when you want to restrict deleting when model has related models. For instance restrict category deleting if category has some products included. Seems to be a common case.

endihunter commented 5 years ago

@zoryamba your PR is merged now, thanks for contribution!

zoryamba commented 5 years ago

@endihunter could you please update composer package? maybe it could be automated?