Paths

Table of Contentst

Differential D13544

[web] Swap order of conditions in ChatThreadComposer onSelectUserFromSearch
ClosedPublic
Actions

Authored by ashoat on Mon, Sep 30, 10:01 PM.

Tags

None

Referenced Files

	F2860969: D13544.id.diff
	Tue, Oct 1, 8:38 PM

	F2860567: D13544.diff
	Tue, Oct 1, 6:24 PM

	F2860419: D13544.id44725.diff
	Tue, Oct 1, 5:43 PM

	F2860263: D13544.id44725.diff
	Tue, Oct 1, 4:51 PM

	F2859961: D13544.id44774.diff
	Tue, Oct 1, 3:10 PM

	F2853623: D13544.id44725.diff
	Mon, Sep 30, 10:29 PM

	F2853526: D13544.id.diff
	Mon, Sep 30, 10:27 PM

	F2853351: D13544.diff
	Mon, Sep 30, 10:11 PM

Subscribers

None

Details

Reviewers

tomek
kamil
inka

Commits

rCOMM528961244cdc: [web] Swap order of conditions in ChatThreadComposer onSelectUserFromSearch

Summary

This is safe because when sortedMembers in usePotentialMemberItems sets notFriendNotice, it also sets an alert.

Depends on D13543

Test Plan

Careful reading of the code indicates that this is a no-op

Diff Detail

Repository

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

ashoat created this revision.Mon, Sep 30, 10:01 PM

ashoat mentioned this in D13545: [native][web] Go to PRIVATE chat when the user selects themselves in chat composer.

ashoat added a child revision: D13545: [native][web] Go to PRIVATE chat when the user selects themselves in chat composer.Mon, Sep 30, 10:04 PM

Harbormaster completed remote builds in B31960: Diff 44725.Mon, Sep 30, 10:52 PM

ashoat requested review of this revision.Mon, Sep 30, 10:52 PM

tomek accepted this revision.Tue, Oct 1, 8:07 AM

This revision is now accepted and ready to land.Tue, Oct 1, 8:07 AM

Closed by commit rCOMM528961244cdc: [web] Swap order of conditions in ChatThreadComposer onSelectUserFromSearch (authored by ashoat). · Explain WhyTue, Oct 1, 9:07 AM

This revision was automatically updated to reflect the committed changes.

ashoat added a commit: rCOMM528961244cdc: [web] Swap order of conditions in ChatThreadComposer onSelectUserFromSearch.

Revision Contents
Changeset List

Path

Size

web/

chat/

chat-thread-composer.react.js

19 lines

Diff 44725

web/chat/chat-thread-composer.react.js

Loading...