FCM send_message return type is not compatible with previous output

[sevdog]

The implementation with firebase SDK has changed the output of the send_message method

Before it was a dict

django-push-notifications/push_notifications/gcm.py

Lines 202 to 204 in 7d28052

	return ret[0] if len(ret) == 1 else ret
	else:
	return _cm_send_request(None, data, cloud_type=cloud_type, **kwargs)

Now it is a BatchResponse

django-push-notifications/push_notifications/gcm.py

Lines 188 to 190 in 0f79181

	return messaging.BatchResponse(ret)
	else:
	return messaging.BatchResponse([])

This means that any code which relies on the output to be a dict now need to work-around the new type.

This is a breaking change which should have been highlighted or avoided.

[sevdog]

My point is that previously it was possible to handle the output of different Device (ie: WebpushDevice and GCMDevice) with the same code because they were returning dict/list[dict] according if you were calling the method on a single instance or on the queryset.

Now the code should be reworked to hande the BatchResponse from firebase SDK.

My question is: is this project willing to keep a common API for .send_message output or is it letting any device kind to provide a different output for such method?