Paths

Table of Contentst

Differential D13544

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

Authored by ashoat on Oct 1 2024, 5:01 AM.

Tags

None

Referenced Files

	F32193689: D13544.1765099126.diff
	Sat, Dec 6, 9:18 AM

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

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

ashoat created this revision.Oct 1 2024, 5:01 AM

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.Oct 1 2024, 5:04 AM

Harbormaster completed remote builds in B31960: Diff 44725.Oct 1 2024, 5:52 AM

ashoat requested review of this revision.Oct 1 2024, 5:52 AM

tomek accepted this revision.Oct 1 2024, 3:07 PM

This revision is now accepted and ready to land.Oct 1 2024, 3:07 PM

Closed by commit rCOMM528961244cdc: [web] Swap order of conditions in ChatThreadComposer onSelectUserFromSearch. · Explain WhyOct 1 2024, 4:07 PM

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 44774

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

Loading...