[lib] Add `memberInfoSansPermissionsValidator` to `convertClientDBThreadInfoToRawThreadInfo`
ClosedPublic
Actions

Authored by atul on Jul 17 2024, 12:36 PM.

Details

Reviewers

ashoat
tomek

Commits

rCOMM1a4d31715739: [lib] Add `memberInfoSansPermissionsValidator` to…

Summary

Added memberInfoSansPermissionsValidator to convertClientDBThreadInfoToRawThreadInfo.

In preparation for client migrations. Moved the more "relevant" validators earlier in conditional so we're doing (almost always) useless legacyMemberInfoValidator check all the time.

Depends on D12779

Test Plan

Will be tested implicitly by client migrations. We have unit tests for memberInfoSansPermissionsValidator.

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

atul created this revision.Jul 17 2024, 12:36 PM

Harbormaster completed remote builds in B30427: Diff 42400.Jul 17 2024, 12:53 PM

atul requested review of this revision.Jul 17 2024, 12:53 PM

ashoat accepted this revision.Jul 17 2024, 1:01 PM

This revision is now accepted and ready to land.Jul 17 2024, 1:01 PM

Can also cherry-pick and land this to simplify stack before rebasing and addressing all feedback.

cherry-pick and land

This revision was landed with ongoing or failed builds.Jul 17 2024, 1:52 PM

Closed by commit rCOMM1a4d31715739: [lib] Add `memberInfoSansPermissionsValidator` to… (authored by atul). · Explain Why

This revision was automatically updated to reflect the committed changes.

atul added a commit: rCOMM1a4d31715739: [lib] Add `memberInfoSansPermissionsValidator` to….

Harbormaster completed remote builds in B30430: Diff 42404.Jul 17 2024, 2:05 PM

Revision Contents
Changeset List

Path

Size

lib/

utils/

thread-ops-utils.js

8 lines

Diff 42405

View Options

lib/utils/thread-ops-utils.js

[lib] Add `memberInfoSansPermissionsValidator` to `convertClientDBThreadInfoToRawThreadInfo`ClosedPublicActions