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

FCM integration #56

Open
zdnk opened this issue Apr 13, 2023 · 15 comments
Open

FCM integration #56

zdnk opened this issue Apr 13, 2023 · 15 comments

Comments

@zdnk
Copy link

zdnk commented Apr 13, 2023

Hello,

we have all tokens from FCM for iOS as well and we would like to import them in Customer.IO. However we also need to collect the new tokens as FCM tokens if I understand correctly because you cannot configure CustomerIO to send it as ASPN to some users as FCM to others right?

When is the integration planned?

@ami-aman
Copy link
Contributor

Hi @zdnk

Our Expo config plugin currently only support APNS for iOS and FCM for Android.

We have FCM integration request on our roadmap and have added your vote to the request. We will notify you when this is prioritized. Thanks for your feedback. We really appreciate it !

@zdnk
Copy link
Author

zdnk commented Apr 17, 2023

What if I manually call CustomerIO.registerDeviceToken in javascript with the FCM token? Will that work?

@ami-aman
Copy link
Contributor

ami-aman commented Apr 17, 2023

I am afraid that might lead to device token conflict i.e you will register FCM token using javascript and native module would generate an APNS token and register it.

@zdnk
Copy link
Author

zdnk commented Apr 17, 2023

even if I have in my Expo project in the app.config.js disabled the notification registration?

ios: {
          useFrameworks: 'static',
          disableNotificationRegistration: true,
}

@ami-aman
Copy link
Contributor

Yes, even if you have disableNotificationRegistration : true then also this workaround might not work as expected. AppDelegate.m or AppDelegate.mm file would have delegate methods for APNS and if you attempt to send a FCM push notification, your device might never receive it. Let me know if you need more clarity on this. I would be happy to explain further.

@zdnk
Copy link
Author

zdnk commented Apr 17, 2023

Ok. What can I do to make it work somehow? Because unfortunately we are using firebase and we have been using it with the old app version as well. I am open to suggestion where our backend register the devices in the customerio. But I need some solution :)

@ami-aman
Copy link
Contributor

ami-aman commented Apr 19, 2023

@zdnk

I apologies for the delay.

As a temporary solution, you could refer to the steps outlined in our React Native documentation. Ensure that you select FCM as you are using it for push notifications and choose either Objective-C or Swift based on your project. However, please note that the drawback of this workaround is that the changes made will only persist until you execute the 'prebuild --clean' command.

Do let us if this helps us. Feel free to comment if you need any assistance.

@zdnk
Copy link
Author

zdnk commented Apr 19, 2023

Well, se are working with Expo managed for now. Using suggested solution by you does require bare workflow.

What about adding option to simply completely disable push notification registration? This would actually solve the issue as it will allow developers to use FCM manually as workaround like I am doing right now. And probably would be easy to implement, right?

@zdnk
Copy link
Author

zdnk commented Sep 22, 2023

Any update here? Making it work together with Expo notifications would be also enough.

@levibostian
Copy link
Contributor

Adding FCM support to our Expo plugin is continuing to be tracked and prioritized internally. We do not have any updates to share at this time. We will update this issue when we have updates to share.

Thank you, everyone!

@zdnk
Copy link
Author

zdnk commented Jul 8, 2024

9 months later, whats the update?

@levibostian
Copy link
Contributor

Thank you for following up on this feature request. We understand that waiting for updates can be frustrating, and we sincerely apologize for any inconvenience this may have caused.

Please know that we are continuing to track and prioritize this feature internally. Our team is dedicated to ensuring that we deliver high-quality updates, and this sometimes requires careful planning and evaluation to get things right.

Your feedback is incredibly important to us, and we truly appreciate your patience and understanding as we work through our development pipeline.

Have a great day!

@billinghamj
Copy link

This is looking to be a blocker for us also. Bit disappointing that this is taking so long

@Nikit-Singh
Copy link

Hey @levibostian, any updates on this?

@pablogdcr
Copy link

Also a blocker for us, please let us know if you have any updates on this!

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

No branches or pull requests

6 participants