feat(expo): enhances notification config retrieval
Description
Expo 54 will deprecate the use of notification in expo config in favor of expo-notifications expo plugin : https://docs.expo.dev/versions/v54.0.0/config/app/#notification
Improves notification configuration by first checking expo-notifications plugin for icon and color.
Falls back to deprecated config.notification settings. Warns about missing notification icon for Android.
Related issues
fixes: https://github.com/invertase/react-native-firebase/issues/8664
Release Summary
Checklist
- I read the Contributor Guide and followed the process outlined there for submitting PRs.
- [X] Yes
- My change supports the following platforms;
- [X]
Android - [X]
iOS - [X]
Other(macOS, web)
- [X]
- My change includes tests;
- [X]
e2etests added or updated inpackages/\*\*/e2e - [X]
jesttests added or updated inpackages/\*\*/__tests__
- [X]
- [X] I have updated TypeScript types that are affected by my change.
- This is a breaking change;
- [ ] Yes
- [X] No
Test Plan
Think react-native-firebase is great? Please consider supporting the project with any of the below:
- 👉 Star this repo on GitHub ⭐️
- 👉 Follow
React Native FirebaseandInvertaseon Twitter
The latest updates on your projects. Learn more about Vercel for GitHub.
| Project | Deployment | Preview | Comments | Updated (UTC) |
|---|---|---|---|---|
| react-native-firebase | Preview | Comment | Sep 23, 2025 3:42pm |
Hello 👋, this PR has been opened for more than 2 months 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 15 days until this gets closed automatically
Ping @mikehardy - @MichaelVerdon to get your opinion on this
just closing and reopening to make sure CI runs, it appears to be hung for some reason
just closing and reopening to make sure CI runs, it appears to be hung for some reason
I did a rebase on latest main + fix lint
duplicate test b2637599164e42220b6aec14faf74e1dfddb6bf0 to make sure both configuration are working
Any ETA for next version with this change?
Hello 👋, this PR has been opened for more than 2 months 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 15 days until this gets closed automatically
Ping @mikehardy @MichaelVerdon, this is not stale
apologies for the delay - been spinning through all the Invertase npmjs packages implementing their new Trusted Publish thing so we can keep releasing software at all, and it's delayed me going through PRs here. This is on the radar though