Differential D15396

[web] remove invariant from `useThreadInfoForPossiblyPendingThread`
ClosedPublic
Actions

Authored by kamil on Sep 30 2025, 10:54 AM.

Details

Reviewers

tomek

Commits

rCOMM7486b6b048cd: [web] remove invariant from `useThreadInfoForPossiblyPendingThread`

Summary

ENG-11329.

We shouldn't use invariant here

Test Plan

Logout on web and check if app is not crashing

Diff Detail

Repository

rCOMM Comm

Branch

fixes

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

kamil created this revision.Sep 30 2025, 10:54 AM

kamil held this revision as a draft.

Herald added a subscriber: ashoat. · View Herald TranscriptSep 30 2025, 10:54 AM

kamil published this revision for review.Sep 30 2025, 10:55 AM

tomek accepted this revision.Sep 30 2025, 12:26 PM

This revision is now accepted and ready to land.Sep 30 2025, 12:26 PM

This revision was landed with ongoing or failed builds.Sep 30 2025, 1:04 PM

Closed by commit rCOMM7486b6b048cd: [web] remove invariant from `useThreadInfoForPossiblyPendingThread`. · Explain Why

This revision was automatically updated to reflect the committed changes.

kamil added a commit: rCOMM7486b6b048cd: [web] remove invariant from `useThreadInfoForPossiblyPendingThread`.

Revision Contents
Changeset List

Path

Size

web/

utils/

thread-utils.js

8 lines

Diff 50478

View Options

[web] remove invariant from `useThreadInfoForPossiblyPendingThread`ClosedPublicActions