Page MenuHomePhabricator
Differential D9545

[web] Fix thread creation from media
ClosedPublic
Actions

Authored by ashoat on Oct 19 2023, 1:15 PM.
Tags
None
Referenced Files
F2167078: D9545.id.diff
Tue, Jul 2, 5:48 AM
Unknown Object (File)
Sat, Jun 29, 6:38 PM
Unknown Object (File)
Sat, Jun 29, 3:40 AM
Unknown Object (File)
Sat, Jun 8, 1:19 AM
Unknown Object (File)
May 28 2024, 8:16 AM
Unknown Object (File)
May 28 2024, 8:15 AM
Unknown Object (File)
May 28 2024, 8:15 AM
Unknown Object (File)
May 28 2024, 8:09 AM
View All 54 Files
Subscribers
will

Details

Reviewers
bartek
atul
tomek
Commits
rCOMMea70657bd6b7: [web] Fix thread creation from media
Summary

This solves ENG-5145, a regression introduced in D7434.

The issue is that pending threads don't appear in the ThreadStore. Instead of trying to pull the ThreadInfo from the ThreadStore, we should drill it through from the callsites.

Test Plan

I confirmed that I'm again able to create a thread from media in my local environment (the bug no longer repros)

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Revision Contents
Changeset List

Diff 32273

View Options

web/chat/chat-input-bar.react.js

Loading...
View Options

web/chat/chat-message-list-container.react.js

Loading...
View Options

web/input/input-state-container.react.js

Loading...
View Options

web/input/input-state.js

Loading...