Skip to content

Commit 19198f0

Browse files
priscilawebdevandrewshie-sentry
authored andcommitted
fix(quick-start): Task not marked as seen when sidebar is open (#91702)
1 parent d17b33a commit 19198f0

File tree

1 file changed

+14
-6
lines changed

1 file changed

+14
-6
lines changed

static/app/components/onboardingWizard/content.tsx

Lines changed: 14 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -335,6 +335,11 @@ function ExpandedTaskGroup({sortedTasks, hidePanel}: ExpandedTaskGroupProps) {
335335

336336
const markCompletionTimeout = useRef<number | undefined>(undefined);
337337

338+
const unseenDoneTasks = useMemo(
339+
() => sortedTasks.filter(task => taskIsDone(task) && !task.completionSeen),
340+
[sortedTasks]
341+
);
342+
338343
function completionTimeout(time: number): Promise<void> {
339344
window.clearTimeout(markCompletionTimeout.current);
340345
return new Promise(resolve => {
@@ -343,16 +348,16 @@ function ExpandedTaskGroup({sortedTasks, hidePanel}: ExpandedTaskGroupProps) {
343348
}
344349

345350
const markTasksAsSeen = useCallback(() => {
346-
const unseenDoneTasks = sortedTasks
351+
const tasksToMarkComplete = sortedTasks
347352
.filter(task => taskIsDone(task) && !task.completionSeen)
348353
.map(task => ({...task, completionSeen: true}));
349354

350355
if (isDemoModeActive()) {
351-
for (const unseenDoneTask of unseenDoneTasks) {
352-
updateDemoWalkthroughTask(unseenDoneTask);
356+
for (const task of tasksToMarkComplete) {
357+
updateDemoWalkthroughTask(task);
353358
}
354359
} else {
355-
mutateOnboardingTasks.mutate(unseenDoneTasks);
360+
mutateOnboardingTasks.mutate(tasksToMarkComplete);
356361
}
357362
}, [mutateOnboardingTasks, sortedTasks]);
358363

@@ -367,11 +372,14 @@ function ExpandedTaskGroup({sortedTasks, hidePanel}: ExpandedTaskGroupProps) {
367372
);
368373

369374
useEffect(() => {
370-
markSeenOnOpen();
375+
if (unseenDoneTasks.length > 0) {
376+
markSeenOnOpen();
377+
}
378+
371379
return () => {
372380
window.clearTimeout(markCompletionTimeout.current);
373381
};
374-
}, []); // eslint-disable-line react-hooks/exhaustive-deps
382+
}, [unseenDoneTasks, markSeenOnOpen]);
375383

376384
return (
377385
<Fragment>

0 commit comments

Comments
 (0)