Page MenuHomePhabricator
Differential D10654

[lib] Remove instances of `LegacyThreadInfo` from `DrawerUtils`
ClosedPublic
Actions

Authored by atul on Jan 16 2024, 12:55 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Mar 23, 9:35 PM
Unknown Object (File)
Sun, Mar 23, 9:35 PM
Unknown Object (File)
Sun, Mar 23, 9:34 PM
Unknown Object (File)
Fri, Mar 21, 3:33 AM
Unknown Object (File)
Wed, Mar 19, 7:25 PM
Unknown Object (File)
Mon, Mar 17, 12:34 PM
Unknown Object (File)
Mar 1 2025, 6:02 AM
Unknown Object (File)
Feb 21 2025, 10:55 PM
View All 86 Files
Subscribers
None

Details

Reviewers
ashoat
ginsu
tomek
Commits
rCOMMafd1a3c40871: [lib] Remove instances of `LegacyThreadInfo` from `DrawerUtils`
Summary

Part of https://linear.app/comm/issue/ENG-6366/rename-minimallyencodedrawthreadinfo-to-rawthreadinfo

Prerequisite to deprecating LegacyResolvedThreadInfo which we should be able to remove completely since it's entirely a client "UI type."

Depends on D10646

Test Plan

flow

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Revision Contents
Changeset List

PathSize
lib/
utils/
6 lines

Diff 36019

View Options

lib/utils/drawer-utils.react.js

Show All 14 Linesimport {
updateCalendarQueryActionTypes, updateCalendarQueryActionTypes,
} from 'lib/actions/entry-actions.js'; } from 'lib/actions/entry-actions.js';
import { ChatMentionContextProvider } from 'lib/components/chat-mention-provider.react.js'; import { ChatMentionContextProvider } from 'lib/components/chat-mention-provider.react.js';
import { EditUserAvatarProvider } from 'lib/components/edit-user-avatar-provider.react.js'; import { EditUserAvatarProvider } from 'lib/components/edit-user-avatar-provider.react.js';
import { import {
ModalProvider, ModalProvider,
useModalContext, useModalContext,
} from 'lib/components/modal-provider.react.js'; } from 'lib/components/modal-provider.react.js';
import { StaffContextProvider } from 'lib/components/staff-provider.react.js';
import { import {
createLoadingStatusSelector, createLoadingStatusSelector,
combineLoadingStatuses, combineLoadingStatuses,
} from 'lib/selectors/loading-selectors.js'; } from 'lib/selectors/loading-selectors.js';
import { isLoggedIn } from 'lib/selectors/user-selectors.js'; import { isLoggedIn } from 'lib/selectors/user-selectors.js';
import { extractMajorDesktopVersion } from 'lib/shared/version-utils.js'; import { extractMajorDesktopVersion } from 'lib/shared/version-utils.js';
import { TunnelbrokerProvider } from 'lib/tunnelbroker/tunnelbroker-context.js'; import { TunnelbrokerProvider } from 'lib/tunnelbroker/tunnelbroker-context.js';
import type { LoadingStatus } from 'lib/types/loading-types.js'; import type { LoadingStatus } from 'lib/types/loading-types.js';
▲ Show 20 Lines • Show All 153 Lines • ▼ Show 20 Linesclass App extends React.PureComponent<Props> {
render(): React.Node { render(): React.Node {
let content; let content;
if (this.props.loggedIn) { if (this.props.loggedIn) {
content = ( content = (
<> <>
<WebEditThreadAvatarProvider> <WebEditThreadAvatarProvider>
<EditUserAvatarProvider> <EditUserAvatarProvider>
<StaffContextProvider>
{this.renderMainContent()} {this.renderMainContent()}
{this.props.modals} {this.props.modals}
</StaffContextProvider>
</EditUserAvatarProvider> </EditUserAvatarProvider>
</WebEditThreadAvatarProvider> </WebEditThreadAvatarProvider>
</> </>
); );
} else { } else {
content = ( content = (
<> <>
{this.renderLoginPage()} {this.renderLoginPage()}
▲ Show 20 Lines • Show All 223 Lines • Show Last 20 Lines