Paths

Table of Contentst

Differential D13338

[web] Fix broken pending threads
ClosedPublic
Actions

Authored by tomek on Fri, Sep 13, 10:03 AM.

Tags

None

Referenced Files

	F2842179: D13338.diff
	Sun, Sep 29, 12:15 PM

	F2837327: D13338.id.diff
	Sat, Sep 28, 6:26 PM

	Unknown Object (File)
	Sat, Sep 28, 1:12 PM

	Unknown Object (File)
	Sat, Sep 28, 12:12 AM

	Unknown Object (File)
	Fri, Sep 27, 12:30 PM

	Unknown Object (File)
	Thu, Sep 26, 12:51 PM

	Unknown Object (File)
	Sat, Sep 21, 8:57 PM

	Unknown Object (File)
	Sat, Sep 21, 4:03 PM

View All 25 Files

Subscribers

None

Details

Reviewers

kamil
bartek
ashoat

Commits

rCOMMc5f34caf9676: [web] Fix broken pending threads

Summary

In D13282 we modified the logic so that instead of an active thread ID we use active thread info that is taken from the thread store. Unfortunately, this change broke the pending threads which aren't present in the store. What we should do instead is to let the component provide the thread store for us.

https://linear.app/comm/issue/ENG-9268/pending-threads-are-broken-on-master-on-the-web

Test Plan

Check if it is possible to create a new thread by sending a text message in a pending thread.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

tomek created this revision.Fri, Sep 13, 10:03 AM

Harbormaster completed remote builds in B31618: Diff 44189.Fri, Sep 13, 10:21 AM

tomek requested review of this revision.Fri, Sep 13, 10:21 AM

bartek accepted this revision.Fri, Sep 13, 10:33 AM

This revision is now accepted and ready to land.Fri, Sep 13, 10:33 AM

bartek added inline comments.Fri, Sep 13, 10:36 AM

web/chat/chat-input-bar.react.js
692 ↗	(On Diff #44189)	This probably can be narrowed down

ashoat accepted this revision.Fri, Sep 13, 1:36 PM

ashoat added inline comments.

web/input/input-state-container.react.js
660–663 ↗	(On Diff #44189)	We can just pass in `this.cancelPendingUpload` directly if we define it as an arrow function

Feedback

Harbormaster completed remote builds in B31619: Diff 44190.Fri, Sep 13, 1:57 PM

Closed by commit rCOMMc5f34caf9676: [web] Fix broken pending threads (authored by tomek, committed by ashoat). · Explain WhyFri, Sep 13, 2:01 PM

This revision was automatically updated to reflect the committed changes.

ashoat added a commit: rCOMMc5f34caf9676: [web] Fix broken pending threads.

Revision Contents
Changeset List

Path

Size

web/

chat/

chat-input-bar.react.js

14 lines

input/

input-state-container.react.js

47 lines

2 lines

Diff 44191

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

Loading...

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

Loading...

web/input/input-state.js

Loading...