Upgrade to new notifications endpoint

rainbow-me / rainbow

🌈‒ the Ethereum wallet that lives in your pocket

https://rainbow.me

GNU General Public License v3.0

3.95k stars 627 forks source link

Closed jinchung closed 1 month ago

jinchung commented 1 month ago

Fixes APP-887

Adds support for the new notifications endpoint (REST PUT) which adds support for select L2 notifications
Removes the old explorer redux (which used websockets) which is no longer needed
Moves support for global notification topics (e.g, points) to the new endpoint as well
Migrates existing users to the new endpoint

Please note: APP-985 is still a known issue

linear[bot] commented 1 month ago

brunobar79 commented 1 month ago

Launch in simulator or device for d8ae563b0397cd85c70200af63081f9ef966c9ef

brunobar79 commented 1 month ago

Launch in simulator or device for fa8a22821e4b88e4a79f53bc05ccb54084a058f5

brunobar79 commented 1 month ago

Launch in simulator or device for bc2d20ec483d1a4c1657ceab219f92cfe3379af7

derHowie commented 1 month ago

not rerequesting, but the newest changes lgtm ✅

jinchung commented 1 month ago

merged early as all tests had passed except for lint, and now lint passed in the final run.

brunobar79 commented 1 month ago

Launch in simulator or device for 703607d547497ca846a1085118f95735af8324c9

sentry-io[bot] commented 1 month ago

This pull request was deployed and Sentry observed the following issues:

‼️ Error: [notifications]: Error while getting and saving FCM token Promise$argument_0(src/notifications/permissions) View Issue
‼️ Error: [wallet]: Error generating account for keychain consoleTransport(src/logger/index) View Issue
‼️ Error: [wallet]: Error in getPrivateKey consoleTransport(src/logger/index) View Issue
‼️ Error: KC unavailable for PKEY lookup consoleTransport(src/logger/index) View Issue
‼️ Error: [wallet]: Error in getPrivateKey consoleTransport(src/logger/index) View Issue

_{Did you find this useful? React with a 👍 or 👎}