Page MenuHomePhabricator

[lib] Split thread-types
ClosedPublic

Authored by michal on May 10 2023, 9:49 AM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Apr 15, 12:11 AM
Unknown Object (File)
Sat, Apr 13, 12:05 AM
Unknown Object (File)
Wed, Apr 10, 11:59 PM
Unknown Object (File)
Tue, Apr 9, 12:30 AM
Unknown Object (File)
Fri, Apr 5, 5:00 AM
Unknown Object (File)
Fri, Apr 5, 5:00 AM
Unknown Object (File)
Fri, Apr 5, 5:00 AM
Unknown Object (File)
Fri, Apr 5, 5:00 AM
Subscribers

Details

Summary

Splits thread-types.js into two files: thread-permission-types.js and `thread-types-enum.js. It's done for the same reason as when we extracted the message enum - to remove a dependency cycle (that would be otherwise introduced in the next diffs).

There's one small additional change: validators in thread-types-enum.js now have to be explicitly annotated (because they are now imported across files), and this required changing the flow type of tcomb.dict() function to accept string literal unions/enums and not just strings.

Depends on D7749

Test Plan

Run yarn jest-all and yarn flow-all. Check if web and landing work.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Remove unnecessary changes

Accepting to save you rebasing

lib/types/thread-permission-types.js
1 ↗(On Diff #26358)

Add a newline here

This revision is now accepted and ready to land.May 12 2023, 2:12 AM