From b757e8e7748f497bc01acda34579df508b6338de Mon Sep 17 00:00:00 2001
From: zkldi <20380519+zkldi@users.noreply.github.com>
Date: Tue, 13 Dec 2022 01:30:24 +0000
Subject: [PATCH] fix: remember folder/charts when scrimbling somewhat

---
 .../components/targets/AddNewGoalForQuestModal.tsx    | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/client/src/components/targets/AddNewGoalForQuestModal.tsx b/client/src/components/targets/AddNewGoalForQuestModal.tsx
index 1d9dffa09..00df3acee 100644
--- a/client/src/components/targets/AddNewGoalForQuestModal.tsx
+++ b/client/src/components/targets/AddNewGoalForQuestModal.tsx
@@ -211,7 +211,18 @@ function RenderGoalChartPicker({
 		[type, data]
 	);
 
+	const [isFirstRender, setIsFirstRender] = useState(true);
+
 	useEffect(() => {
+		// DON'T RUN THIS HOOK ON FIRST RENDER
+		// this should reset the selected folder/charts when the type is changed
+		// but on the first render, we're potentially inheriting from initial state
+		// so don't immediately wipe it!
+		if (isFirstRender) {
+			setIsFirstRender(false);
+			return;
+		}
+
 		setData(null);
 
 		onChange({ type, data: null as any });