Page MenuHomePhabricator

Use initial notifications from queue rather than from launch options
ClosedPublic

Authored by marcin on Mar 13 2023, 3:47 AM.
Tags
None
Referenced Files
F1577150: D7058.diff
Thu, Apr 18, 6:16 PM
Unknown Object (File)
Fri, Apr 12, 3:02 AM
Unknown Object (File)
Fri, Apr 12, 3:02 AM
Unknown Object (File)
Fri, Apr 12, 3:02 AM
Unknown Object (File)
Fri, Apr 12, 3:02 AM
Unknown Object (File)
Fri, Apr 12, 2:58 AM
Unknown Object (File)
Mar 8 2024, 1:34 AM
Unknown Object (File)
Mar 7 2024, 8:58 PM
Subscribers

Details

Summary

This differential is a fix for the iOS part of this issue: https://linear.app/comm/issue/ENG-3269/app-erroneously-opens-to-chats-at-random-times. Initially when app was entering the foreground in respone to tapping the notif we were querying
initial launch options for notification that launched the app (no the one that was tapped, but the one that booted the app). In case there was any we would navigate user to the thread corresponding to this notification. This resulted in erroneous navigation since the thread correpsonding to
the lastest notif that foregrounded (not booted!) the app might had been different.

Test Plan

Build the app and check that tapping notification always navigates us to the correct thread.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Looks good! Thanks for linking to the Linear issue and jotting down detailed notes there.

This revision is now accepted and ready to land.Mar 13 2023, 5:03 PM

Why hasn't this been landed? It's an urgent issue and the diff was accepted on Monday...

Why hasn't this been landed? It's an urgent issue and the diff was accepted on Monday...

I will make sure to land differentials of this kind more efficiently in future.