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

receiving notifications while the app/chat is open #214

Open
nanopone opened this issue Apr 13, 2023 · 1 comment
Open

receiving notifications while the app/chat is open #214

nanopone opened this issue Apr 13, 2023 · 1 comment
Labels
bug Something isn't working

Comments

@nanopone
Copy link

Describe the bug
when reading a chat and receiving a message in it, siskin sends a push notification anyway, which is annoying and not expected since other messaging apps do not do this. it seems like an oversight

To Reproduce
Steps to reproduce the behavior:

  1. open a chat
  2. receive a message from inside of that chat

Expected behavior
i expect siskin to not send any push notifications either when the chat is open or when the app is open at all

Details (please complete the following information):

  • Siskin Version: 7.3.3
  • iOS version: 16.3.1
  • iPhone model: iphone 13 mini
@nanopone nanopone added the bug Something isn't working label Apr 13, 2023
@hantu85
Copy link
Contributor

hantu85 commented Apr 13, 2023

Siskin does not generate nor send push notifications - it displays them.

You've also reported #213 which in fact is causing this issue as well (in my opinion). Most likely your server doesn't care if your XMPP client is connected and sends push notifications even if your client is connected. This in my opinion is an issue with the implementation of XEP-0357: Push Notifications on the sever side as in the introduction of this XEP it is stated:

The purpose of push notifications is to inform users of new messages or other pertinent information even when they have no XMPP clients online.

In this case, you do have a connected client and you receive notification anyway.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants