Skip to content
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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(#38): make module TurboModule-compatible #59

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mateusz1913
Copy link

Closes #38

馃憢 This PR adds TurboModule support

List of changes:

  • create JS Codegen spec
  • use install_modules_dependencies util in podspec if possible, to install React deps
  • migrate Android module class to conform to codegenerated spec when new arch enabled
  • copy codegen-ed to oldarch Android source set to make module work when new arch disabled
  • migrate iOS module class to conform to codegenerated spec when new arch enabled

- create JS Codegen spec
- use install_modules_dependencies util in podspec if possible, to install React deps
- migrate Android module class to conform to codegenerated spec when new arch enabled
- copy codegen-ed to oldarch Android source set to make module work when new arch disabled
- migrate iOS module class to conform to codegenerated spec when new arch enabled
@mateusz1913
Copy link
Author

Hi @LinusU , let me know if there's anything I'd need to improve in this PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Migrate to new architecture
1 participant