Page MenuHomePhabricator

[lib] Move `messageTypes` enum to a separate file
ClosedPublic

Authored by michal on Apr 21 2023, 7:57 AM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Apr 15, 3:39 PM
Unknown Object (File)
Fri, Apr 12, 4:21 PM
Unknown Object (File)
Wed, Apr 10, 10:24 AM
Unknown Object (File)
Fri, Apr 5, 5:32 AM
Unknown Object (File)
Fri, Apr 5, 5:32 AM
Unknown Object (File)
Fri, Apr 5, 5:32 AM
Unknown Object (File)
Fri, Apr 5, 5:31 AM
Unknown Object (File)
Fri, Apr 5, 5:18 AM
Subscribers

Details

Summary

No logic changes. This is done to avoid ayclic imports between message-types and files in types/messages after adding the validators

Test Plan

Run app on web and ios, check if they throw any unexpected errors.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Makes sense! I wonder if we should put the new file in lib/types/messages, but I don't feel strongly

This revision is now accepted and ready to land.Apr 25 2023, 3:14 AM