Page MenuHomePhabricator

[native] Move NotifPermissionAlertInfo to lib

Authored by michal on Mar 10 2023, 7:28 AM.
Referenced Files
Unknown Object (File)
Fri, Apr 12, 1:13 AM
Unknown Object (File)
Fri, Apr 12, 1:13 AM
Unknown Object (File)
Fri, Apr 12, 1:12 AM
Unknown Object (File)
Fri, Apr 12, 1:08 AM
Unknown Object (File)
Tue, Mar 26, 2:19 PM
Unknown Object (File)
Mar 13 2024, 1:58 AM
Unknown Object (File)
Feb 22 2024, 12:10 AM
Unknown Object (File)
Feb 20 2024, 11:49 PM



We want to show the notif permission modal on web less often and we can reuse NotifPermissionAlertInfo for this. This diff moves it from native redux to the shared redux setup and the base reducer. The next diff will actually use it on web.

Test Plan

Check if redux is correctly updated after showing the alerts on native. Test both a fresh device and when updating. The web was tested with the next diff, and it also correctly updates redux.

Diff Detail

rCOMM Comm
Lint Not Applicable
Tests Not Applicable

Event Timeline

Looks ok - it doesn't seem necessary to do a migration.

A test plan should include more checks:

  • does it work on web?
  • does it work on fresh native?
  • does it work when updating native?

I'm not sure but only 2nd or 3rd is covered by the current test plan.

This revision is now accepted and ready to land.Mar 13 2023, 7:22 AM