-
Notifications
You must be signed in to change notification settings - Fork 16
fix: miscellaneous issues #140
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
base: master
Are you sure you want to change the base?
Conversation
Xml error deatils parsing for aws s3 provided error for files upload and download operations.
switch (this) { | ||
case PushGateway.fcm: | ||
return 'gcm'; | ||
return 'fcm'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is fcm
value allowed by the server?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Isn't that's the real update we needed after decommission of GCM support by Google? Referring this googleblog
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OR am I missing something here!?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As we tested, the fcm
value is allowed
…tched users count
fix: error handling in files apis.
Fixes issue of not getting proper error details when file upload fails.
fix: unrecoverable subscription.
Fixes issue of no-op when
subscribe()
called on inactive or errored subscrption.fix: generate file upload url API.
Fixes issue of generate upload file url to incorporate token when configured or set explicitly.
fix: mapping of gcm push notification gateway to fcm type
Discard use of gcm and replaced with fcm for push notification service type of google.
test: pubnub access manager tests