Skip to content

refactor(app-dist)!: migrate to TypeScript#8860

Open
MichaelVerdon wants to merge 22 commits intomainfrom
app-distribution-ts-migrations
Open

refactor(app-dist)!: migrate to TypeScript#8860
MichaelVerdon wants to merge 22 commits intomainfrom
app-distribution-ts-migrations

Conversation

@MichaelVerdon
Copy link
Collaborator

Description

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:

@vercel
Copy link

vercel bot commented Jan 27, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
react-native-firebase Ready Ready Preview, Comment Mar 3, 2026 2:03pm

Request Review

Copy link
Member

@russellwheatley russellwheatley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking good so far, @MichaelVerdon 👍

@MichaelVerdon MichaelVerdon changed the title feat(app-dist): typescript migrations refactor(app-dist): migrate to TypeScript Feb 9, 2026
@github-actions
Copy link

Hello 👋, this PR has been opened for more than 14 days with no activity on it.

If you think this is a mistake please comment and ping a maintainer to get this merged ASAP! Thanks for contributing!

You have 7 days until this gets closed automatically

@github-actions github-actions bot added the Stale label Feb 23, 2026
Copy link
Member

@russellwheatley russellwheatley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also need to reinstate index.d.ts and call it namespaced.ts. Namespaced should use namespaced types. Also need to deprecate all api on it. See storage types/namespaced.ts: https://github.com/invertase/react-native-firebase/pull/8824/changes#diff-fd5d9fcbc03d8f4891bacac55f4e1f793eb5c1c95a5acdb6ad85f1bbd14ee205R15-R1248

@MichaelVerdon MichaelVerdon changed the title refactor(app-dist): migrate to TypeScript refactor(app-dist)!: migrate to TypeScript Mar 3, 2026
Copy link
Member

@russellwheatley russellwheatley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

getting there!

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