chore: created tickets to replace todos

Author: lposen

android/src/main/java/com/iterable/reactnative/RNIterableAPIModule.java

@@ -101,7 +101,7 @@ public void initializeWithApiKey(String apiKey, ReadableMap configReadableMap, S
 
         IterableApi.getInstance().getInAppManager().addListener(this);
 
-        // TODO: Figure out what the error cases are and handle them appropriately
+        // MOB-10421: Figure out what the error cases are and handle them appropriately
         // This is just here to match the TS types and let the JS thread know when we are done initializing
         promise.resolve(true);
     }
@@ -592,7 +592,7 @@ public String onAuthTokenRequested() {
     @Override
     public void onTokenRegistrationSuccessful(String authToken) {
         IterableLogger.v(TAG, "authToken successfully set");
-        //TODO: Pass successhandler to event listener
+        // MOB-10422: Pass successhandler to event listener
         sendEvent(EventName.handleAuthSuccessCalled.name(), null);
     }
 

example/src/hooks/useIterableApp.tsx

@@ -128,7 +128,7 @@ export const IterableAppProvider: FunctionComponent<
         const routeNames = [Route.Commerce, Route.Inbox, Route.User];
         for (const route of routeNames) {
           if (url.includes(route.toLowerCase())) {
-            // TODO: Figure out typing for this
+            // [MOB-10418](https://iterable.atlassian.net/browse/MOB-10418): Figure out typing for this
             // eslint-disable-next-line @typescript-eslint/ban-ts-comment
             // @ts-ignore
             navigation.navigate(route);
@@ -184,7 +184,7 @@ export const IterableAppProvider: FunctionComponent<
         .finally(() => {
           // For some reason, ios is throwing an error on initialize.
           // To temporarily fix this, we're using the finally block to login.
-          // TODO: Find out why initialize is throwing an error on ios
+          // MOB-10419: Find out why initialize is throwing an error on ios
           setIsInitialized(true);
           if (getUserId()) {
             login();

src/core/classes/Iterable.ts

@@ -25,8 +25,8 @@ import { IterableLogger } from './IterableLogger';
 const RNIterableAPI = NativeModules.RNIterableAPI;
 const RNEventEmitter = new NativeEventEmitter(RNIterableAPI);
 
+/* eslint-disable tsdoc/syntax */
 /**
- * @hideconstructor
  * The main class for the Iterable React Native SDK.
  *
  * The majority of the high-level functionality can be accomplished through the
@@ -42,6 +42,7 @@ const RNEventEmitter = new NativeEventEmitter(RNIterableAPI);
  * // OR
  * Iterable.setUserId('myUserId');
  */
+/* eslint-enable tsdoc/syntax */
 export class Iterable {
   /**
    * Manager for in app messages
@@ -225,7 +226,7 @@ export class Iterable {
    * ## Registering device token:
    *
    * If `IterableConfig.autoPushRegisteration` is set to `true`, calling
-   * setUserI`d automatically registers the device for push notifications and
+   * `setUserId` automatically registers the device for push notifications and
    * sends the `deviceId` and token to Iterable.
    *
    * ## Optional JWT token parameter:
@@ -745,8 +746,8 @@ export class Iterable {
   /**
    * tsdoc/syntax needs to be disabled as it conflicts with the mermaid syntax.
    * unfortunately, disabling it inline does not appear to work.
-   * eslint-disable tsdoc/syntax
    */
+  /* eslint-disable tsdoc/syntax */
   /**
    * Handle a universal link.
    *
@@ -819,6 +820,7 @@ export class Iterable {
    *  }
    * ```
    */
+  /* eslint-enable tsdoc/syntax */
   static handleAppLink(link: string): Promise<boolean> {
     Iterable.logger.log('handleAppLink');
 
@@ -941,8 +943,7 @@ export class Iterable {
         IterableEventName.handleInAppCalled,
         (messageDict) => {
           const message = IterableInAppMessage.fromDict(messageDict);
-          // TODO: Check if we can use chain operator (?.) here instead
-
+          // MOB-10423: Check if we can use chain operator (?.) here instead
           const result = Iterable.savedConfig.inAppHandler!(message);
           RNIterableAPI.setInAppShowResponse(result);
         }
@@ -952,7 +953,7 @@ export class Iterable {
     if (Iterable.savedConfig.authHandler) {
       let authResponseCallback: IterableAuthResponseResult;
       RNEventEmitter.addListener(IterableEventName.handleAuthCalled, () => {
-        // TODO: Check if we can use chain operator (?.) here instead
+        // MOB-10423: Check if we can use chain operator (?.) here instead
 
         Iterable.savedConfig.authHandler!()
           .then((promiseResult) => {
@@ -1008,7 +1009,7 @@ export class Iterable {
     }
 
     function callUrlHandler(url: string, context: IterableActionContext) {
-      // TODO: Figure out if this is purposeful
+      // MOB-10424: Figure out if this is purposeful
       // eslint-disable-next-line eqeqeq
       if (Iterable.savedConfig.urlHandler?.(url, context) == false) {
         Linking.canOpenURL(url)
@@ -1027,7 +1028,7 @@ export class Iterable {
   /**
    * Retrieves the version number from the package.json file.
    *
-   * @returns {string} The version number as specified in the package.json file.
+   * @returns The version number as specified in the package.json file.
    *
    * @internal
    */

src/core/hooks/useDeviceOrientation.tsx

@@ -13,6 +13,7 @@ export interface IterableDeviceOrientation {
   isPortrait: boolean;
 }
 
+/* eslint-disable tsdoc/syntax */
 /**
  * Custom hook to get the current device orientation.
  *
@@ -28,14 +29,15 @@ export interface IterableDeviceOrientation {
  * The `useEffect` hook only includes `width` in its dependency array. This is because the height and width are typically updated together,
  * and including only `width` prevents unnecessary re-renders.
  */
+/* eslint-enable tsdoc/syntax */
 export function useDeviceOrientation(): IterableDeviceOrientation {
   const { height, width } = useWindowDimensions();
 
   const [isPortrait, setIsPortrait] = useState<boolean>(height >= width);
 
   useEffect(() => {
     setIsPortrait(height >= width);
-    //  TODO: why is height not included in the dependency array?
+    // MOB-10425: why is height not included in the dependency array?
     // eslint-disable-next-line react-hooks/exhaustive-deps
   }, [width]);
 

src/inApp/classes/IterableInAppMessage.ts

@@ -155,7 +155,7 @@ export class IterableInAppMessage {
    */
   isSilentInbox(): boolean {
     return (
-      // TODO: Figure out if this is purposeful
+      // MOB-10424: Figure out if this is purposeful
       // eslint-disable-next-line eqeqeq
       this.saveToInbox && this.trigger.type == IterableInAppTriggerType.never
     );
@@ -199,7 +199,7 @@ export class IterableInAppMessage {
       messageId,
       campaignId,
       trigger,
-      // TODO: Speak to the team about `IterableInAppMessage` requiring a date
+      // MOB-10426: Speak to the team about `IterableInAppMessage` requiring a date
       // object, but being passed a number in this case
       // eslint-disable-next-line @typescript-eslint/ban-ts-comment
       //  @ts-ignore

src/inbox/classes/IterableInboxDataModel.ts

@@ -258,13 +258,13 @@ export class IterableInboxDataModel {
   ): IterableInAppMessage[] {
     let sortedFilteredMessages = messages.slice();
 
-    // TODO: Figure out if this is purposeful
+    // MOB-10424: Figure out if this is purposeful
     // eslint-disable-next-line eqeqeq
     if (this.filterFn != undefined) {
       sortedFilteredMessages = sortedFilteredMessages.filter(this.filterFn);
     }
 
-    // TODO: Figure out if this is purposeful
+    // MOB-10424: Figure out if this is purposeful
     // eslint-disable-next-line eqeqeq
     if (this.comparatorFn != undefined) {
       sortedFilteredMessages.sort(this.comparatorFn);

src/inbox/components/IterableInbox.tsx

@@ -273,7 +273,7 @@ export const IterableInbox = ({
       removeInboxChangedListener();
       inboxDataModel.endSession(visibleMessageImpressions);
     };
-    //  TODO: figure out if missing dependency is a bug
+    //  MOB-10427: figure out if missing dependency is a bug
     // eslint-disable-next-line react-hooks/exhaustive-deps
   }, []);
 
@@ -290,7 +290,7 @@ export const IterableInbox = ({
         inboxDataModel.endSession(visibleMessageImpressions);
       }
     }
-    //  TODO: figure out if missing dependency is a bug
+    //  MOB-10427: figure out if missing dependency is a bug
     // eslint-disable-next-line react-hooks/exhaustive-deps
   }, [appState]);
 
@@ -304,14 +304,14 @@ export const IterableInbox = ({
         inboxDataModel.endSession(visibleMessageImpressions);
       }
     }
-    //  TODO: figure out if missing dependency is a bug
+    //  MOB-10427: figure out if missing dependency is a bug
     // eslint-disable-next-line react-hooks/exhaustive-deps
   }, [isFocused]);
 
   //updates the visible rows when visible messages changes
   useEffect(() => {
     inboxDataModel.updateVisibleRows(visibleMessageImpressions);
-    //  TODO: figure out if missing dependency is a bug
+    //  MOB-10427: figure out if missing dependency is a bug
     // eslint-disable-next-line react-hooks/exhaustive-deps
   }, [visibleMessageImpressions]);
 
@@ -320,7 +320,7 @@ export const IterableInbox = ({
     if (isMessageDisplay) {
       returnToInbox();
     }
-    //  TODO: figure out if missing dependency is a bug
+    //  MOB-10427: figure out if missing dependency is a bug
     // eslint-disable-next-line react-hooks/exhaustive-deps
   }, [returnToInboxTrigger]);
 
@@ -364,7 +364,7 @@ export const IterableInbox = ({
     setSelectedRowViewModelIdx(index);
 
     Iterable.trackInAppOpen(
-      // TODO: Have a safety check for models[index].inAppMessage
+      // MOB-10428: Have a safety check for models[index].inAppMessage
       // eslint-disable-next-line @typescript-eslint/ban-ts-comment
       // @ts-ignore
       models[index].inAppMessage,
@@ -477,7 +477,7 @@ export const IterableInbox = ({
 
   const inboxAnimatedView = (
     <Animated.View
-      //  TODO: Change to use `StyleSheet.create` for styles, per best practices
+      // MOB-10429: Change to use `StyleSheet.create` for styles, per best practices
       // eslint-disable-next-line react-native/no-inline-styles
       style={{
         transform: [

src/inbox/components/IterableInboxMessageCell.tsx

@@ -152,7 +152,7 @@ function defaultMessageListLayout(
       >
         {thumbnailURL ? (
           <Image
-            // TODO: Use stylesheet according to best practices
+            // MOB-10429: Use stylesheet according to best practices
             // eslint-disable-next-line react-native/no-inline-styles
             style={{ height: 80, width: 80 }}
             source={{ uri: thumbnailURL }}

src/inbox/components/IterableInboxMessageList.tsx

@@ -119,7 +119,7 @@ export const IterableInboxMessageList = ({
 
       updateVisibleMessageImpressions(rowInfos);
     },
-    // TODO: Figure out if we need the missing dependencies
+    // MOB-10427: Figure out if we need the missing dependencies
     // eslint-disable-next-line react-hooks/exhaustive-deps
     []
   );