Page MenuHomePhabricator
Differential D10462

[lib] Fix typo in useMentionTypeaheadUserSuggestions
ClosedPublic
Actions

Authored by rohan on Dec 26 2023, 11:59 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Apr 5, 2:14 PM
Unknown Object (File)
Sat, Apr 5, 1:29 AM
Unknown Object (File)
Fri, Apr 4, 11:37 PM
Unknown Object (File)
Fri, Apr 4, 2:33 PM
Unknown Object (File)
Thu, Apr 3, 10:56 PM
Unknown Object (File)
Mar 11 2025, 7:57 PM
Unknown Object (File)
Mar 4 2025, 4:27 PM
Unknown Object (File)
Mar 3 2025, 4:16 PM
View All 80 Files
Subscribers
tomek

Details

Reviewers
atul
ginsu
inka
Commits
rCOMMa37fd35408a6: [lib] Fix typo in useMentionTypeaheadUserSuggestions
Summary

Small typo I noticed that I introduced

Test Plan

flow just to make sure I didn't miss an instance

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Revision Contents
Changeset List

PathSize
lib/
shared/
6 lines

Diff 35020

View Options

lib/shared/mention-utils.js

// @flow // @flow
import { oldValidUsernameRegexString } from './account-utils.js'; import { oldValidUsernameRegexString } from './account-utils.js';
import SentencePrefixSearchIndex from './sentence-prefix-search-index.js'; import SentencePrefixSearchIndex from './sentence-prefix-search-index.js';
import { threadOtherMembers } from './thread-utils.js'; import { threadOtherMembers } from './thread-utils.js';
import { stringForUserExplicit } from './user-utils.js'; import { stringForUserExplicit } from './user-utils.js';
import type { import type {
MinimallyEncodedRelativeMemberInfo, MinimallyEncodedRelativeMemberInfo,
MinimallyEncodedResolvedThreadInfo, MinimallyEncodedResolvedThreadInfo,
MinimallyEncodedThreadInfo, MinimallyEncodedThreadInfo,
} from '../types/minimally-encoded-thread-permissions-types.js'; } from '../types/minimally-encoded-thread-permissions-types.js';
import { threadTypes } from '../types/thread-types-enum.js'; import { threadTypes } from '../types/thread-types-enum.js';
import type { import type {
ThreadInfo, ThreadInfo,
RelativeMemberInfo, LegacyRelativeMemberInfo,
ResolvedThreadInfo, ResolvedThreadInfo,
ChatMentionCandidates, ChatMentionCandidates,
} from '../types/thread-types.js'; } from '../types/thread-types.js';
import { idSchemaRegex, chatNameMaxLength } from '../utils/validation-utils.js'; import { idSchemaRegex, chatNameMaxLength } from '../utils/validation-utils.js';
export type TypeaheadMatchedStrings = { export type TypeaheadMatchedStrings = {
+textBeforeAtSymbol: string, +textBeforeAtSymbol: string,
+query: string, +query: string,
}; };
export type Selection = { export type Selection = {
+start: number, +start: number,
+end: number, +end: number,
}; };
type MentionTypeaheadUserSuggestionItem = { type MentionTypeaheadUserSuggestionItem = {
+type: 'user', +type: 'user',
+userInfo: RelativeMemberInfo | MinimallyEncodedRelativeMemberInfo, +userInfo: LegacyRelativeMemberInfo | MinimallyEncodedRelativeMemberInfo,
}; };
type MentionTypeaheadChatSuggestionItem = { type MentionTypeaheadChatSuggestionItem = {
+type: 'chat', +type: 'chat',
+threadInfo: ResolvedThreadInfo | MinimallyEncodedResolvedThreadInfo, +threadInfo: ResolvedThreadInfo | MinimallyEncodedResolvedThreadInfo,
}; };
export type MentionTypeaheadSuggestionItem = export type MentionTypeaheadSuggestionItem =
▲ Show 20 Lines • Show All 63 Lines • ▼ Show 20 Lines ) {
return text.slice(0, selection.start).match(regex); return text.slice(0, selection.start).match(regex);
} }
return null; return null;
} }
function getMentionTypeaheadUserSuggestions( function getMentionTypeaheadUserSuggestions(
userSearchIndex: SentencePrefixSearchIndex, userSearchIndex: SentencePrefixSearchIndex,
threadMembers: $ReadOnlyArray< threadMembers: $ReadOnlyArray<
RelativeMemberInfo | MinimallyEncodedRelativeMemberInfo, LegacyRelativeMemberInfo | MinimallyEncodedRelativeMemberInfo,
>, >,
viewerID: ?string, viewerID: ?string,
usernamePrefix: string, usernamePrefix: string,
): $ReadOnlyArray<MentionTypeaheadUserSuggestionItem> { ): $ReadOnlyArray<MentionTypeaheadUserSuggestionItem> {
const userIDs = userSearchIndex.getSearchResults(usernamePrefix); const userIDs = userSearchIndex.getSearchResults(usernamePrefix);
const usersInThread = threadOtherMembers(threadMembers, viewerID); const usersInThread = threadOtherMembers(threadMembers, viewerID);
return usersInThread return usersInThread
▲ Show 20 Lines • Show All 42 Lines • ▼ Show 20 Lines): {
const newSelectionStart = newText.length - newSuffixText.length + 1; const newSelectionStart = newText.length - newSuffixText.length + 1;
return { newText, newSelectionStart }; return { newText, newSelectionStart };
} }
function getUserMentionsCandidates( function getUserMentionsCandidates(
threadInfo: ThreadInfo | MinimallyEncodedThreadInfo, threadInfo: ThreadInfo | MinimallyEncodedThreadInfo,
parentThreadInfo: ?ThreadInfo | ?MinimallyEncodedThreadInfo, parentThreadInfo: ?ThreadInfo | ?MinimallyEncodedThreadInfo,
): $ReadOnlyArray<RelativeMemberInfo | MinimallyEncodedRelativeMemberInfo> { ): $ReadOnlyArray<
LegacyRelativeMemberInfo | MinimallyEncodedRelativeMemberInfo,
> {
if (threadInfo.type !== threadTypes.SIDEBAR) { if (threadInfo.type !== threadTypes.SIDEBAR) {
return threadInfo.members; return threadInfo.members;
} }
if (parentThreadInfo) { if (parentThreadInfo) {
return parentThreadInfo.members; return parentThreadInfo.members;
} }
// This scenario should not occur unless the user logs out while looking at a // This scenario should not occur unless the user logs out while looking at a
// sidebar. In that scenario, the Redux store may be cleared before ReactNav // sidebar. In that scenario, the Redux store may be cleared before ReactNav
Show All 19 Lines