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.

Sign up for GitHub

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

Login redirect with tests #9

Open

marcellamaki wants to merge 4 commits into docker-landing from login-redirect

Collaborator

marcellamaki commented Dec 4, 2018 •

edited

Loading

Issue #3 / Issue #7
Issue #8

Checks to see if a user's email domain is included in the whitelisted domains in the config file
Creates a "redirect" page when login fails

with options to

return to the home page
log in again.
email MB team

Adds basic unit tests to determine validate what the user sees when they are not logged in

@mikefab Please review.

Marcella added 4 commits

November 30, 2018 16:40


Refactor landing page to include new redirect on invalid login

b285c94


Correct to fetch whitelist domain from jwt rather than state


Add basic unit tests for landing page and valid login component

f56f6f7


Correct bug with landing page rendering and remove duplicate jwt toke…

07a9f53

…n checks

marcellamaki requested a review from mikefab

December 4, 2018 19:37

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment