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

Add error pages for unhandled exceptions #269

Merged
merged 2 commits into from
Sep 24, 2018
Merged

Conversation

irisfaraway
Copy link
Member

@irisfaraway irisfaraway commented Sep 20, 2018

https://eaflood.atlassian.net/browse/WC-308

If an error occurs, we don't want users to be kicked out to a scary or unclear error page. Instead, we should present a page which uses the same GOV.UK templates, explains an issue has occurred and states what their next steps are.

https://eaflood.atlassian.net/browse/WC-308

If an error occurs, we don't want users to be kicked out to a scary or unclear error page. Instead, we should present a page which uses the same GOV.UK templates, explains an issue has occured and states what their next steps are.
@irisfaraway irisfaraway self-assigned this Sep 20, 2018
The ErrorsController serves a specific page for the error if one exists, or a generic one if there is no match.

This is based on the implementation from FRAE (https://github.com/DEFRA/flood-risk-engine/)
@irisfaraway irisfaraway merged commit 9ed1a21 into master Sep 24, 2018
@irisfaraway irisfaraway deleted the feature/error-pages branch September 24, 2018 09:06
@irisfaraway irisfaraway added the enhancement New feature or request label Oct 18, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
2 participants