Skip to content

chore(app-distribution): deprecations for v8 API ahead of future major release#8587

Merged
mikehardy merged 1 commit into
mainfrom
app-distribution-deprecations
Jul 1, 2025
Merged

chore(app-distribution): deprecations for v8 API ahead of future major release#8587
mikehardy merged 1 commit into
mainfrom
app-distribution-deprecations

Conversation

@MichaelVerdon

Copy link
Copy Markdown
Contributor

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

vercel Bot commented Jun 25, 2025

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
react-native-firebase ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 1, 2025 7:31pm

@@ -1,4 +1,5 @@
import { getApp } from '@react-native-firebase/app';
import { MODULAR_DEPRECATION_ARG } from '../../../app/lib/common';

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I've just noticed this, and it ought to be changed in the other PRs for deprecations. Import like this:

import { MODULAR_DEPRECATION_ARG } from '@react-native-firebase/app/lib/common';

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

All done on all PRs

@MichaelVerdon MichaelVerdon marked this pull request as ready for review June 25, 2025 12:54
@mikehardy mikehardy force-pushed the app-distribution-deprecations branch from d330c01 to 849864f Compare July 1, 2025 12:54

@mikehardy mikehardy left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM - another wonderful small one - rebased on main and squashed to single commit, pending clean CI run

@mikehardy mikehardy force-pushed the app-distribution-deprecations branch from 849864f to b0ab667 Compare July 1, 2025 19:19
@mikehardy mikehardy merged commit 757fa95 into main Jul 1, 2025
15 of 16 checks passed
@mikehardy mikehardy deleted the app-distribution-deprecations branch July 1, 2025 19:19
@mikehardy mikehardy removed the Workflow: Pending Merge Waiting on CI or similar label Jul 1, 2025
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.

3 participants