Page MenuHomePhabricator

Fix broken in-app notification on native
ClosedPublic

Authored by marcin on Wed, Sep 4, 4:04 AM.
Tags
None
Referenced Files
F2842713: D13237.diff
Sun, Sep 29, 3:26 PM
Unknown Object (File)
Fri, Sep 27, 5:53 PM
Unknown Object (File)
Thu, Sep 26, 2:40 AM
Unknown Object (File)
Mon, Sep 23, 2:18 PM
Unknown Object (File)
Mon, Sep 23, 3:31 AM
Unknown Object (File)
Tue, Sep 17, 5:27 PM
Unknown Object (File)
Tue, Sep 17, 2:40 PM
Unknown Object (File)
Mon, Sep 16, 1:18 AM
Subscribers

Details

Summary

This differential fixes broken thiock thread in-app notifications on Android and iOS. They were broken since our code considered them to be old schema notifications and converted them by adding authoritativeKeyserverID prefix.

Test Plan
  1. Create thick thread using Kamil UI button.
  2. Send notifications to iOS and Android devices. Ensure that in-app notifications are correctly handled: not displayed when thread is active and displayed in other cases

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable