Page MenuHomePhabricator
Differential D3506

[web] [refactor] simplify username logic in message preview
ClosedPublic
Actions

Authored by • benschac on Mar 23 2022, 12:33 PM.
Tags
None
Referenced Files
F6271953: D3506.diff
Fri, Apr 25, 2:59 PM
Unknown Object (File)
Thu, Apr 24, 6:22 PM
Unknown Object (File)
Sat, Apr 19, 2:39 PM
Unknown Object (File)
Sat, Apr 19, 12:32 PM
Unknown Object (File)
Sat, Apr 19, 10:05 AM
Unknown Object (File)
Sat, Apr 19, 7:56 AM
Unknown Object (File)
Sat, Apr 19, 4:36 AM
Unknown Object (File)
Sat, Apr 19, 2:50 AM
View All Files
Subscribers
• adrian
ashoat
• karol
tomek

Details

Reviewers
atul
• jacek
tomek
ashoat
Commits
rCOMM426e725ba042: [web] [refactor] simplify username logic in message preview
Summary

simplify rendering logic for username in message preview

Test Plan

username logic is the same as production. No new functionality

Diff Detail

Repository
rCOMM Comm
Branch
arcpatch-D3506
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

• benschac created this revision.Mar 23 2022, 12:33 PM
Herald added subscribers: • karol, • adrian, tomek, ashoat. · View Herald TranscriptMar 23 2022, 12:33 PM
• benschac added a parent revision: D3505: [web] [refactor] message preview styles solidified into one ternary statement.Mar 23 2022, 12:34 PM
Harbormaster completed remote builds in B7558: Diff 10629.Mar 23 2022, 12:38 PM
• benschac requested review of this revision.Mar 23 2022, 12:38 PM
• benschac updated this revision to Diff 10631.Mar 23 2022, 12:41 PM

make hasUserName const

• benschac updated this revision to Diff 10632.Mar 23 2022, 12:42 PM

remove hasUsername const

Harbormaster completed remote builds in B7560: Diff 10631.Mar 23 2022, 12:46 PM
Harbormaster completed remote builds in B7561: Diff 10632.Mar 23 2022, 12:48 PM
• benschac added a child revision: D3508: [web] [refactor] move hasUsername logic to variable.Mar 23 2022, 1:08 PM
• benschac added reviewers: • jacek, tomek.Mar 23 2022, 1:55 PM
• benschac planned changes to this revision.Mar 24 2022, 5:20 AM
• benschac added inline comments.
web/chat/message-preview.react.js
50 ↗(On Diff #10632)

This logic is wrong.

• benschac updated this revision to Diff 10665.Mar 24 2022, 9:17 AM

fix logic

Harbormaster completed remote builds in B7594: Diff 10665.Mar 24 2022, 9:22 AM
ashoat accepted this revision.Mar 24 2022, 6:02 PM
ashoat added inline comments.
web/chat/message-preview.react.js
28 ↗(On Diff #10665)

Let's move this closer to where it's used. Line 48 makes the most sense

This revision is now accepted and ready to land.Mar 24 2022, 6:02 PM
• benschac updated this revision to Diff 10684.Mar 25 2022, 7:32 AM

diff review

Harbormaster completed remote builds in B7608: Diff 10684.Mar 25 2022, 7:58 AM
• benschac updated this revision to Diff 10704.Mar 25 2022, 10:37 AM

rebase

Harbormaster completed remote builds in B7622: Diff 10704.Mar 25 2022, 10:42 AM
Closed by commit rCOMM426e725ba042: [web] [refactor] simplify username logic in message preview (authored by • benschac). · Explain WhyMar 25 2022, 10:55 AM
This revision was automatically updated to reflect the committed changes.
• benschac added a commit: rCOMM426e725ba042: [web] [refactor] simplify username logic in message preview.

Revision Contents
Changeset List

PathSize
web/
chat/
38 lines

Diff 10704

View Options

web/chat/message-preview.react.js

// @flow // @flow
import { messageTypes, type MessageType } from '../../types/message-types'; import { messageTypes, type MessageType } from '../../types/message-types';
import { addMembersMessageSpec } from './add-members-message-spec'; import { addMembersMessageSpec } from './add-members-message-spec';
import { changeRoleMessageSpec } from './change-role-message-spec'; import { changeRoleMessageSpec } from './change-role-message-spec';
import { changeSettingsMessageSpec } from './change-settings-message-spec'; import { changeSettingsMessageSpec } from './change-settings-message-spec';
import { createEntryMessageSpec } from './create-entry-message-spec'; import { createEntryMessageSpec } from './create-entry-message-spec';
import { createSidebarMessageSpec } from './create-sidebar-message-spec'; import { createSidebarMessageSpec } from './create-sidebar-message-spec';
import { createSubThreadMessageSpec } from './create-sub-thread-message-spec'; import { createSubThreadMessageSpec } from './create-sub-thread-message-spec';
import { createThreadMessageSpec } from './create-thread-message-spec'; import { createThreadMessageSpec } from './create-thread-message-spec';
import { deleteEntryMessageSpec } from './delete-entry-message-spec'; import { deleteEntryMessageSpec } from './delete-entry-message-spec';
import { editEntryMessageSpec } from './edit-entry-message-spec'; import { editEntryMessageSpec } from './edit-entry-message-spec';
import { joinThreadMessageSpec } from './join-thread-message-spec'; import { joinThreadMessageSpec } from './join-thread-message-spec';
import { leaveThreadMessageSpec } from './leave-thread-message-spec'; import { leaveThreadMessageSpec } from './leave-thread-message-spec';
import { type MessageSpec } from './message-spec'; import { type MessageSpec } from './message-spec';
import { multimediaMessageSpec } from './multimedia-message-spec'; import { multimediaMessageSpec } from './multimedia-message-spec';
import { reactionMessageSpec } from './reaction-message-spec';
import { removeMembersMessageSpec } from './remove-members-message-spec'; import { removeMembersMessageSpec } from './remove-members-message-spec';
import { restoreEntryMessageSpec } from './restore-entry-message-spec'; import { restoreEntryMessageSpec } from './restore-entry-message-spec';
import { sidebarSourceMessageSpec } from './sidebar-source-message-spec'; import { sidebarSourceMessageSpec } from './sidebar-source-message-spec';
import { textMessageSpec } from './text-message-spec'; import { textMessageSpec } from './text-message-spec';
import { unsupportedMessageSpec } from './unsupported-message-spec'; import { unsupportedMessageSpec } from './unsupported-message-spec';
import { updateRelationshipMessageSpec } from './update-relationship-message-spec'; import { updateRelationshipMessageSpec } from './update-relationship-message-spec';
export const messageSpecs: { export const messageSpecs: {
Show All 13 Linesexport const messageSpecs: {
[messageTypes.DELETE_ENTRY]: deleteEntryMessageSpec, [messageTypes.DELETE_ENTRY]: deleteEntryMessageSpec,
[messageTypes.RESTORE_ENTRY]: restoreEntryMessageSpec, [messageTypes.RESTORE_ENTRY]: restoreEntryMessageSpec,
[messageTypes.UNSUPPORTED]: unsupportedMessageSpec, [messageTypes.UNSUPPORTED]: unsupportedMessageSpec,
[messageTypes.IMAGES]: multimediaMessageSpec, [messageTypes.IMAGES]: multimediaMessageSpec,
[messageTypes.MULTIMEDIA]: multimediaMessageSpec, [messageTypes.MULTIMEDIA]: multimediaMessageSpec,
[messageTypes.UPDATE_RELATIONSHIP]: updateRelationshipMessageSpec, [messageTypes.UPDATE_RELATIONSHIP]: updateRelationshipMessageSpec,
[messageTypes.SIDEBAR_SOURCE]: sidebarSourceMessageSpec, [messageTypes.SIDEBAR_SOURCE]: sidebarSourceMessageSpec,
[messageTypes.CREATE_SIDEBAR]: createSidebarMessageSpec, [messageTypes.CREATE_SIDEBAR]: createSidebarMessageSpec,
[messageTypes.REACTION]: reactionMessageSpec,
}); });