This addresses https://linear.app/comm/issue/ENG-9971/[error-invalid-state-err]-in-identitysearchprovider
We should be using identitySearchSocket instead of socket.current could be null before we properly set it
Paths
| Differential D14094 Authored by will on Dec 7 2024, 11:14 AM.
Details Summary This addresses https://linear.app/comm/issue/ENG-9971/[error-invalid-state-err]-in-identitysearchprovider We should be using identitySearchSocket instead of socket.current could be null before we properly set it Test Plan cargo check Didn't get to testing this properly. Putting up this diff for now to get it into review, but I'll ensure we're not getting spammed with: ERROR [Error: INVALID_STATE_ERR] before landing
Diff Detail
Event TimelineHerald added subscribers: tomek, ashoat. · View Herald TranscriptDec 7 2024, 11:14 AM2024-12-07 11:14:17 (UTC-8) will edited the test plan for this revision. (Show Details)Dec 7 2024, 11:14 AM2024-12-07 11:14:34 (UTC-8) Harbormaster completed remote builds in B32962: Diff 46257.Dec 7 2024, 11:32 AM2024-12-07 11:32:01 (UTC-8) This revision is now accepted and ready to land.Dec 7 2024, 12:36 PM2024-12-07 12:36:13 (UTC-8) Closed by commit rCOMM379d8575a713: [identity] use identitySearchSocket instead of socket.current (authored by will). · Explain WhyDec 7 2024, 12:55 PM2024-12-07 12:55:39 (UTC-8) This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 46257 lib/identity-search/identity-search-context.js
|