combine the useEffects in FetchMoreGalleriesData

Author: marjisound

dotcom-rendering/src/components/FetchMoreGalleriesData.importable.tsx

@@ -101,6 +101,11 @@ export const FetchMoreGalleriesData = ({
 			.then((fetchedData) => {
 				setData(fetchedData);
 				setError(undefined);
+				addDiscussionIds(
+					fetchedData.trails
+						.map((trail) => trail.discussion?.discussionId)
+						.filter(isNonNullable),
+				);
 			})
 			.catch((err) => {
 				setError(
@@ -110,15 +115,6 @@ export const FetchMoreGalleriesData = ({
 			});
 	}, [ajaxUrl]);
 
-	useEffect(() => {
-		if (!data) return;
-		addDiscussionIds(
-			data.trails
-				.map((trail) => trail.discussion?.discussionId)
-				.filter(isNonNullable),
-		);
-	}, [data]);
-
 	if (error) {
 		// Send the error to Sentry and then prevent the element from rendering
 		window.guardian.modules.sentry.reportError(error, 'more-galleries');