Fix the signin process to use a route by default

Westmoor.DowntimePlanner/ClientApp/src/app/app.module.ts

@@ -108,6 +108,7 @@ import { SignInGuard } from './sign-in-guard.service';
         canActivate: [AuthGuard],
         data: {role: 'Admin'}
       },
+      {path: 'signin', component: HomeComponent, pathMatch: 'full', canActivate: [SignInGuard]},
       {path: 'signout', component: HomeComponent, pathMatch: 'full', canActivate: [SignOutGuard]},
       {path: 'oidc-callback', component: HomeComponent, pathMatch: 'full', canActivate: [OidcCallbackGuard]},
       {path: 'terms-of-service', component: TermsOfServiceComponent, pathMatch: 'full'}

Westmoor.DowntimePlanner/ClientApp/src/app/home/home.component.html

@@ -115,7 +115,7 @@ <h2>Completed downtimes</h2>
 <ng-template #guestHeader>
   <h1 class="mb-4">Welcome guest!</h1>
 
-  <button class="btn btn-primary btn-lg mb-4" routerLink="/sign-in">Sign in</button>
+  <button class="btn btn-primary btn-lg mb-4" [routerLink]="['/signin']">Sign in</button>
 
   <p>
     The Westmoor Downtime Planner is an app design to simplify your D&D downtime experience.

Westmoor.DowntimePlanner/ClientApp/src/app/nav-menu/nav-menu.component.html

@@ -43,7 +43,7 @@
           </ng-container>
           <ng-template #signedOut>
             <li class="nav-item">
-              <a class="nav-link text-dark" href="#" (click)="signIn()">Sign in</a>
+              <a class="nav-link text-dark" [routerLink]="['/signin']">Sign in</a>
             </li>
           </ng-template>
         </ul>

Westmoor.DowntimePlanner/ClientApp/src/app/nav-menu/nav-menu.component.ts

@@ -19,8 +19,4 @@ export class NavMenuComponent {
   toggle() {
     this.isExpanded = !this.isExpanded;
   }
-
-  signIn() {
-    this.auth.signIn().subscribe();
-  }
 }

Westmoor.DowntimePlanner/ClientApp/src/app/sign-in-guard.service.ts

@@ -0,0 +1,22 @@
+import { Injectable } from '@angular/core';
+import { ActivatedRouteSnapshot, CanActivate, RouterStateSnapshot } from '@angular/router';
+import { AuthService } from './auth.service';
+import { switchMap } from 'rxjs/operators';
+import { NEVER } from 'rxjs';
+
+@Injectable({
+  providedIn: 'root'
+})
+export class SignInGuard implements CanActivate {
+  constructor(
+    private auth: AuthService
+  ) {
+  }
+
+  canActivate(next: ActivatedRouteSnapshot, state: RouterStateSnapshot) {
+    return this.auth.signIn()
+      .pipe(
+        switchMap(() => NEVER)
+      );
+  }
+}