[lib] Fix up client ThreadStore validation
ClosedPublic
Actions

Authored by ashoat on Wed, Jun 19, 5:16 PM.

Details

Reviewers

kamil
marcin

Commits

rCOMMf006ec332ed1: [lib] Fix up client ThreadStore validation

Summary

I noticed we use tcomb validators on the client side here, so I figured it would be good to fix them before we insert any ThickRawThreadInfos into SQLite.

Depends on D12495

Test Plan

Flow

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

ashoat created this revision.Wed, Jun 19, 5:16 PM

Herald added a subscriber: tomek. · View Herald TranscriptWed, Jun 19, 5:16 PM

ashoat mentioned this in D12495: [lib] Introduce RawThickThreadInfo type.Wed, Jun 19, 5:26 PM

Harbormaster completed remote builds in B29800: Diff 41532.Wed, Jun 19, 6:32 PM

ashoat requested review of this revision.Wed, Jun 19, 6:32 PM

marcin added a child revision: D12539: Utilise subscription from ThickMemberInfo when building a notif.Fri, Jun 21, 4:46 AM

marcin accepted this revision.Mon, Jun 24, 10:20 AM

This revision is now accepted and ready to land.Mon, Jun 24, 10:20 AM

Closed by commit rCOMMf006ec332ed1: [lib] Fix up client ThreadStore validation (authored by ashoat). · Explain WhyWed, Jun 26, 11:39 AM

This revision was automatically updated to reflect the committed changes.

ashoat added a commit: rCOMMf006ec332ed1: [lib] Fix up client ThreadStore validation.

Revision Contents
Changeset List

Path

Size

lib/

permissions/

minimally-encoded-raw-thread-info-validators.js

15 lines

utils/

thread-ops-utils.js

6 lines

Diff 41733

View Options

lib/permissions/minimally-encoded-raw-thread-info-validators.js