HomePhabricator
Diffusion Comm 893f9352eeff

Possible solution for broken iOS notifications

Description

Possible solution for broken iOS notifications

Summary:

IMPORTANT: This differential is for test release purposes.

The context for this change is described here: https://linear.app/comm/issue/ENG-4453#comment-4d1770b3. We introduce dummy body to the encrypted notification to make sure we comply to apple docs. Additionally we make necessary changes to
NotificationService extension to prevent from displaying notification that contains ENCRYPTED text.

Test Plan: Build release iOS app. Ensure that notifications, rescinds and bade-only notifications work correctly

Reviewers: tomek, atul, bartek, ashoat

Reviewed By: ashoat

Differential Revision: https://phab.comm.dev/D8638

Details

Provenance
marcinAuthored on Aug 4 2023, 7:35 AM
Reviewer
ashoat
Differential Revision
D8638: Possible solution for broken iOS notifications
Parents
rCOMMda5391ff5719: Implement isStaffRelease method in PlatformSpecificTools
Branches
Unknown
Tags
Unknown