Tweak return type of accounts-model load

Unnecessary ?

Author: RoyEJohnson

src/app/models/accounts-model.ts

@@ -10,7 +10,7 @@ function cached<T>(fn: () => T) {
             cachedResult = fn();
             valid = true;
         }
-        return cachedResult;
+        return cachedResult as T;
     };
 
     cachedFn.invalidate = () => {

src/app/models/usermodel.ts

@@ -111,11 +111,11 @@ const userModel = {
 };
 
 const throttledLoginCheck = throttle((setData) => {
-    accountsModel.load()?.then((oldUser) => {
+    accountsModel.load().then((oldUser) => {
         accountsModel.load.invalidate();
-        accountsModel.load()?.then((newUser) => {
+        accountsModel.load().then((newUser) => {
             if (!isEqual(oldUser, newUser)) {
-                userModel.load()?.then(setData);
+                userModel.load().then(setData);
             }
         });
     });
@@ -128,7 +128,7 @@ function useUserModel() {
         const check = () => throttledLoginCheck(setData);
 
         window.addEventListener('focus', check);
-        userModel.load()?.then(setData);
+        userModel.load().then(setData);
 
         return () => window.removeEventListener('focus', check);
     }, []);

src/app/pages/details/common/track-link.ts

@@ -4,7 +4,7 @@ import type {TrackedMouseEvent} from '~/components/shell/router-helpers/use-link
 
 let userInfo: Partial<UserModelType>;
 
-userModel.load()?.then((i) => {
+userModel.load().then((i) => {
     userInfo = i;
 });