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.

Sign up for GitHub

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

Jump to bottom

Add warning when push notification cannot be sent due to an invalid userId or userId+activationId combination #841

Closed
zcgandcomp opened this issue May 31, 2024 · 0 comments
Closed

Add warning when push notification cannot be sent due to an invalid userId or userId+activationId combination #841

zcgandcomp opened this issue May 31, 2024 · 0 comments
Assignees
@banterCZ
Labels
release

Comments

@zcgandcomp
Copy link
Member

zcgandcomp commented May 31, 2024 •
edited
Loading

Currently if userId or combination of userId and activationId doesn't have any activation to send push notification to, we do not send the push, but push server doesn't provide any warning.

Add to log WARN message that no device was found to send the push notification to.

(Alternatively we can raise an exception if the combination is not valid - TBD)
@zcgandcomp zcgandcomp changed the title Enable notification when push notification cannot be send due to a invalid userId or userId+activationId combination Add warning when push notification cannot be send due to a invalid userId or userId+activationId combination Sep 30, 2024
@banterCZ banterCZ changed the title Add warning when push notification cannot be send due to a invalid userId or userId+activationId combination Add warning when push notification cannot be sent due to an invalid userId or userId+activationId combination Oct 1, 2024
banterCZ added a commit that referenced this issue Oct 1, 2024
@banterCZ
Fix #841: Add warning when push notification cannot be sent due to an…
e385762 
… invalid userId or userId+activationId combination
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@banterCZ banterCZ

Labels
release
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@banterCZ @zcgandcomp