Suppress conflict warnings for failed reservations

This is normal any time you open an asset someone else is working on. Closes #995
LibraryOfCongress · May 24, 2019 · 79b7480 · 79b7480
1 parent fc5cc5d
commit 79b7480
Showing 1 changed file with 18 additions and 12 deletions.
diff --git a/concordia/static/js/action-app/index.js b/concordia/static/js/action-app/index.js
@@ -978,20 +978,26 @@ export class ActionApp {
                 this.updateViewer();
             })
             .fail((jqXHR, textStatus, errorThrown) => {
-                console.error(
-                    'Unable to reserve asset: %s %s',
-                    textStatus,
-                    errorThrown
-                );
+                if (jqXHR.status == 409) {
+                    return;
+                } else {
+                    console.error(
+                        'Unable to reserve asset: %s %s',
+                        textStatus,
+                        errorThrown
+                    );
 
-                this.reportError(
-                    'reservation',
-                    `Unable to reserve asset`,
-                    errorThrown ? `${textStatus}: ${errorThrown}` : textStatus
-                );
+                    this.reportError(
+                        'reservation',
+                        `Unable to reserve asset`,
+                        errorThrown
+                            ? `${textStatus}: ${errorThrown}`
+                            : textStatus
+                    );
 
-                this.assetReserved = false;
-                this.updateViewer();
+                    this.assetReserved = false;
+                    this.updateViewer();
+                }
             });
     }