Page MenuHomePhabricator
Differential D11567

[lib/native/web] introduce alert types
ClosedPublic
Actions

Authored by ginsu on Sun, Apr 7, 8:21 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Apr 23, 3:07 AM
Unknown Object (File)
Mon, Apr 22, 5:23 PM
Unknown Object (File)
Sun, Apr 21, 7:10 AM
Unknown Object (File)
Tue, Apr 16, 2:04 PM
Unknown Object (File)
Sun, Apr 14, 6:43 PM
Unknown Object (File)
Sun, Apr 14, 12:23 PM
Unknown Object (File)
Sat, Apr 13, 6:44 AM
Unknown Object (File)
Fri, Apr 12, 9:41 AM
View All 12 Files
Subscribers
ashoat
tomek

Details

Reviewers
atul
inka
Commits
rCOMM9e6a658b2b6c: [lib/native/web] introduce alert types
Summary

This diff introduces the alert info + alert store types and starts to refactor + rename notifPermissionAlertInfo => alertInfo.

Please note that some of the renaming is not done yet. This is because we also need to do some refactoring to turn the notif permission alert info reducer to be the alert info reducer before we can change the variable names in some instances.

Linear task: https://linear.app/comm/issue/ENG-7620/introduce-alertinfos-alertinfostore-type

Depends on D11598

Test Plan

flow

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

ginsu created this revision.Sun, Apr 7, 8:21 PM
Herald added subscribers: tomek, ashoat. · View Herald TranscriptSun, Apr 7, 8:21 PM
Harbormaster returned this revision to the author for changes because remote builds failed.Sun, Apr 7, 8:23 PM
Harbormaster failed remote builds in B28033: Diff 38879!
ginsu requested review of this revision.Sun, Apr 7, 8:24 PM
ginsu edited the summary of this revision. (Show Details)
ginsu added reviewers: atul, inka.

will make sure ci passes before landing

ginsu added a child revision: D11568: [lib/native/web] refactor notif permission alert info reducer.Sun, Apr 7, 8:40 PM
atul accepted this revision.Mon, Apr 8, 6:18 AM
atul added inline comments.
lib/reducers/notif-permission-alert-info-reducer.js
7 ↗(On Diff #38879)

Should we also change name of reduce* function? What about the filename?

This revision is now accepted and ready to land.Mon, Apr 8, 6:18 AM
atul added inline comments.Mon, Apr 8, 6:25 AM
lib/reducers/notif-permission-alert-info-reducer.js
7 ↗(On Diff #38879)

Actually makes sense that you're just handling types in this diff.

ginsu edited the summary of this revision. (Show Details)Tue, Apr 9, 10:16 AM
ginsu added a parent revision: D11598: [native] handle successful connection from connect farcaster bottom sheet.
ginsu updated this revision to Diff 38990.Wed, Apr 10, 10:54 AM

rebase before landing + make sure ci passes

Harbormaster completed remote builds in B28130: Diff 38990.Wed, Apr 10, 11:09 AM
Closed by commit rCOMM9e6a658b2b6c: [lib/native/web] introduce alert types (authored by ginsu). · Explain WhyWed, Apr 10, 11:16 AM
This revision was automatically updated to reflect the committed changes.
ginsu added a commit: rCOMM9e6a658b2b6c: [lib/native/web] introduce alert types.

Revision Contents
Changeset List

PathSize
lib/
reducers/
10 lines
types/
27 lines
4 lines
utils/
20 lines
4 lines
native/
push/
4 lines
redux/
4 lines
4 lines
4 lines
web/
redux/
4 lines
4 lines

Diff 38994

View Options

lib/reducers/notif-permission-alert-info-reducer.js

Loading...
View Options

lib/types/alert-types.js

Loading...
View Options

lib/types/redux-types.js

Loading...
View Options

lib/utils/push-alerts.js

Loading...
View Options

lib/utils/reducers-utils.test.js

Loading...
View Options

native/push/push-handler.react.js

Loading...
View Options

native/redux/default-state.js

Loading...
View Options

native/redux/persist.js

Loading...
View Options

native/redux/state-types.js

Loading...
View Options

web/redux/default-state.js

Loading...
View Options

web/redux/redux-setup.js

Loading...