feat: migrate AppDelegate to Swift and remove legacy files for improved architecture #660
Conversation
…update types for SDK functions
…nse types for better null handling
❌ 5 blocking issues (5 total)
This is from Qlty Cloud, the successor to Code Climate Quality. Learn more. |
![qlty-cloud-legacy[bot]](https://avatars.githubusercontent.com/in/9403?s=60&v=4){kind=small}
| @@ -0,0 +1,113 @@ | |||
| import { type TurboModule, TurboModuleRegistry } from 'react-native'; | |||
There was a problem hiding this comment.
| @@ -0,0 +1,4 @@ | |||
| import { NativeEventEmitter, NativeModules } from 'react-native'; | |||
There was a problem hiding this comment.
| @@ -0,0 +1,9 @@ | |||
| import TurboIterableModule from './NativeRNIterableApi'; // new arch | |||
There was a problem hiding this comment.
| @@ -714,6 +721,7 @@ export class Iterable { | |||
| ) { | |||
| Iterable?.logger?.log('updateUser'); | |||
|
|
|||
| // @ts-ignore | |||
There was a problem hiding this comment.
Use "@ts-expect-error" instead of "@ts-ignore", as "@ts-ignore" will do nothing if the following line is error-free. [eslint:@typescript-eslint/ban-ts-comment]
| // @ts-ignore | |
| // @ts-expect-error |
Replace "@ts-ignore" with "@ts-expect-error".
| dataRegion: this.dataRegion, | ||
| // @ts-ignore |
There was a problem hiding this comment.
Use "@ts-expect-error" instead of "@ts-ignore", as "@ts-ignore" will do nothing if the following line is error-free. [eslint:@typescript-eslint/ban-ts-comment]
| // @ts-ignore | |
| // @ts-expect-error |
Replace "@ts-ignore" with "@ts-expect-error".
| @@ -0,0 +1,113 @@ | |||
| import { type TurboModule, TurboModuleRegistry } from 'react-native'; | |||
There was a problem hiding this comment.
| @@ -0,0 +1,4 @@ | |||
| import { NativeEventEmitter, NativeModules } from 'react-native'; | |||
There was a problem hiding this comment.
| @@ -0,0 +1,9 @@ | |||
| import TurboIterableModule from './NativeRNIterableApi'; // new arch | |||
There was a problem hiding this comment.
| @@ -714,6 +721,7 @@ export class Iterable { | |||
| ) { | |||
| Iterable?.logger?.log('updateUser'); | |||
|
|
|||
| // @ts-ignore | |||
There was a problem hiding this comment.
Use "@ts-expect-error" instead of "@ts-ignore", as "@ts-ignore" will do nothing if the following line is error-free. [eslint:@typescript-eslint/ban-ts-comment]
| // @ts-ignore | |
| // @ts-expect-error |
Replace "@ts-ignore" with "@ts-expect-error".
| dataRegion: this.dataRegion, | ||
| // @ts-ignore |
There was a problem hiding this comment.
Use "@ts-expect-error" instead of "@ts-ignore", as "@ts-ignore" will do nothing if the following line is error-free. [eslint:@typescript-eslint/ban-ts-comment]
| // @ts-ignore | |
| // @ts-expect-error |
Replace "@ts-ignore" with "@ts-expect-error".
🔹 JIRA Ticket(s) if any
✏️ Description