Page MenuHomePhabricator
Differential D8645

[lib, web] Use sessionId from keyserverStore
ClosedPublic
Actions

Authored by inka on Jul 27 2023, 8:16 AM.
Tags
None
Referenced Files
F2196764: D8645.id29479.diff
Fri, Jul 5, 1:29 PM
F2196619: D8645.diff
Fri, Jul 5, 1:10 PM
F2196446: D8645.id29139.diff
Fri, Jul 5, 12:42 PM
Unknown Object (File)
Thu, Jul 4, 9:22 AM
Unknown Object (File)
Wed, Jul 3, 3:42 AM
Unknown Object (File)
Tue, Jul 2, 7:10 PM
Unknown Object (File)
Fri, Jun 21, 10:26 PM
Unknown Object (File)
Tue, Jun 18, 10:06 PM
View All 60 Files
Subscribers
ashoat
tomek

Details

Reviewers
michal
kamil
Commits
rCOMM01332e50f17b: [lib, web] Use sessionId from keyserverStore
Summary

issue: https://linear.app/comm/issue/ENG-4457/refactor-sessionid-field
Start reducing sessionID in keyserverStore and using it in places where we were using sessionID redux field.

Test Plan

Ran yarn flow-all. Checked that sessionID field in keyserverStore has the same value as sessionID redux field when user loges out,
logs in.

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Revision Contents
Changeset List

PathSize
lib/
selectors/
4 lines
5 lines
4 lines
shared/
7 lines
web/
redux/
8 lines
selectors/
3 lines

Diff 29139

View Options

lib/selectors/account-selectors.js

Loading...
View Options

lib/selectors/keyserver-selectors.js

Loading...
View Options

lib/selectors/server-calls.js

Loading...
View Options

lib/shared/session-utils.js

Loading...
View Options

web/redux/redux-setup.js

Loading...
View Options

web/selectors/socket-selectors.js

Loading...