FCM.Net icon indicating copy to clipboard operation
FCM.Net copied to clipboard

FCM updated their api.

Open Almeonamy opened this issue 7 years ago • 11 comments

FCM updated their api. Do you plan to update your library for new api version? Now default sounds are not supported in iOS by FCM.Net. https://firebase.google.com/docs/reference/fcm/rest/v1/projects.messages

Almeonamy avatar Dec 26 '17 07:12 Almeonamy

Yes, soon I will make a new version available. Feel free to submit a Pull Request :)

angelobelchior avatar Dec 26 '17 16:12 angelobelchior

What exactly did they change? based on the docs it looks like Notification only has Title / Body but FCM's notification contains additional properties like Badge. Is this what has changed? Looks like they added platform specific objects at the root as well.

Does this mean badge updates won't work?

rburnham52 avatar Feb 06 '18 06:02 rburnham52

Ah i see, FCM.net is using Legacy format. https://firebase.google.com/docs/cloud-messaging/http-server-ref

rburnham52 avatar Feb 06 '18 07:02 rburnham52

I started working on this and it seems like a bigger change that i thought. The V1 is an entirely new endpoint that requires the ProjectId for the Url and authenticates using OAuth 2.0.

It's looking like some pretty big change to sender that i was hoping not to do. At the moment i'm just creating a new V1 namespace, that way anyone using the old format should still work. But there seems to be enough changes to question, should this be a separate package?

rburnham52 avatar Feb 06 '18 09:02 rburnham52

In my opinion the legacy implementation should be a separate package, as there are too many changes and I think it would be hard to maintain both api versions in the same library. However, whatever will be the final decision: I'm currently looking for a good option to send fcm messages using the new API version as I need the separation of values for iOS and Android. So could help to implement the new api version here rather than writing an own implementation. So if you already started implementation, maybe I could contribute.

martinhey avatar Feb 22 '18 17:02 martinhey

@martinhey I will begin soon to change the api to use the new features. Feel free to contribute :)

angelobelchior avatar Feb 22 '18 18:02 angelobelchior

of course... let's keep in touch and define subtasks

martinhey avatar Feb 22 '18 19:02 martinhey

@martinhey so @angelobelchior i'm just starting out, but does this package not support V1? Should I not consider this Package? Perhaps update the Readme.md with the details.

markalanevans avatar May 08 '18 17:05 markalanevans

is time_to_live change to ttl? because i can't set it in my projects now

ejdrian313 avatar Jul 27 '18 09:07 ejdrian313

@angelobelchior What is the latest situation on this subject?

ofcerci4826 avatar Jan 25 '21 11:01 ofcerci4826

@ofcerci4826 We ended up using FCMSharp instead.

rburnham52 avatar Jan 27 '21 06:01 rburnham52