Fix default FCM_MAX_RECIPIENTS

jazzband / django-push-notifications

Send push notifications to mobile devices through GCM or APNS in Django.

MIT License

2.23k stars 605 forks source link

Fix default FCM_MAX_RECIPIENTS #725

Closed djch closed 1 month ago

djch commented 1 month ago

Deployed the new release with FCM v1 API support and immediately discovered the default value is too high, triggering a messages must not contain more than 500 elements error from the Firebase SDK.

FCM v1 API seems to have changed this limit, so django-push-notifications should adhere to that.

djch commented 1 month ago

Doc reference: https://firebase.google.com/docs/cloud-messaging/send-message#send-messages-to-multiple-devices