Skip to content
Projects
Groups
Snippets
Help
Loading...
Help
Submit feedback
Contribute to GitLab
Sign in
Toggle navigation
C
canifa_note
Project
Project
Details
Activity
Releases
Cycle Analytics
Repository
Repository
Files
Commits
Branches
Tags
Contributors
Graph
Compare
Charts
Issues
0
Issues
0
List
Board
Labels
Milestones
Merge Requests
0
Merge Requests
0
CI / CD
CI / CD
Pipelines
Jobs
Schedules
Charts
Wiki
Wiki
Snippets
Snippets
Members
Members
Collapse sidebar
Close sidebar
Activity
Graph
Charts
Create a new issue
Jobs
Commits
Issue Boards
Open sidebar
Vũ Hoàng Anh
canifa_note
Commits
6eaab52e
Commit
6eaab52e
authored
Feb 22, 2026
by
Steven
Browse files
Options
Browse Files
Download
Email Patches
Plain Diff
fix(web): handle delete memo errors and only run post-delete actions on success
parent
1d9d4cd7
Changes
1
Hide whitespace changes
Inline
Side-by-side
Showing
1 changed file
with
9 additions
and
8 deletions
+9
-8
hooks.ts
web/src/components/MemoActionMenu/hooks.ts
+9
-8
No files found.
web/src/components/MemoActionMenu/hooks.ts
View file @
6eaab52e
...
...
@@ -98,15 +98,16 @@ export const useMemoActionHandlers = ({ memo, onEdit, setDeleteDialogOpen }: Use
},
[
setDeleteDialogOpen
]);
const
confirmDeleteMemo
=
useCallback
(
async
()
=>
{
await
deleteMemo
(
memo
.
name
,
{
onSuccess
:
()
=>
{
// If this was a comment, refresh the parent memo's comments list so it disappears from the UI
if
(
memo
.
parent
)
{
queryClient
.
invalidateQueries
({
queryKey
:
memoKeys
.
comments
(
memo
.
parent
)
});
}
},
});
try
{
await
deleteMemo
(
memo
.
name
);
}
catch
(
error
:
unknown
)
{
handleError
(
error
,
toast
.
error
,
{
context
:
"Delete memo"
,
fallbackMessage
:
"An error occurred"
});
return
;
}
toast
.
success
(
t
(
"message.deleted-successfully"
));
if
(
memo
.
parent
)
{
queryClient
.
invalidateQueries
({
queryKey
:
memoKeys
.
comments
(
memo
.
parent
)
});
}
if
(
isInMemoDetailPage
)
{
navigateTo
(
"/"
);
}
...
...
Write
Preview
Markdown
is supported
0%
Try again
or
attach a new file
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment