-
Notifications
You must be signed in to change notification settings - Fork 589
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
notifications: general marketing #5206
Conversation
Taking a look after expo meeting |
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.
Looked good on Android as well, QA Passed ✅
https://github.com/rainbow-me/rainbow/assets/41752922/f44b8ff7-55db-46cc-9035-1bdd495518a3
also did a check on notifications from WC and mainnet txns 👍🏽
logger.info(`NotificationsHandler: handling marketing notification`, { | ||
notification, | ||
}); | ||
const data = (notification.data as unknown) as MarketingNotificationData; |
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.
idk what double type casting does but just noting it in case its not necessary
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.
can't cast it to MarketingNotificationData
without casting to unknown first, TS bs
…brody/QR-code-scanner-fixes * 'develop' of https://github.com/rainbow-me/rainbow: notifications: general marketing (#5206) [APP-946]: Fixes swap input currencylist value bug (#5221)
Fixes APP-934
What changed (plus any additional context for devs)
added simple handling for marketing focused notifications, we already technically had support for them but now it allows us to navigate to different screens
Screen recordings / screenshots
https://cloud.skylarbarrera.com/RPReplay_Final1701273411.MP4
What to test
android, can't build locally atm 😮💨