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

Don't include registrations test page in production #267

Merged
merged 2 commits into from
Sep 18, 2018

Conversation

irisfaraway
Copy link
Member

We have a page at /fo/registrations or /bo/registrations which is used for testing. However we don't want this to be available in production, so this disables the route when the engine is used in a production environment.

We have a page at /fo/registrations or /bo/registrations which is used for testing. However we don't want this to be available in production, so this disables the route when the engine is used in a production environment.
Copy link
Member

@Cruikshanks Cruikshanks left a comment

Choose a reason for hiding this comment

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

Good shout!

@irisfaraway irisfaraway merged commit 53f8823 into master Sep 18, 2018
@irisfaraway irisfaraway deleted the fix/hide-test-page branch September 18, 2018 10:13
@irisfaraway irisfaraway added the bug Something isn't working label Oct 18, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
2 participants