Page MenuHomePhabricator

[native] Move NotifPermissionAlertInfo to lib
ClosedPublic

Authored by michal on Mar 10 2023, 7:28 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Mar 26, 2:19 PM
Unknown Object (File)
Wed, Mar 13, 1:58 AM
Unknown Object (File)
Feb 22 2024, 12:10 AM
Unknown Object (File)
Feb 20 2024, 11:49 PM
Unknown Object (File)
Feb 20 2024, 9:52 PM
Unknown Object (File)
Feb 20 2024, 9:52 PM
Unknown Object (File)
Feb 19 2024, 9:40 AM
Unknown Object (File)
Feb 19 2024, 4:46 AM
Subscribers

Details

Summary

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

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
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