Check if corpus deletion succeeds, fix #140

CHANGELOG.md

@@ -27,6 +27,10 @@ As this project is a user-facing application, the places in the semantic version
 - `RouteButton` component replaces `router-link > ActionButton`
 - `ActionButton` uses `<button>` and is tabbable
 
+### Fixed
+
+- Error handling when deleting corpus
+
 ### Removed
 
 - `useVariant`

src/corpus/CorpusDelete.vue

@@ -6,14 +6,18 @@ import RouteButton from "@/components/RouteButton.vue";
 import Section from "@/components/Section.vue";
 import PendingContent from "@/spin/PendingContent.vue";
 import useCorpus from "./corpus.composable";
+import { useCorpusStore } from "@/store/corpus.store";
 
 const router = useRouter();
 const corpusId = useCorpusIdParam();
 const { deleteCorpus } = useCorpus(corpusId);
+const corpusStore = useCorpusStore();
 
 async function doDelete() {
   await deleteCorpus();
-  router.push("/corpus");
+  if (!(corpusId in corpusStore.corpora)) {
+    router.push("/corpus");
+  }
 }
 </script>
 

src/corpus/corpus.composable.js

@@ -12,7 +12,7 @@ export default function useCorpus(corpusId) {
   const corpusStore = useCorpusStore();
   const { refreshJwt } = useAuth();
   const mink = useMinkBackend();
-  const { loadCorpora } = useCorpora();
+  const { loadCorpora, refreshCorpora } = useCorpora();
   const { alertError } = useMessenger();
   const { loadConfig } = useConfig(corpusId);
 
@@ -39,7 +39,7 @@ export default function useCorpus(corpusId) {
     // The backend will have updated the remote JWT, so refresh our copy.
     // The backend uses the corpus list within it when listing available corpora.
     await refreshJwt();
-    corpusStore.removeCorpus(corpusId_);
+    await refreshCorpora();
   }
 
   return { loadCorpus, deleteCorpus };