Page MenuHomePhabricator
Differential D13320

[web][native] Update UI to not display GENESIS over thick threads
ClosedPublic
Actions

Authored by inka on Sep 13 2024, 3:11 AM.
Tags
None
Referenced Files
F5358825: D13320.diff
Wed, Apr 9, 11:57 AM
Unknown Object (File)
Wed, Apr 9, 12:25 AM
Unknown Object (File)
Tue, Apr 8, 8:49 PM
Unknown Object (File)
Tue, Apr 8, 8:48 PM
Unknown Object (File)
Tue, Apr 8, 8:33 PM
Unknown Object (File)
Tue, Apr 1, 9:16 AM
Unknown Object (File)
Tue, Apr 1, 6:09 AM
Unknown Object (File)
Tue, Apr 1, 1:05 AM
View All 72 Files
Subscribers
None

Details

Reviewers
tomek
kamil
ashoat
Commits
rCOMM529b78e6344c: [web][native] Update UI to not display GENESIS over thick threads
Summary

issue: ENG-8931

Test Plan

Please see the screenshots:

Screenshot 2024-09-13 at 12.08.33.png (704×702 px, 137 KB)

Screenshot 2024-09-13 at 12.08.12.png (662×798 px, 74 KB)

Diff Detail

Repository
rCOMM Comm
Branch
inka/GENESIS_UI
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

inka created this revision.Sep 13 2024, 3:11 AM
inka edited the test plan for this revision. (Show Details)EditedSep 13 2024, 3:12 AM
inka attached a referenced file: F2679332: Screenshot 2024-09-13 at 12.08.12.png. (Show Details)
inka attached a referenced file: F2679333: Screenshot 2024-09-13 at 12.08.33.png. (Show Details)

I changed some things compared to what we discussed on linear: I updated spacing, and changed the color of the icon on web to match breadcrumbs

inka added inline comments.Sep 13 2024, 3:16 AM
web/chat/chat-thread-list.css
297 ↗(On Diff #44124)

Text is 18px high, and has padding padding: 8px 0 2px 0;. The icon is 19px high. I wanted the text and icon to be more or less aligned

Harbormaster completed remote builds in B31587: Diff 44124.Sep 13 2024, 3:28 AM
inka requested review of this revision.Sep 13 2024, 3:28 AM
tomek accepted this revision.Sep 13 2024, 3:55 AM
tomek added inline comments.
native/chat/chat-thread-list-item.react.js
154–158 ↗(On Diff #44124)

I don't think the memoization is beneficial here.

This revision is now accepted and ready to land.Sep 13 2024, 3:55 AM
inka added a child revision: D13333: [lib] update threadLabel to handle thick threads.Sep 13 2024, 5:45 AM
inka removed a child revision: D13333: [lib] update threadLabel to handle thick threads.
inka updated this revision to Diff 44171.Sep 13 2024, 5:49 AM

Remove unecessary memos

tomek accepted this revision.Sep 13 2024, 5:56 AM
Harbormaster completed remote builds in B31611: Diff 44171.Sep 13 2024, 6:06 AM
ashoat accepted this revision.Sep 13 2024, 7:12 AM
inka added a parent revision: D13335: [lib][web][native] Refactor usingOlmViaTunnelbrokerForDMs.Sep 13 2024, 7:19 AM
inka mentioned this in D13336: [lib] Update useAncestorThreads to consider thick threads.Sep 13 2024, 7:23 AM
inka added a child revision: D13336: [lib] Update useAncestorThreads to consider thick threads.Sep 13 2024, 7:24 AM
Closed by commit rCOMM529b78e6344c: [web][native] Update UI to not display GENESIS over thick threads (authored by inka). · Explain WhySep 13 2024, 8:59 AM
This revision was automatically updated to reflect the committed changes.
inka added a commit: rCOMM529b78e6344c: [web][native] Update UI to not display GENESIS over thick threads.
inka mentioned this in rCOMM1a48c411011f: [lib] Update useAncestorThreads to consider thick threads.

Revision Contents
Changeset List

Diff 44171

View Options

native/chat/chat-thread-list-item.react.js

Show All 30 Lines
async function createOrUpdateFarcasterChannelTag( async function createOrUpdateFarcasterChannelTag(
viewer: Viewer, viewer: Viewer,
request: CreateOrUpdateFarcasterChannelTagRequest, request: CreateOrUpdateFarcasterChannelTagRequest,
): Promise<CreateOrUpdateFarcasterChannelTagResponse> { ): Promise<CreateOrUpdateFarcasterChannelTagResponse> {
if (DISABLE_TAGGING_FARCASTER_CHANNEL) { if (DISABLE_TAGGING_FARCASTER_CHANNEL) {
throw new ServerError('internal_error'); throw new ServerError('internal_error');
} }
const [serverCommunityInfos, blobDownload] = await Promise.all([ const [communityInfos, blobDownload] = await Promise.all([
fetchCommunityInfos(viewer), fetchCommunityInfos(viewer, [request.commCommunityID]),
getFarcasterChannelTagBlob(request.farcasterChannelID), getFarcasterChannelTagBlob(request.farcasterChannelID),
]); ]);
const communityInfo = serverCommunityInfos.find( if (communityInfos.length !== 1) {
community => community.id === request.commCommunityID,
);
if (!communityInfo) {
throw new ServerError('invalid_parameters'); throw new ServerError('invalid_parameters');
} }
if (blobDownload.found) { if (blobDownload.found) {
throw new ServerError('already_in_use'); throw new ServerError('already_in_use');
} }
const blobHolder = uuid.v4(); const blobHolder = uuid.v4();
▲ Show 20 Lines • Show All 107 Lines • Show Last 20 Lines
View Options

native/components/thread-ancestors-label.react.js

Show All 30 Lines
async function createOrUpdateFarcasterChannelTag( async function createOrUpdateFarcasterChannelTag(
viewer: Viewer, viewer: Viewer,
request: CreateOrUpdateFarcasterChannelTagRequest, request: CreateOrUpdateFarcasterChannelTagRequest,
): Promise<CreateOrUpdateFarcasterChannelTagResponse> { ): Promise<CreateOrUpdateFarcasterChannelTagResponse> {
if (DISABLE_TAGGING_FARCASTER_CHANNEL) { if (DISABLE_TAGGING_FARCASTER_CHANNEL) {
throw new ServerError('internal_error'); throw new ServerError('internal_error');
} }
const [serverCommunityInfos, blobDownload] = await Promise.all([ const [communityInfos, blobDownload] = await Promise.all([
fetchCommunityInfos(viewer), fetchCommunityInfos(viewer, [request.commCommunityID]),
getFarcasterChannelTagBlob(request.farcasterChannelID), getFarcasterChannelTagBlob(request.farcasterChannelID),
]); ]);
const communityInfo = serverCommunityInfos.find( if (communityInfos.length !== 1) {
community => community.id === request.commCommunityID,
);
if (!communityInfo) {
throw new ServerError('invalid_parameters'); throw new ServerError('invalid_parameters');
} }
if (blobDownload.found) { if (blobDownload.found) {
throw new ServerError('already_in_use'); throw new ServerError('already_in_use');
} }
const blobHolder = uuid.v4(); const blobHolder = uuid.v4();
▲ Show 20 Lines • Show All 107 Lines • Show Last 20 Lines
View Options

web/chat/chat-thread-list-item.react.js

Show All 30 Lines
async function createOrUpdateFarcasterChannelTag( async function createOrUpdateFarcasterChannelTag(
viewer: Viewer, viewer: Viewer,
request: CreateOrUpdateFarcasterChannelTagRequest, request: CreateOrUpdateFarcasterChannelTagRequest,
): Promise<CreateOrUpdateFarcasterChannelTagResponse> { ): Promise<CreateOrUpdateFarcasterChannelTagResponse> {
if (DISABLE_TAGGING_FARCASTER_CHANNEL) { if (DISABLE_TAGGING_FARCASTER_CHANNEL) {
throw new ServerError('internal_error'); throw new ServerError('internal_error');
} }
const [serverCommunityInfos, blobDownload] = await Promise.all([ const [communityInfos, blobDownload] = await Promise.all([
fetchCommunityInfos(viewer), fetchCommunityInfos(viewer, [request.commCommunityID]),
getFarcasterChannelTagBlob(request.farcasterChannelID), getFarcasterChannelTagBlob(request.farcasterChannelID),
]); ]);
const communityInfo = serverCommunityInfos.find( if (communityInfos.length !== 1) {
community => community.id === request.commCommunityID,
);
if (!communityInfo) {
throw new ServerError('invalid_parameters'); throw new ServerError('invalid_parameters');
} }
if (blobDownload.found) { if (blobDownload.found) {
throw new ServerError('already_in_use'); throw new ServerError('already_in_use');
} }
const blobHolder = uuid.v4(); const blobHolder = uuid.v4();
▲ Show 20 Lines • Show All 107 Lines • Show Last 20 Lines
View Options

web/chat/chat-thread-list.css

Show All 30 Lines
async function createOrUpdateFarcasterChannelTag( async function createOrUpdateFarcasterChannelTag(
viewer: Viewer, viewer: Viewer,
request: CreateOrUpdateFarcasterChannelTagRequest, request: CreateOrUpdateFarcasterChannelTagRequest,
): Promise<CreateOrUpdateFarcasterChannelTagResponse> { ): Promise<CreateOrUpdateFarcasterChannelTagResponse> {
if (DISABLE_TAGGING_FARCASTER_CHANNEL) { if (DISABLE_TAGGING_FARCASTER_CHANNEL) {
throw new ServerError('internal_error'); throw new ServerError('internal_error');
} }
const [serverCommunityInfos, blobDownload] = await Promise.all([ const [communityInfos, blobDownload] = await Promise.all([
fetchCommunityInfos(viewer), fetchCommunityInfos(viewer, [request.commCommunityID]),
getFarcasterChannelTagBlob(request.farcasterChannelID), getFarcasterChannelTagBlob(request.farcasterChannelID),
]); ]);
const communityInfo = serverCommunityInfos.find( if (communityInfos.length !== 1) {
community => community.id === request.commCommunityID,
);
if (!communityInfo) {
throw new ServerError('invalid_parameters'); throw new ServerError('invalid_parameters');
} }
if (blobDownload.found) { if (blobDownload.found) {
throw new ServerError('already_in_use'); throw new ServerError('already_in_use');
} }
const blobHolder = uuid.v4(); const blobHolder = uuid.v4();
▲ Show 20 Lines • Show All 107 Lines • Show Last 20 Lines