bcgov / bc-wallet-mobile

BC Wallet to hold Verifiable Credentials
Apache License 2.0
53 stars 44 forks source link

Push Notifications mediator message failure on iOS after Credo updates #1959

Open bryce-mcmath opened 3 weeks ago

bryce-mcmath commented 3 weeks ago

Description

Since the Credo updates we now see error logs related to push notifications on iOS, like the following:

PushNotificationsFcmSetDeviceInfoMessage: Failed to validate class.\nAn instance of PushNotificationsFcmSetDeviceInfoMessage has failed the validation:\n - property devicePlatform has failed the following constraints: devicePlatform must be a string \n

devicePlatform doesn't appear in the wallet codebase anywhere so I'm not yet sure what's causing the issue.

Worth checking with the mediator team to see if they updated any dependencies recently.

bryce-mcmath commented 1 week ago

The issue has been fixed and push notifications are working again, but there is still some error logs related to validation of the response summarized in the below ticket:

1976