Skip to content

Commit

Permalink
fix(snapshots) remove workaround for fixed bug, fix notification message
Browse files Browse the repository at this point in the history
  • Loading branch information
edlerd committed Aug 29, 2023
1 parent 9e161d9 commit cbbcaa5
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 13 deletions.
13 changes: 1 addition & 12 deletions src/api/snapshots.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -59,18 +59,7 @@ export const deleteSnapshotBulk = (
)
)
.then(resolve)
.catch((e: Error) => {
// A hack to ignore this error, should be removed once fixed in
// @see https://github.com/lxc/lxd/issues/11538
const msg =
"Instance snapshot record count doesn't match instance snapshot volume record count";
const isNotError = e.toString().includes(msg);
if (isNotError) {
setTimeout(() => resolve(e), 1000);
} else {
reject(e);
}
});
.catch(reject);
});
};

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@ const SnapshotBulkDelete: FC<Props> = ({
title: "Confirm delete",
children: (
<>
This will permanently delete <b>{count}</b>
This will permanently delete <b>{count}</b>{" "}
{pluralizeSnapshot(count)}
.<br />
This action cannot be undone, and can result in data loss.
Expand Down

0 comments on commit cbbcaa5

Please sign in to comment.