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 support for custom error pages #3

Merged
merged 1 commit into from
Jul 4, 2017
Merged

Conversation

36degrees
Copy link
Contributor

This sets up a custom 404 page and a generic error page which is used for all other 4xx and 5xx status codes.

In order to allow for both config and the built site to be uploaded we:

  • add nginx config for error pages which will be included within the location block
  • modify middleman config to build into /deploy/public
  • add the new build directory to .gitignore
  • modify the CloudFoundry manifest to upload the entire /deploy/ directory
  • add a Staticfile configured to serve only /deploy/public and include the custom nginx config

This sets up a custom 404 page and a generic error page which is used for all other 4xx and 5xx status codes.

In order to allow for both config and the built site to be uploaded we:

- add nginx config for error pages which will be included within the location block
- modify middleman config to build into /deploy/public
- add the new build directory to .gitignore
- modify the CloudFoundry manifest to upload the entire /deploy/ directory
- add a Staticfile configured to serve only /deploy/public and include the custom nginx config
Copy link
Contributor

@alex-ju alex-ju left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As much as I could test seems flawless

@36degrees 36degrees merged commit 8060039 into master Jul 4, 2017
@36degrees 36degrees deleted the custom-error-pages branch July 4, 2017 13:06
@36degrees
Copy link
Contributor Author

Thanks for the review!

@hannalaakso
Copy link
Member

Thanks for taking as though this PR Ollie, that was really helpful. To me the logic of it seems sound.

jonty-uk-gov-mirror pushed a commit to uk-gov-mirror/DFE-Digital.claim-design-system that referenced this pull request Apr 11, 2021
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.

3 participants