Merge pull request #208 from Automattic/fix/post-setup-campaign-error

fix: situation that overwrote mc campaign id
Automattic · May 11, 2020 · 7194b3d · 7194b3d
2 parents 7e2d9c4 + 928d955
commit 7194b3d
Showing 1 changed file with 3 additions and 5 deletions.
diff --git a/src/newsletter-editor/editor/index.js b/src/newsletter-editor/editor/index.js
@@ -64,15 +64,13 @@ const Editor = compose( [
 			props
 				.apiFetchWithErrorHandling( getFetchDataConfig( { postId: props.postId } ) )
 				.then( result => {
-					props.editPost( getEditPostPayload( result ) );
+					const postUpdate = getEditPostPayload( result );
+					postUpdate.meta.color_palette = props.colorPalette;
+					props.editPost( postUpdate );
 				} );
 		}
 	}, [ props.isPublishingOrSavingPost ]);
 
-	useEffect(() => {
-		props.editPost( { meta: { color_palette: props.colorPalette } } );
-	}, []);
-
 	// Lock or unlock post publishing.
 	useEffect(() => {
 		if ( props.isReady ) {