Before sending a push notification, we used to check if the sending atom and the recipient atom belong to the same user. This check does not seem to work for group chats (in which messages are forwarded by the group atom). We remove this check as the overall experience is better without it.
Pull request checklist
Please check if your PR fulfills the following requirements:
[ ] Affected Tests have been added/altered (for bug fixes / features)
[ ] Docs have been reviewed and added/updated if needed (for bug fixes / features)
[ ] Build was run locally and mvn install succeeds
Pull request type
Please check the type of change your PR introduces:
[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
Before sending a push notification, we used to check if the sending atom and the recipient atom belong to the same user. This check does not seem to work for group chats (in which messages are forwarded by the group atom). We remove this check as the overall experience is better without it.
Pull request checklist
Please check if your PR fulfills the following requirements:
mvn installsucceedsPull request type
Please check the type of change your PR introduces:
What is the current behavior?
What is the new behavior?
-
-
Does this introduce a breaking change?
Other information