Skip to content

Conversation

@russellwheatley
Copy link
Member

@russellwheatley russellwheatley commented Dec 9, 2025

Description

  • Created an index.ts which exports for consumers only, separated modular in modular.ts and namespaced.ts for separation of concerns.
  • Created a types/app.ts for consumer types, and types/internal.ts for internal implementation types.
  • Updated paths in package.json to allow peer dependencies (i.e. other RNFB packages) to maintain access to internal and common API.
  • Updated other packages that already use typescript to ensure they can still access app package internal API (common/ and internal/).
  • Refactored functions, removed types.d.ts file and put in types/functions.ts like app package.

Related issues

Release Summary

Checklist

  • I read the Contributor Guide and followed the process outlined there for submitting PRs.
    • Yes
  • My change supports the following platforms;
    • Android
    • iOS
    • Other (macOS, web)
  • My change includes tests;
    • e2e tests added or updated in packages/\*\*/e2e
    • jest tests added or updated in packages/\*\*/__tests__
  • I have updated TypeScript types that are affected by my change.
  • This is a breaking change;
    • Yes
    • No

Test Plan


Think react-native-firebase is great? Please consider supporting the project with any of the below:

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.

2 participants