Fixed stric issues for ErrorHandlingUtils and AccessibleElement

src/Common/ErrorHandlingUtils.ts

@@ -28,10 +28,10 @@ export const getErrorMessage = (error: string | Error = ""): string => {
 };
 
 export const getErrorStack = (error: string | Error): string => {
-  return typeof error === "string" ? undefined : error.stack;
+  return typeof error === "string" ? "" : error.stack === undefined ? "" : error.stack;
 };
 
-const sendNotificationForError = (errorMessage: string, errorCode: number | string): void => {
+const sendNotificationForError = (errorMessage: string, errorCode: number | string | undefined): void => {
   if (errorCode === HttpStatusCodes.Forbidden) {
     if (errorMessage?.toLowerCase().indexOf("sharedoffer is disabled for your account") > 0) {
       return;

src/Explorer/Controls/AccessibleElement/AccessibleElement.tsx

@@ -2,9 +2,9 @@ import * as React from "react";
 import * as Constants from "../../../Common/Constants";
 
 interface AccessibleElementProps extends React.HtmlHTMLAttributes<HTMLElement> {
-  as: string; // tag element name
+  as?: string; // tag element name
-  onActivated: (event: React.SyntheticEvent<HTMLElement>) => void;
+  onActivated?: (event: React.SyntheticEvent<HTMLElement>) => void;
-  "aria-label": string;
+  "aria-label"?: string;
   tabIndex?: number;
 }
 
@@ -16,7 +16,9 @@ export class AccessibleElement extends React.Component<AccessibleElementProps> {
     if (event.charCode === Constants.KeyCodes.Space || event.charCode === Constants.KeyCodes.Enter) {
       event.stopPropagation();
       event.preventDefault();
-      this.props.onActivated(event);
+      if (this.props.onActivated !== undefined) {
+        this.props.onActivated(event);
+      }
     }
   };
 
@@ -27,11 +29,15 @@ export class AccessibleElement extends React.Component<AccessibleElementProps> {
 
     const tabIndex = this.props.tabIndex === undefined ? 0 : this.props.tabIndex;
 
-    return React.createElement(this.props.as, {
+    return this.props.as !== undefined ? (
-      ...elementProps,
+      React.createElement(this.props.as, {
-      onKeyPress: this.onKeyPress,
+        ...elementProps,
-      onClick: this.props.onActivated,
+        onKeyPress: this.onKeyPress,
-      tabIndex,
+        onClick: this.props.onActivated,
-    });
+        tabIndex,
+      })
+    ) : (
+      <></>
+    );
   }
 }

tsconfig.strict.json

@@ -139,7 +139,9 @@
     "./src/userContext.test.ts",
     "src/Common/EntityValue.tsx",
     "./src/Platform/Hosted/Components/SwitchAccount.tsx",
-    "./src/Platform/Hosted/Components/SwitchSubscription.tsx"
+    "./src/Platform/Hosted/Components/SwitchSubscription.tsx",
+    "./src/Common/ErrorHandlingUtils.ts",
+    "./src/Explorer/Controls/AccessibleElement/AccessibleElement.tsx"
   ],
   "include": [
     "src/CellOutputViewer/transforms/**/*",