Refactor connected components to streamline media offload logic

- Removed redundant image handling from the ConnectedLayout and OffloadMedia components.
- Simplified the callSync function by eliminating unnecessary image parameters.
- Enhanced clarity and maintainability of the codebase by reducing complexity in state management and effect hooks.

Author: selul

assets/src/dashboard/parts/connected/index.js

@@ -64,27 +64,6 @@ const ConnectedLayout = ({
 	const [ canSave, setCanSave ] = useState( false );
 	const [ settings, setSettings ] = useState( siteSettings );
 
-	useEffect( () => {
-		const urlSearchParams = new URLSearchParams( window.location.search );
-		const params = Object.fromEntries( urlSearchParams.entries() );
-		let images = [];
-
-		if ( Object.prototype.hasOwnProperty.call( params, 'optimole_action' ) ) {
-			for ( let i = 0; 20 > i; i++ ) {
-				if ( Object.prototype.hasOwnProperty.call( params, i ) ) {
-					if ( ! isNaN( params[i]) ) {
-						images[i] = parseInt( params[i], 10 );
-					}
-				}
-			}
-
-			params.images = images;
-			params.url = window.location.href.split( '?' )[0] + ( Object.prototype.hasOwnProperty.call( params, 'paged' ) ? '?paged=' + params.paged : '' );
-			setQueryArgs( params );
-			setTab( 'settings' );
-			setMenu( 'offload_media' );
-		}
-	}, []);
 
 	useEffect( () => {
 		if ( isConnected && hasApplication ) {

assets/src/dashboard/parts/connected/settings/OffloadMedia.js

@@ -82,19 +82,6 @@ const OffloadMedia = ({ settings, canSave, setSettings, setCanSave }) => {
 	const isOffloadingInProgress = 'disabled' !== settings['offloading_status'];
 	const isRollbackInProgress = 'disabled' !== settings['rollback_status'];
 
-	useEffect( () => {
-		if ( ! queryArgs?.optimole_action || ! queryArgs?.images ) {
-			return;
-		}
-
-		if ( 'offload_images' === queryArgs.optimole_action ) {
-			onOffloadMedia( queryArgs.images );
-		}
-
-		if ( 'rollback_images' === queryArgs.optimole_action ) {
-			onRollbackdMedia( queryArgs.images );
-		}
-	}, []);
 
 	useEffect( () => {
 		if ( isOffloadingInProgress ) {
@@ -126,8 +113,7 @@ const OffloadMedia = ({ settings, canSave, setSettings, setCanSave }) => {
 			setOffloadLimitReached( false );
 
 			callSync({
-				action: 'offload_images',
-				images: imageIds
+				action: 'offload_images'
 			});
 
 			setCanSave( false );
@@ -148,8 +134,7 @@ const OffloadMedia = ({ settings, canSave, setSettings, setCanSave }) => {
 					setProcessedImages( 0 );
 
 					callSync({
-						action: 'rollback_images',
-						images: imageIds
+						action: 'rollback_images'
 					});
 
 					setCanSave( false );

assets/src/dashboard/utils/api.js

@@ -497,15 +497,13 @@ export const callSync = ( data ) => {
 
 	setIsLoading( true );
 
-	const images_ids = undefined !== data.images ? data.images : [];
 
 	apiFetch({
 		path: optimoleDashboardApp.routes['number_of_images_and_pages'],
 		method: 'POST',
 		data: {
 			action: data.action,
-			refresh: data.refresh || false,
-			images: images_ids
+			refresh: data.refresh || false
 		},
 		parse: false
 	})
@@ -577,8 +575,7 @@ export const callSync = ( data ) => {
 			setTimeout( () => {
 				callSync({
 					action: data.action,
-					refresh: true,
-					images: images_ids
+					refresh: true
 				});
 			}, 10000 );
 		})