Page MenuHomePhabricator

[lib][native][web] Make Flow types for Redux state read-only
ClosedPublic

Authored by ashoat on Jul 31 2023, 11:33 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Nov 1, 7:39 AM
Unknown Object (File)
Fri, Nov 1, 7:39 AM
Unknown Object (File)
Fri, Nov 1, 7:37 AM
Unknown Object (File)
Fri, Nov 1, 3:47 AM
Unknown Object (File)
Sep 26 2024, 5:34 PM
Unknown Object (File)
Sep 20 2024, 8:39 PM
Unknown Object (File)
Sep 20 2024, 8:39 PM
Unknown Object (File)
Sep 20 2024, 8:39 PM
Subscribers

Details

Summary

This probably should've been done a long time ago. Best practice is to modernize a Flow type whenever you touch it, but evidently people were too intimidated to try and modernize BaseAppState and its various related types.

I went ahead and did this. It looks like it went well, with no new Flow errors.

Test Plan

Flow

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage