Skip to content

Commit 13e1cff

Browse files
committed
Bugfix(commons): Prevent double publish causing dual success/failure modals
1 parent d71a3ec commit 13e1cff

1 file changed

Lines changed: 9 additions & 4 deletions

File tree

apps/modeling-commons-frontend/app/composables/model/useModelDraftForm.ts

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -352,10 +352,15 @@ export default function useModelDraftForm(opts: UseModelDraftFormOptions = {}) {
352352
showActionFailedToast("Missing title", "Add a title before publishing.");
353353
return null;
354354
}
355-
await patch.flush();
356-
await patch({ visibility, tags: collectTagNames(formState.value) });
357-
await patch.flush();
358-
return publish();
355+
publishing.value = true;
356+
try {
357+
await patch.flush();
358+
await patch({ visibility, tags: collectTagNames(formState.value) });
359+
await patch.flush();
360+
return await publish();
361+
} finally {
362+
publishing.value = false;
363+
}
359364
}
360365

361366
async function discard(): Promise<void> {

0 commit comments

Comments
 (0)