-
Notifications
You must be signed in to change notification settings - Fork 2.7k
Open
Labels
otherIndicates an issue or PR lacks a `triage/foo` label and requires one.Indicates an issue or PR lacks a `triage/foo` label and requires one.
Description
The issue
In internal/msggateway/hub_server.go, the pushToUser function has this check:
if !client.IsBackground || (client.IsBackground && client.PlatformID != constant.IOSPlatformID) { ... }
It's like saying “not background OR (background AND not iOS)” which is hard to follow.
The code should just say “if the client is background AND iOS, skip the push”:
if client.IsBackground && client.PlatformID == constant.IOSPlatformID {
and push for everyone else in the same way as today.
Additional information
No response
Metadata
Metadata
Assignees
Labels
otherIndicates an issue or PR lacks a `triage/foo` label and requires one.Indicates an issue or PR lacks a `triage/foo` label and requires one.