Page MenuHomePhabricator

Rename `MinimallyEncodedThreadCurrentUserInfo` to `ThreadCurrentUserInfo`
ClosedPublic

Authored by atul on Jan 24 2024, 9:23 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Nov 23, 5:45 AM
Unknown Object (File)
Fri, Nov 22, 11:46 PM
Unknown Object (File)
Thu, Nov 7, 4:00 AM
Unknown Object (File)
Tue, Oct 29, 5:03 PM
Unknown Object (File)
Oct 15 2024, 10:59 PM
Unknown Object (File)
Oct 15 2024, 10:59 PM
Unknown Object (File)
Oct 15 2024, 10:59 PM
Unknown Object (File)
Oct 15 2024, 10:59 PM
Subscribers
None

Details

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

atul requested review of this revision.Jan 24 2024, 9:41 PM
tomek added inline comments.
lib/permissions/minimally-encoded-thread-permissions.test.js
10–11 ↗(On Diff #36116)

There are no longer any MinimallyEncoded* types in the codebase.

Shouldn't we also have no minimallyEncoded* validators in that case?

This revision is now accepted and ready to land.Jan 25 2024, 2:45 AM
lib/permissions/minimally-encoded-thread-permissions.test.js
10–11 ↗(On Diff #36116)

Yup, those get removed in subsequent diff.