fix: unarchive action doesn't update archive page (#23987)

This commit is contained in:
Min Idzelis 2025-11-19 06:29:02 -05:00 committed by GitHub
parent a6c76e78d6
commit 2152f20b6c
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 5 additions and 6 deletions

View File

@ -607,8 +607,7 @@ test.describe('Timeline', () => {
visibility: 'timeline', visibility: 'timeline',
ids: [assetToArchive.id], ids: [assetToArchive.id],
}); });
console.log('Skipping assertion - TODO - fix bug with not removing asset from timeline-manager after unarchive'); await expect(thumbnailUtils.withAssetId(page, assetToArchive.id)).toHaveCount(0);
// await expect(thumbnail.withAssetId(page, assetToArchive.id)).toHaveCount(0);
await page.getByText('Photos', { exact: true }).click(); await page.getByText('Photos', { exact: true }).click();
await thumbnailUtils.expectInViewport(page, assetToArchive.id); await thumbnailUtils.expectInViewport(page, assetToArchive.id);
}); });

View File

@ -24,12 +24,12 @@
const { clearSelect, getOwnedAssets } = getAssetControlContext(); const { clearSelect, getOwnedAssets } = getAssetControlContext();
const handleArchive = async () => { const handleArchive = async () => {
const isArchived = unarchive ? AssetVisibility.Timeline : AssetVisibility.Archive; const visibility = unarchive ? AssetVisibility.Timeline : AssetVisibility.Archive;
const assets = [...getOwnedAssets()].filter((asset) => asset.visibility !== isArchived); const assets = [...getOwnedAssets()].filter((asset) => asset.visibility !== visibility);
loading = true; loading = true;
const ids = await archiveAssets(assets, isArchived as AssetVisibility); const ids = await archiveAssets(assets, visibility as AssetVisibility);
if (ids) { if (ids) {
onArchive?.(ids, isArchived ? AssetVisibility.Archive : AssetVisibility.Timeline); onArchive?.(ids, visibility);
clearSelect(); clearSelect();
} }
loading = false; loading = false;