HomePhabricator
Diffusion Comm 4d8ecee26f33

[CommCoreModule] Introduce `SocketStatus` enum

Description

[CommCoreModule] Introduce SocketStatus enum

Summary:

  1. Introduce SocketStatus type (defaulted to the same four states as readyState for now.. can always reassess later if we need to represent some other state)
  2. Add SocketStatus socketStatus to NetworkModule as a public member
  3. Add GlobalNetworkSingleton::getSocketStatus() getter to retrieve the networkModule socket status

Test Plan: Was able to call comm::GlobalNetworkSingleton::instance.getSocketStatus(); outside of a scheduleOrRun and got back expected response.

Reviewers: ashoat, def-au1t, palys-swm, karol-bisztyga

Reviewed By: ashoat, karol-bisztyga

Subscribers: Adrian, benschac

Differential Revision: https://phabricator.ashoat.com/D3066

Details

Provenance
atulAuthored on Feb 1 2022, 1:22 PM
Reviewer
ashoat
Differential Revision
Restricted Differential Revision
Parents
rCOMM8f70b079dcfe: [server] [1/n] Adding `migrations.js`
Branches
Unknown
Tags
Unknown
Reverted By
rCOMM5ab133de3dcc: Revert "[CommCoreModule] Introduce `SocketStatus` enum"