Add t&cs check to loggedInNavGuard

In loggedInNavGuard check that user has accepted t&cs and send them to
profile page if not.

Checking for `requiredFieldsComplete` on the user profile won't actually
check for t&cs and this is what we really want to check.

Author: ArayB

src/dashboard/cd-dashboard.vue

@@ -81,10 +81,6 @@
     async created() {
       await this.loadProfile();
 
-      if (this.userProfile && this.userProfile.requiredFieldsComplete === false) {
-        this.$router.push(`/dashboard/profile/${this.userProfile.userId}/edit`);
-      }
-
       await this.getUserDojos();
       this.setUserDimension();
     },

src/router/loggedInNavGuard.js

@@ -2,5 +2,19 @@ import UserService from '@/users/service';
 
 export default async function (to, from, next) {
   const loggedInUser = (await UserService.getCurrentUser()).body;
-  next(loggedInUser.login ? undefined : { name: 'Login', query: { referer: to.fullPath } });
+  if (loggedInUser.login === null) {
+    next({ name: 'Login', query: { referer: to.fullPath } });
+    return null;
+  }
+
+  if (
+    loggedInUser.user &&
+    loggedInUser.user.termsConditionsAccepted === false
+  ) {
+    next({ path: `/dashboard/profile/${loggedInUser.user.id}/edit` });
+    return null;
+  }
+
+  next();
+  return null;
 }