HomePhabricator
Diffusion Comm 05921b8302a9

[lib/native/web] refactor notif permission alert info reducer

Description

[lib/native/web] refactor notif permission alert info reducer

Summary:
This diff refactors the notif permission alert info reducer => alert info reducer. Aside from just renaming the reducer, the reducer now returns an AlertStore instead of just an AlertInfo

Please note this diff + subsequent diffs won't be landed until the migration diff which will be introduced here shortly

Linear task: https://linear.app/comm/issue/ENG-7619/rename-all-the-notifpermissionalertinfo-⇒-alertinfo + https://linear.app/comm/issue/ENG-7621/refactor-reducenotifpermissionalertinfo-to-handle-a-mapping-of-alert

Depends on D11567

Test Plan:
flow + confirmed the redux state using the redux dev tools and that the existing push notif permission alert still functions as expected

web:

Screenshot 2024-04-07 at 11.42.37 PM.png (288×1 px, 63 KB)

native:

Screenshot 2024-04-07 at 11.49.24 PM.png (110×644 px, 26 KB)

Reviewers: atul, inka

Reviewed By: atul

Subscribers: ashoat, tomek

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

Details

Provenance
ginsuAuthored on Apr 5 2024, 9:45 PM
Reviewer
atul
Differential Revision
D11568: [lib/native/web] refactor notif permission alert info reducer
Parents
rCOMM1d328ce5d786: [lib] remove reference to nonexistent keyserver endpoint
Branches
Unknown
Tags
Unknown