Projects tigase _server server-core Issues #1527
Add support for showing accounts with push notifications enabled as `away` (#1527)
Closed
Andrzej Wójcik (Tigase) opened 3 months ago

Many user accounts that have push notifications enabled are shown as offline for other users even though account owner clients can be notified about upcoming messages. For many iOS clients, being offline but available to delivery message by push notifications is a normal state and users that see those accounts as offline tend not to send messages as they expect their message will not reach the recipient.

Andrzej Wójcik (Tigase) added "Related" Customers/Alianza#145 3 months ago
Andrzej Wójcik (Tigase) commented 3 months ago

Additionally, due to that devices can be "hanging" for a while without being unregistered after user ends usage of the mobile app, it may be desired to have a TTL for enabling push notifications for particular device.

Referenced from commit 3 months ago
Andrzej Wójcik (Tigase) changed state to 'In Progress' 3 months ago
Previous Value Current Value
Open
In Progress
Andrzej Wójcik (Tigase) changed state to 'In QA' 3 months ago
Previous Value Current Value
In Progress
In QA
Andrzej Wójcik (Tigase) commented 3 months ago

We also have an ad-hoc command to list and disable push notifications for selected devices just in case if some device is stuck as enabled.

@kobit @wojtek Feature was implemented and tested locally. As the next step, we could deploy it on sure.im/tigase.im and verify its behavior on a real environment. Or should we use a different environment to enable this feature?

Wojtek, could you look at the documentation for this feature? is it OK? or we need to change something?

Referenced from commit 3 months ago
wojciech.kapcia@tigase.net commented 3 months ago

Feature was implemented and tested locally. As the next step, we could deploy it on sure.im/tigase.im and verify its behavior on a real environment. Or should we use a different environment to enable this feature?

I think testing on tigase.im/sure.im would be a good idea

Wojtek, could you look at the documentation for this feature? is it OK? or we need to change something?

Synchronised github repo and manually forced RTD build to make it up to date: https://docs.tigase.net/en/latest/Tigase_Administration/Configuration/_Configuration.html#push-devices-presence

  1. is there any default for deviceRegistrationTTL?

Apart the docs look ok - I did some light editor change.

Andrzej Wójcik (Tigase) commented 3 months ago

There is no default for deviceRegistrationTTL, so the value is by default null. With that we have the same behavior as previously - no TTL. The reason for leaving TTL disabled is that normally push components should inform XMPP server when it was not possible to deliver notification due to application removal on the users device or similar issue and that would result in removal of the entry from the list of push enabled devices/clients for the user hosted on XMPP server.

wojciech.kapcia@tigase.net changed fields 3 months ago
Name Previous Value Current Value
Version
8.4.0
tigase-server-8.4.0
wojciech.kapcia@tigase.net added to iteration "tigase-server-8.4.0" 3 months ago
Andrzej Wójcik (Tigase) changed state to 'Closed' 3 months ago
Previous Value Current Value
In QA
Closed
issue 1 of 1
Type
Task
Priority
Normal
Assignee
Version
tigase-server-8.4.0
Sprints
n/a
Customer
n/a
Iterations
Issue Votes (0)
Watchers (3)
Reference
tigase/_server/server-core#1527
Please wait...
Page is in error, reload to recover