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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Override error view more easily #31408

Closed
wants to merge 29 commits into from
Closed

Conversation

@buismaarten
Copy link
Contributor

buismaarten commented Feb 9, 2020

At the moment I have to override the renderHttpException method to use a custom error view. Maybe it's a idea to have something like this that will make it easy to override the new getHttpExceptionView method.

Would love to get some feedback 馃檹

axlon and others added 9 commits Feb 6, 2020
鈥efresh (#31381)

* Allow multiple paths to be passed to migrate fresh and migrate refresh

* Update FreshCommand.php

* Update RefreshCommand.php

Co-authored-by: Taylor Otwell <taylor@laravel.com>
Signed-off-by: Mior Muhammad Zaki <crynobone@gmail.com>
-> was broken for e.g. PhpStorm
@buismaarten buismaarten requested review from taylorotwell and GrahamCampbell and removed request for taylorotwell Feb 9, 2020
@buismaarten buismaarten changed the base branch from 6.x to master Feb 9, 2020
@buismaarten buismaarten changed the base branch from master to 6.x Feb 9, 2020
@buismaarten buismaarten changed the base branch from 6.x to master Feb 9, 2020
@buismaarten buismaarten changed the base branch from master to 6.x Feb 9, 2020
axlon and others added 4 commits Feb 6, 2020
鈥efresh (#31381)

* Allow multiple paths to be passed to migrate fresh and migrate refresh

* Update FreshCommand.php

* Update RefreshCommand.php

Co-authored-by: Taylor Otwell <taylor@laravel.com>
Signed-off-by: Mior Muhammad Zaki <crynobone@gmail.com>
@buismaarten buismaarten changed the base branch from 6.x to master Feb 9, 2020
@buismaarten buismaarten closed this Feb 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

7 participants
You can鈥檛 perform that action at this time.