-
Notifications
You must be signed in to change notification settings - Fork 41
2.0.0-alpha #589
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
2.0.0-alpha #589
Conversation
This commit refactors the `IterableAction` and `IterableActionContext` classes. It moves them from the `src` folder to the `ts` folder. Additionally, it removes the unused `use strict` directive and renames the `index` file. The changes aim to improve the code organization and maintainability.
Refactor IterableInbox.tsx and remove unused imports Add comment to IterableInboxMessageCell.tsx Add comment to IterableInboxMessageDisplay.tsx Add comments and descriptions to IterableLogger.ts Add a description to IterableUtil.ts Add comment to useAppStateListener.ts Add comment to useDeviceOrientation.tsx
Refactor IterableInbox.tsx and improve code readability
…sdk-to-use-recommended-scaffolding [MOB-9669] update-rn-sdk-to-use-recommended-scaffolding
…-rules-stricter [MOB-10141] make lint rules stricter
…a/MOB-10208-fix-npmignore
…OB-9680-new-fix-lefthook
…/MOB-10412-remove-integration-testing-code
…OB-10290-create-loader-for-rniterableapi
…nerator [MOB-9676] docs and doc generator
…nore [MOB-10208] Update .npmignore to include additional default ignores and test files
…fthook [MOB-9680] Fix lefthook
…ader-for-rniterableapi [MOB-10290] create-loader-for-rniterableapi
…tegration-testing-code [MOB-10412] remove-integration-testing-code
…s-into-tickets [MOB-10289] Make todos into tickets
[MOB-10430] Cleans up things not addressed in tickets
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The PR diff size of 67396 lines exceeds the maximum allowed for the inline comments feature.
|
There are too many results to compare The test coverage on the diff in this pull request is 38.1% (50% is the threshold). This pull request will bring the total coverage in the repository to 38.0% (-17.4% change). View more on Code Climate. |
evantk91
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just looked through the PR list.
Can you verify MOB-9270? We are using the latest iOS18 / Android 15. Is this just due to pointing to the released versions?
I think MOB-9748 is separate since this is dealing with updating release process.
You're correct re: MOB-9748. |
✏️ Description
This includes the following tickets in the epic MOB-9459: