Elgato_dark/fleet
1
0
Fork 0
mirror of https://github.com/fleetdm/fleet synced 2026-05-23 00:49:03 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 106

Fleet UI: Fix released bug where deleting multiple pages of hosts only deleted first 50 (#12247)

Browse source
This commit is contained in:
RachelElysia 2023-06-09 12:56:09 -04:00 committed by GitHub
parent e4c38dfb7c
commit 215331cd08
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 11 additions and 15 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
changes/11885-fix-bulk-delete-hosts Normal file
View file

@ -0,0 +1 @@
Users can delete multiple pages of hosts

25
frontend/pages/hosts/ManageHostsPage/ManageHostsPage.tsx
View file

@ -1003,23 +1003,18 @@ const ManageHostsPage = ({
const onDeleteHostSubmit = async () => {
setIsUpdatingHosts(true);
let action = hostsAPI.destroyBulk(selectedHostIds);
if (isAllMatchingHostsSelected) {
const teamId = isAnyTeamSelected ? currentTeamId ?? null : null;
const labelId = selectedLabel?.id;
action = hostsAPI.destroyByFilter({
teamId,
query: searchQuery,
status,
labelId,
});
}
const teamId = isAnyTeamSelected ? currentTeamId ?? null : null;
const labelId = selectedLabel?.id;
try {
await action;
await (isAllMatchingHostsSelected
? hostsAPI.destroyByFilter({
teamId,
query: searchQuery,
status,
labelId,
})
: hostsAPI.destroyBulk(selectedHostIds));
const successMessage = `${
selectedHostIds.length === 1 ? "Host" : "Hosts"