pacnpal/thrilltrack-explorer
1
0
Fork 0
mirror of https://github.com/pacnpal/thrilltrack-explorer.git synced 2025-12-22 03:51:12 -05:00
Code Issues Packages Projects Releases Wiki Activity

Refactor: Apply smart merging to all refreshes

Browse Source
This commit is contained in:
gpt-engineer-app[bot]
2025-10-06 19:43:36 +00:00
parent fd20c02e5e
commit c3b0ae3d76
1 changed files with 47 additions and 44 deletions
Download Patch File Download Diff File Expand all files Collapse all files

25
src/components/moderation/ModerationQueue.tsx
View File

@@ -520,11 +520,10 @@ export const ModerationQueue = forwardRef<ModerationQueueRef>((props, ref) => {
return a.id.localeCompare(b.id); return a.id.localeCompare(b.id);
}); });
// Use smart merging for silent refreshes if strategy is 'merge' // ALWAYS use smart merge to detect actual changes (prevents flashing)
if (silent && refreshStrategy === 'merge') {
const mergeResult = smartMergeArray(items, formattedItems, { const mergeResult = smartMergeArray(items, formattedItems, {
compareFields: ['status', 'reviewed_at', 'reviewed_by', 'reviewer_notes', 'submission_type', 'entity_name', 'park_name'], compareFields: ['status', 'reviewed_at', 'reviewed_by', 'reviewer_notes', 'submission_type', 'entity_name', 'park_name'],
preserveOrder: true, preserveOrder: silent, // Only preserve order for silent refreshes
addToTop: true, addToTop: true,
}); });
@@ -538,11 +537,12 @@ export const ModerationQueue = forwardRef<ModerationQueueRef>((props, ref) => {
updated: mergeResult.changes.updated.length, updated: mergeResult.changes.updated.length,
removed: mergeResult.changes.removed.length, removed: mergeResult.changes.removed.length,
totalItems: mergeResult.items.length, totalItems: mergeResult.items.length,
silent,
}); });
// Only apply protection map if needed // Only apply protection map for silent refreshes
let finalItems = mergeResult.items; let finalItems = mergeResult.items;
if (preserveInteraction && interactingWith.size > 0) { if (silent && preserveInteraction && interactingWith.size > 0) {
finalItems = mergeResult.items.map(item => { finalItems = mergeResult.items.map(item => {
if (interactingWith.has(item.id)) { if (interactingWith.has(item.id)) {
const currentItem = items.find(i => i.id === item.id); const currentItem = items.find(i => i.id === item.id);
@@ -557,19 +557,22 @@ export const ModerationQueue = forwardRef<ModerationQueueRef>((props, ref) => {
setItems(finalItems); setItems(finalItems);
} }
// Only set new items count if there are genuinely new items // For non-silent refreshes, reset the counter
if (actuallyNewItems > 0) { if (!silent) {
setNewItemsCount(0);
} else if (actuallyNewItems > 0) {
// For silent refreshes, show new items count
setNewItemsCount(actuallyNewItems); setNewItemsCount(actuallyNewItems);
} }
} else { } else {
// No changes detected - skip all state updates // No changes detected - skip all state updates
console.log('✅ No changes detected, keeping current state'); console.log('✅ No changes detected, keeping current state');
}
} else { // For non-silent refreshes, still reset the counter even if no changes
// Full replacement for non-silent refreshes or 'replace' strategy if (!silent) {
setItems(formattedItems);
setNewItemsCount(0); setNewItemsCount(0);
} }
}
} catch (error: any) { } catch (error: any) {
console.error('Error fetching moderation items:', error); console.error('Error fetching moderation items:', error);
console.error('Error details:', { console.error('Error details:', {