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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: Config to exlcude message push #392

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

mahmoud-elmorabea
Copy link
Contributor

@mahmoud-elmorabea mahmoud-elmorabea commented Mar 17, 2025

The solution creates a Gradle property customerio.reactnative.excludePushMessaging that customers of our RN SDK can use to exclude push messaging dependency. This flag can also be used by our Expo plugin to do the same thing.

What happens when you use this flag?

  • The push messaging dependency is not included on Android which means that Firebase isn't included either preventing customers from having to provide a google-services.json if they don't want to use push
  • Our JS interface for RN and Expo will still have push messaging related API but if customers calls them while disabling push via the flag, they will either crash (runtime error) or print an error log saying push isn't enabled

@mahmoud-elmorabea mahmoud-elmorabea self-assigned this Mar 17, 2025
Copy link

github-actions bot commented Mar 17, 2025

Sample app builds 📱

Below you will find the list of the latest versions of the sample apps. It's recommended to always download the latest builds of the sample apps to accurately test the pull request.


  • APN: 392.1.0 (29036983)

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.

1 participant