Page MenuHomePhabricator
Files F2906973

D3418.diff
No OneTemporary
Actions

D3418.diff
View Options

diff --git a/lib/hooks/toggle-unread-status.js b/lib/hooks/toggle-unread-status.js
new file mode 100644
--- /dev/null
+++ b/lib/hooks/toggle-unread-status.js
@@ -0,0 +1,55 @@
+// @flow
+
+import * as React from 'react';
+
+import {
+ setThreadUnreadStatus,
+ setThreadUnreadStatusActionTypes,
+} from '../actions/activity-actions';
+import type {
+ SetThreadUnreadStatusPayload,
+ SetThreadUnreadStatusRequest,
+} from '../types/activity-types';
+import type { ThreadInfo } from '../types/thread-types';
+import { useDispatchActionPromise, useServerCall } from '../utils/action-utils';
+
+function useToggleUnreadStatus(
+ threadInfo: ThreadInfo,
+ mostRecentNonLocalMessage: ?string,
+ afterAction: () => void,
+): () => void {
+ const dispatchActionPromise = useDispatchActionPromise();
+ const boundSetThreadUnreadStatus: (
+ request: SetThreadUnreadStatusRequest,
+ ) => Promise<SetThreadUnreadStatusPayload> = useServerCall(
+ setThreadUnreadStatus,
+ );
+ const toggleUnreadStatus = React.useCallback(() => {
+ const { unread } = threadInfo.currentUser;
+ const request = {
+ threadID: threadInfo.id,
+ unread: !unread,
+ latestMessage: mostRecentNonLocalMessage,
+ };
+ dispatchActionPromise(
+ setThreadUnreadStatusActionTypes,
+ boundSetThreadUnreadStatus(request),
+ undefined,
+ {
+ threadID: threadInfo.id,
+ unread: !unread,
+ },
+ );
+ afterAction();
+ }, [
+ threadInfo,
+ mostRecentNonLocalMessage,
+ dispatchActionPromise,
+ afterAction,
+ boundSetThreadUnreadStatus,
+ ]);
+
+ return toggleUnreadStatus;
+}
+
+export default useToggleUnreadStatus;
diff --git a/web/chat/chat-thread-list-item-menu.react.js b/web/chat/chat-thread-list-item-menu.react.js
--- a/web/chat/chat-thread-list-item-menu.react.js
+++ b/web/chat/chat-thread-list-item-menu.react.js
@@ -3,19 +3,8 @@
import classNames from 'classnames';
import * as React from 'react';
-import {
- setThreadUnreadStatusActionTypes,
- setThreadUnreadStatus,
-} from 'lib/actions/activity-actions';
-import type {
- SetThreadUnreadStatusPayload,
- SetThreadUnreadStatusRequest,
-} from 'lib/types/activity-types';
+import useToggleUnreadStatus from 'lib/hooks/toggle-unread-status';
import type { ThreadInfo } from 'lib/types/thread-types';
-import {
- useServerCall,
- useDispatchActionPromise,
-} from 'lib/utils/action-utils';
import SWMansionIcon from '../SWMansionIcon.react';
import css from './chat-thread-list-item-menu.css';
@@ -38,37 +27,11 @@
}, []);
const { threadInfo, mostRecentNonLocalMessage } = props;
- const dispatchActionPromise = useDispatchActionPromise();
- const boundSetThreadUnreadStatus: (
- request: SetThreadUnreadStatusRequest,
- ) => Promise<SetThreadUnreadStatusPayload> = useServerCall(
- setThreadUnreadStatus,
- );
- const toggleUnreadStatus = React.useCallback(() => {
- const { unread } = threadInfo.currentUser;
- const request = {
- threadID: threadInfo.id,
- unread: !unread,
- latestMessage: mostRecentNonLocalMessage,
- };
- dispatchActionPromise(
- setThreadUnreadStatusActionTypes,
- boundSetThreadUnreadStatus(request),
- undefined,
- {
- threadID: threadInfo.id,
- unread: !unread,
- },
- );
- hideMenu();
- }, [
+ const toggleUnreadStatus = useToggleUnreadStatus(
threadInfo,
mostRecentNonLocalMessage,
- dispatchActionPromise,
hideMenu,
- boundSetThreadUnreadStatus,
- ]);
-
+ );
const toggleUnreadStatusButtonText = `Mark as ${
threadInfo.currentUser.unread ? 'read' : 'unread'
}`;

File Metadata

Mime Type
text/plain
Expires
Mon, Oct 7, 10:22 AM (22 h, 1 m)
Storage Engine
blob
Storage Format
Raw Data
Storage Handle
2253727
Default Alt Text
D3418.diff (3 KB)

Event Timeline