Skip to content

Commit 9ac4b16

Browse files
committed
use a callback here
1 parent bc61866 commit 9ac4b16

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

static/app/views/issueDetails/streamline/hooks/useCopyIssueDetails.tsx

+5-4
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
import {useCallback} from 'react';
12
import * as Sentry from '@sentry/react';
23

34
import {addErrorMessage, addSuccessMessage} from 'sentry/actionCreators/indicator';
@@ -134,7 +135,7 @@ export const useCopyIssueDetails = (group: Group, event?: Event) => {
134135
const {data: groupSummaryData} = useGroupSummaryData(group, event);
135136
const {data: autofixData} = useAutofixData({groupId: group.id});
136137

137-
const copyIssueDetails = () => {
138+
const copyIssueDetails = useCallback(() => {
138139
if (!event) {
139140
addErrorMessage(t('Could not copy issue to clipboard'));
140141
return;
@@ -151,16 +152,16 @@ export const useCopyIssueDetails = (group: Group, event?: Event) => {
151152
Sentry.captureException(err);
152153
addErrorMessage(t('Could not copy issue to clipboard'));
153154
});
154-
};
155+
}, [group, event, groupSummaryData, autofixData]);
155156

156157
useHotkeys([
157158
{
158159
match: 'command+alt+c',
159-
callback: () => copyIssueDetails(),
160+
callback: copyIssueDetails,
160161
},
161162
{
162163
match: 'ctrl+alt+c',
163-
callback: () => copyIssueDetails(),
164+
callback: copyIssueDetails,
164165
},
165166
]);
166167

0 commit comments

Comments
 (0)