Page MenuHomePhabricator
Differential D10295

[keyserver] Move `threadStoreValidator` to `redux-state-responders`
ClosedPublic
Actions

Authored by atul on Dec 11 2023, 12:41 PM.
Tags
None
Referenced Files
F3531282: D10295.diff
Wed, Dec 25, 7:01 AM
F3530294: D10295.id34595.diff
Wed, Dec 25, 2:04 AM
Unknown Object (File)
Tue, Dec 17, 5:30 PM
Unknown Object (File)
Nov 11 2024, 4:48 PM
Unknown Object (File)
Nov 6 2024, 1:54 PM
Unknown Object (File)
Nov 5 2024, 10:02 PM
Unknown Object (File)
Nov 5 2024, 7:33 PM
Unknown Object (File)
Nov 4 2024, 2:12 PM
View All 70 Files
Subscribers
None

Details

Reviewers
ashoat
ginsu
tomek
rohan
Commits
rCOMMb780797c5e95: [keyserver] Move `threadStoreValidator` to `redux-state-responders`
Summary

This diff is a follow-up diff to D10294.

Initially when I tried updating threadStoreValidator I had some issues with Jest... and as before it was a circular dependency issue.

Since threadStoreValidator is only being used by initialReduxStateValidator, I moved it right beside (above) it.

Depends on D10294

Test Plan

CI/flow/etc. Will become clear whether or not this validator works when we get to web.

Diff Detail

Repository
rCOMM Comm
Branch
arcpatch-D10295 (branched from master)
Lint
No Lint Coverage
Unit
No Test Coverage

Revision Contents
Changeset List

PathSize
keyserver/
src/
responders/
10 lines
lib/
types/
4 lines

Diff 34595

View Options

keyserver/src/responders/redux-state-responders.js

Loading...
View Options

lib/types/thread-types.js

Loading...