Improve error handling versions / history

browser/data-browser/src/routes/History/useVersions.ts

@@ -1,6 +1,7 @@
 import { Resource, Version, useStore } from '@tomic/react';
 import { useState, useEffect } from 'react';
 import { dedupeVersions } from './versionHelpers';
+import { handleError } from '../../helpers/loggingHandlers';
 
 export interface UseVersionsResult {
   versions: Version[];
@@ -21,6 +22,7 @@ export function useVersions(resource: Resource): UseVersionsResult {
         setVersions(dedupeVersions(history));
       })
       .catch(e => {
+        handleError(e);
         setError(e);
       })
       .finally(() => {

browser/lib/src/resource.ts

@@ -215,12 +215,24 @@ export class Resource {
     const commitsCollection = await store.fetchResourceFromServer(
       this.getCommitsCollection(),
     );
+
+    if (commitsCollection.error) {
+      throw commitsCollection.error;
+    }
+
     const commits = commitsCollection.get(properties.collection.members);
 
     const builtVersions: Version[] = [];
 
     let previousResource = new Resource(this.subject);
 
+    if (!commits) {
+      throw new Error(
+        `Couldn't find commits for ${this.getSubject()} in CommitCollection: ` +
+          this.getCommitsCollection(),
+      );
+    }
+
     for (const commit of commits as unknown as string[]) {
       const commitResource = await store.getResourceAsync(commit);
       const parsedCommit = parseCommitResource(commitResource);