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

Provide a custom service overview for admin #234

Merged
merged 1 commit into from
Feb 4, 2019

Conversation

MKodde
Copy link
Contributor

@MKodde MKodde commented Feb 4, 2019

The admin user now is faced with a potential 504 error after logging on
to the application. As the service overview page is causing much data
to load for the admin user. In production, over 150 SP's are loaded on
this page including all their entity information.

This does not work out! The easy fix for now is simply rendering a
blank page with a small instruction to the admin user, to use the
service switcher and switch to the desired service that way.

For now the simplest of solutions is chosen (render a different template
if you are an admin user. This could and will be improved upon in an
upcoming development effort.

https://www.pivotaltracker.com/story/show/163691688

The admin user now is faced with a potential 504 error after logging on
to the application. As the service overview page is causing much data
to load for the admin user. In production, over 150 SP's are loaded on
this page including all their entity information.

This does not work out! The easy fix for now is simply rendering a
blank page with a small instruction to the admin user, to use the
service switcher and switch to the desired service that way.

For now the simplest of solutions is chosen (render a different template
if you are an admin user. This could and will be improved upon in an
upcoming development effort.

https://www.pivotaltracker.com/story/show/163691688
@MKodde MKodde merged commit de21f43 into develop Feb 4, 2019
@MKodde MKodde deleted the feature/admin-landing-page branch February 4, 2019 13:37
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.

1 participant