[lib] Use tUserID in message-validators.js
ClosedPublic
Actions

Authored by inka on May 9 2024, 5:49 AM.

Details

Reviewers

tomek
kamil
bartek

Commits

rCOMM0f6aa7511798: [lib] Use tUserID in message-validators.js

Summary

issue: ENG-7819

Test Plan

flow
tested that fetch_messages endpoint which uses fetchMessageInfosResponseValidator which uses userInfosValidator works correctly and doesn't throw

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

inka created this revision.May 9 2024, 5:49 AM

Herald added a subscriber: ashoat. · View Herald TranscriptMay 9 2024, 5:49 AM

inka added a parent revision: D11963: [lib] Use tUserID in entry-validators.js pt.3 - serverCreateUpdatesResponseValidator.May 9 2024, 5:49 AM

inka added a child revision: D11965: [lib] Use tUserID in relationship-validators.js.May 9 2024, 6:04 AM

Harbormaster completed remote builds in B28793: Diff 39994.May 9 2024, 6:04 AM

inka requested review of this revision.May 9 2024, 6:04 AM

tomek accepted this revision.May 9 2024, 8:09 AM

tomek added inline comments.

lib/types/user-types.js
29 ↗	(On Diff #39994)	Should this be a `tUserID`?

This revision is now accepted and ready to land.May 9 2024, 8:09 AM

ashoat added inline comments.May 9 2024, 4:01 PM

lib/types/user-types.js
29 ↗	(On Diff #39994)	Looks like handled in D11966

inka edited the test plan for this revision. (Show Details)May 10 2024, 2:28 AM

Closed by commit rCOMM0f6aa7511798: [lib] Use tUserID in message-validators.js (authored by inka). · Explain WhyMay 10 2024, 2:39 AM

This revision was automatically updated to reflect the committed changes.

inka added a commit: rCOMM0f6aa7511798: [lib] Use tUserID in message-validators.js.

Revision Contents
Changeset List

Path

Size

lib/

types/

user-types.js

2 lines

Diff 40056

View Options

[lib] Use tUserID in message-validators.jsClosedPublicActions