refactor: update logging to use optional chaining for safety in IterableInAppManager and IterableInboxDataModel, otherwise expo gets upset

Author: lposen

src/inApp/classes/IterableInAppManager.ts

@@ -36,7 +36,7 @@ export class IterableInAppManager {
    * @returns A Promise that resolves to an array of in-app messages.
    */
   getMessages(): Promise<IterableInAppMessage[]> {
-    Iterable.logger.log('InAppManager.getMessages');
+    Iterable?.logger?.log('InAppManager.getMessages');
 
     return RNIterableAPI.getInAppMessages();
   }
@@ -59,7 +59,7 @@ export class IterableInAppManager {
    * @returns A Promise that resolves to an array of messages marked as `saveToInbox`.
    */
   getInboxMessages(): Promise<IterableInAppMessage[]> {
-    Iterable.logger.log('InAppManager.getInboxMessages');
+    Iterable?.logger?.log('InAppManager.getInboxMessages');
 
     return RNIterableAPI.getInboxMessages();
   }
@@ -86,7 +86,7 @@ export class IterableInAppManager {
     message: IterableInAppMessage,
     consume: boolean
   ): Promise<string | undefined> {
-    Iterable.logger.log('InAppManager.show');
+    Iterable?.logger?.log('InAppManager.show');
 
     return RNIterableAPI.showMessage(message.messageId, consume);
   }
@@ -114,7 +114,7 @@ export class IterableInAppManager {
     location: IterableInAppLocation,
     source: IterableInAppDeleteSource
   ): void {
-    Iterable.logger.log('InAppManager.remove');
+    Iterable?.logger?.log('InAppManager.remove');
 
     return RNIterableAPI.removeMessage(message.messageId, location, source);
   }
@@ -131,7 +131,7 @@ export class IterableInAppManager {
    * ```
    */
   setReadForMessage(message: IterableInAppMessage, read: boolean) {
-    Iterable.logger.log('InAppManager.setRead');
+    Iterable?.logger?.log('InAppManager.setRead');
 
     RNIterableAPI.setReadForMessage(message.messageId, read);
   }
@@ -151,7 +151,7 @@ export class IterableInAppManager {
   getHtmlContentForMessage(
     message: IterableInAppMessage
   ): Promise<IterableHtmlInAppContent> {
-    Iterable.logger.log('InAppManager.getHtmlContentForMessage');
+    Iterable?.logger?.log('InAppManager.getHtmlContentForMessage');
 
     return RNIterableAPI.getHtmlInAppContentForMessage(message.messageId);
   }
@@ -171,7 +171,7 @@ export class IterableInAppManager {
    * ```
    */
   setAutoDisplayPaused(paused: boolean) {
-    Iterable.logger.log('InAppManager.setAutoDisplayPaused');
+    Iterable?.logger?.log('InAppManager.setAutoDisplayPaused');
 
     RNIterableAPI.setAutoDisplayPaused(paused);
   }

src/inbox/classes/IterableInboxDataModel.ts

@@ -97,7 +97,7 @@ export class IterableInboxDataModel {
    * @returns  A promise that resolves to the HTML content of the specified message.
    */
   getHtmlContentForMessageId(id: string): Promise<IterableHtmlInAppContent> {
-    Iterable.logger.log(
+    Iterable?.logger?.log(
       'IterableInboxDataModel.getHtmlContentForItem messageId: ' + id
     );
 
@@ -114,7 +114,7 @@ export class IterableInboxDataModel {
    * @param id - The unique identifier of the message to be marked as read.
    */
   setMessageAsRead(id: string) {
-    Iterable.logger.log('IterableInboxDataModel.setMessageAsRead');
+    Iterable?.logger?.log('IterableInboxDataModel.setMessageAsRead');
 
     RNIterableAPI.setReadForMessage(id, true);
   }
@@ -126,7 +126,7 @@ export class IterableInboxDataModel {
    * @param deleteSource - The source from which the delete action is initiated.
    */
   deleteItemById(id: string, deleteSource: IterableInAppDeleteSource) {
-    Iterable.logger.log('IterableInboxDataModel.deleteItemById');
+    Iterable?.logger?.log('IterableInboxDataModel.deleteItemById');
 
     RNIterableAPI.removeMessage(id, IterableInAppLocation.inbox, deleteSource);
   }