Skip to content
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

RMET-3870 :: merge development into outsystems #39

Merged
merged 5 commits into from
Nov 27, 2024

Conversation

OS-martacarlos
Copy link

This PR merges development into outsystems which contains a fix for RPM-5386

OS-martacarlos and others added 4 commits June 21, 2024 16:25
- adds code specific to outsystems lifecycle aka one signal plugin manager
- adds the code that will be in outsystems wrapper's javascript nodes
- add sonar cloud integration for typescript files

https://outsystemsrd.atlassian.net/browse/RMET-3477
#36)

* feat: update FB Messaging Android library

References: https://outsystemsrd.atlassian.net/browse/RMET-3608

* feat: use Firebase BOM to get FB Cloud Messaging lib

Context: According to the documentation, this is the recommended way of controlling Firebase library versions. This way, using the Firebase Android BOM, an app with multiple Firebase Android libraries will always use compatible versions of these libraries.

References: https://outsystemsrd.atlassian.net/browse/RMET-3608

* Revert "feat: use Firebase BOM to get FB Cloud Messaging lib"

This reverts commit a3e6be3.

* fix: add necessary Firebase library for OneSignal to work with latest version of Cloud Messaging

Context: https://firebase.google.com/support/release-notes/android?_gl=1*d5zec9*_up*MQ..*_ga*OTY5NjA2OTg5LjE3MjYwNTUwMDE.*_ga_CW55HF8NVT*MTcyNjA1NTAwMS4xLjAuMTcyNjA1NTAwMS4wLjAuMA..#messaging_v22-0-0

References: https://outsystemsrd.atlassian.net/browse/RMET-3608

* chore: update changelog

References: https://outsystemsrd.atlassian.net/browse/RMET-3608
- The NotificationDismissReceiver declaration was missing in the plugin.xml , which was causing the receiver to not ever be called and as such, no dismissed notifications were being processed by one signal, causing the resurrections of them all

https://outsystemsrd.atlassian.net/browse/RMET-3870
@OS-martacarlos OS-martacarlos self-assigned this Nov 26, 2024
@OS-martacarlos OS-martacarlos requested a review from a team as a code owner November 26, 2024 14:38
@OS-martacarlos OS-martacarlos merged commit de51f01 into outsystems Nov 27, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants