-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
Added new ErrorController + Preview and enabling there the error render… #12273
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
Comments
Note: add a section about functional tests that mentions the before/after behavior around testing exceptions. |
@yceruto I'm working on this and I'm having lots of problems trying to understand the changes needed in this page: https://symfony.com/doc/current/controller/error_pages.html Before, things worked like this:
Now, how do we do those things?
Thanks! |
If TwigBundle is enabled, same as before, you can create the error pages in:
The difference is that now you can't create non-HTML error pages like
As before, the path prefix is defined when we import the routing config, see recipe changes symfony/recipes#645
No changes.
Yes, it's possible as before, the |
|
I ran into this while upgrading to symfony 5.0. Since the docs haven't been updated yet, it left me very confused until I found this issue. I shall try to open a PR this sunday to hopefully help others who run into the same issue :) |
Hello @dbrekelmans, my previous comment is outdated. I'm sorry, I don't have time to write/fix the documentation, but I can review any PR about it. Note: There is no such a component |
Hi @yceruto, This was more work than I expected, but I managed to do most of it. Would you mind if I open a PR with the changes I have so far? You can preview my changes here: https://github.com/dbrekelmans/symfony-docs/compare/4.4...dbrekelmans:fix_12273 |
Yes, please, submit the changes to this repo, so we can review it, thanks! |
The text was updated successfully, but these errors were encountered: