Prevent multiple exitTo params from being added to the URL

[tgolen]

Fixes #177

Test

Difficult to test, so we'll just see how it works on production.

[Jag96]

I'm seeing some weird behavior if I'm signed out and I try to navigate to a report. If I'm signed out and I go to http://localhost:8080/#/22, I see the URL bar jumps around a bit and ends up with multiple signIn strings in the URL

[Jag96]

Ok, I think this is what we want here:

if (!url) {
    return;
}

// If we already have exitTo, don't redirect
if (url.indexOf('exitTo') !== -1 || url.indexOf('signin') !== -1) {
    return;
}

// If we're at the root, go to sign-in, otherwise add the exitTo
const urlWithExitTo = url !== '/'
    ? `${ROUTES.SIGNIN}/exitTo${url}`
    : ROUTES.SIGNIN;
return Ion.set(IONKEYS.APP_REDIRECT_TO, urlWithExitTo);

I tested this with the following 3 cases:

1. Sign out of the app and go to http://localhost:8080/#/22, confirm you are taken to http://localhost:8080/#/signin/exitTo/22
2. While signed out, navigate to http://localhost:8080/, confirm you are taken to http://localhost:8080/#/signin
3. While signed out, navigate to http://localhost:8080/#/signin/exitTo/22, confirm the URL doesn't change

[tgolen]

Looks like that works well! Thanks. Committing it now

[tgolen]

Updated. I tested this by adding a timeout on the homepage (in componentDidMount) like this:

        setTimeout(() => {
            redirectToSignIn();
        }, 5000);

Then logged in, went straight to a report. It redirected me to signIn, I signed in, then was back on the same report.