@plusscommunities/pluss-feature-builder-web-b 1.0.2-beta.4 → 1.0.2-beta.6

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@plusscommunities/pluss-feature-builder-web-b",
-  "version": "1.0.2-beta.4",
+  "version": "1.0.2-beta.6",
   "description": "Contains the feature builder extension for the pluss communities ",
   "main": "dist/index.cjs.js",
   "scripts": {

package/src/actions/featureDefinitionsIndex.js

@@ -6,25 +6,25 @@ import {
 	updatePermissionNames,
 } from "./featureBuilderStringsActions";
 
-export const FETCH_FEATURES_REQUEST = "FETCH_FEATURES_REQUEST";
-export const FETCH_FEATURES_SUCCESS = "FETCH_FEATURES_SUCCESS";
-export const FETCH_FEATURES_FAILURE = "FETCH_FEATURES_FAILURE";
+const REDUCER_PREFIX = values.reducerKey.toUpperCase();
+export const FETCH_FEATURES_REQUEST = `${REDUCER_PREFIX}_FETCH_FEATURES_REQUEST`;
+export const FETCH_FEATURES_SUCCESS = `${REDUCER_PREFIX}_FETCH_FEATURES_SUCCESS`;
+export const FETCH_FEATURES_FAILURE = `${REDUCER_PREFIX}_FETCH_FEATURES_FAILURE`;
 
-export const FEATURE_CREATE_REQUEST = "FEATURE_CREATE_REQUEST";
-export const FEATURE_CREATE_SUCCESS = "FEATURE_CREATE_SUCCESS";
-export const FEATURE_CREATE_FAILURE = "FEATURE_CREATE_FAILURE";
+export const FEATURE_CREATE_REQUEST = `${REDUCER_PREFIX}_FEATURE_CREATE_REQUEST`;
+export const FEATURE_CREATE_SUCCESS = `${REDUCER_PREFIX}_FEATURE_CREATE_SUCCESS`;
+export const FEATURE_CREATE_FAILURE = `${REDUCER_PREFIX}_FEATURE_CREATE_FAILURE`;
 
-export const FEATURE_EDIT_REQUEST = "FEATURE_EDIT_REQUEST";
-export const FEATURE_EDIT_SUCCESS = "FEATURE_EDIT_SUCCESS";
-export const FEATURE_EDIT_FAILURE = "FEATURE_EDIT_FAILURE";
+export const FEATURE_EDIT_REQUEST = `${REDUCER_PREFIX}_FEATURE_EDIT_REQUEST`;
+export const FEATURE_EDIT_SUCCESS = `${REDUCER_PREFIX}_FEATURE_EDIT_SUCCESS`;
+export const FEATURE_EDIT_FAILURE = `${REDUCER_PREFIX}_FEATURE_EDIT_FAILURE`;
 
-export const FEATURE_DELETE_REQUEST = "FEATURE_DELETE_REQUEST";
-export const FEATURE_DELETE_FAILURE = "FEATURE_DELETE_FAILURE";
+export const FEATURE_DELETE_REQUEST = `${REDUCER_PREFIX}_FEATURE_DELETE_REQUEST`;
+export const FEATURE_DELETE_FAILURE = `${REDUCER_PREFIX}_FEATURE_DELETE_FAILURE`;
 
 // Wizard synchronization action types
-export const SYNC_WIZARD_MODE_FROM_DEFINITION =
-	"SYNC_WIZARD_MODE_FROM_DEFINITION";
-export const SET_WIZARD_MODE = "SET_WIZARD_MODE";
+export const SYNC_WIZARD_MODE_FROM_DEFINITION = `${REDUCER_PREFIX}_SYNC_WIZARD_MODE_FROM_DEFINITION`;
+export const SET_WIZARD_MODE = `${REDUCER_PREFIX}_SET_WIZARD_MODE`;
 
 function fetchFeaturesRequest() {
 	return { type: FETCH_FEATURES_REQUEST };
@@ -160,7 +160,7 @@ export function fetchFeatureDefinitions() {
 			}
 		} catch (err) {
 			// Check if it's a 404 (feature doesn't exist)
-			if (err.response && err.response.status === 404) {
+			if (err.response?.status === 404) {
 				// 404: Feature doesn't exist → create mode
 				dispatch(fetchFeaturesSuccess(null, "create"));
 

package/src/actions/formActions.js

@@ -1,19 +1,9 @@
-/**
- * Form Actions for Feature Builder
- * Manages form state including title, icon, fields, layout, and submission
- * Handles CRUD operations for form fields and layout configuration
- * Coordinates with external menu updates and feature definition actions
- *
- * @namespace formActions
- */
-
 import { featureDefinitionActions } from "../webapi/featuresActions";
 import { values } from "../values.config";
 import {
-	updateFeatureBuilderString,
-	updateFeatureBuilderIcon,
+  updateFeatureBuilderString,
+  updateFeatureBuilderIcon,
 } from "./featureBuilderStringsActions";
-import { toTitleCase } from "../utils/textUtils";
 
 /**
  * @typedef {Object} FieldValues
@@ -40,20 +30,20 @@ import { toTitleCase } from "../utils/textUtils";
  */
 
 export const actionsTypes = {
-	SET_INITIAL_VALUES: "SET_INITIAL_VALUES",
-	SET_TITLE: "SET_TITLE",
-	SET_ICON: "SET_ICON",
-	SET_DISPLAY_NAME: "SET_DISPLAY_NAME",
-	ADD_FIELD: "ADD_FIELD",
-	DELETE_FIELD: "DELETE_FIELD",
-	UPDATE_FIELD: "UPDATE_FIELD",
-	SET_LAYOUT_GRID_ICON: "SET_LAYOUT_GRID_ICON",
-	SET_LAYOUT_TYPE: "SET_LAYOUT_TYPE",
-	SUBMIT_FORM_REQUEST: "SUBMIT_FORM_REQUEST",
-	SUBMIT_FORM_SUCCESS: "SUBMIT_FORM_SUCCESS",
-	SUBMIT_FORM_FAILURE: "SUBMIT_FORM_FAILURE",
-	CLEAR_FORM_SUBMISSION_STATE: "CLEAR_FORM_SUBMISSION_STATE",
-	SET_SUMMARY_FIELD: "SET_SUMMARY_FIELD",
+  SET_INITIAL_VALUES: `${values.reducerKey.toUpperCase()}_SET_INITIAL_VALUES`,
+  SET_TITLE: `${values.reducerKey.toUpperCase()}_SET_TITLE`,
+  SET_ICON: `${values.reducerKey.toUpperCase()}_SET_ICON`,
+  SET_DISPLAY_NAME: `${values.reducerKey.toUpperCase()}_SET_DISPLAY_NAME`,
+  ADD_FIELD: `${values.reducerKey.toUpperCase()}_ADD_FIELD`,
+  DELETE_FIELD: `${values.reducerKey.toUpperCase()}_DELETE_FIELD`,
+  UPDATE_FIELD: `${values.reducerKey.toUpperCase()}_UPDATE_FIELD`,
+  SET_LAYOUT_GRID_ICON: `${values.reducerKey.toUpperCase()}_SET_LAYOUT_GRID_ICON`,
+  SET_LAYOUT_TYPE: `${values.reducerKey.toUpperCase()}_SET_LAYOUT_TYPE`,
+  SUBMIT_FORM_REQUEST: `${values.reducerKey.toUpperCase()}_SUBMIT_FORM_REQUEST`,
+  SUBMIT_FORM_SUCCESS: `${values.reducerKey.toUpperCase()}_SUBMIT_FORM_SUCCESS`,
+  SUBMIT_FORM_FAILURE: `${values.reducerKey.toUpperCase()}_SUBMIT_FORM_FAILURE`,
+  CLEAR_FORM_SUBMISSION_STATE: `${values.reducerKey.toUpperCase()}_CLEAR_FORM_SUBMISSION_STATE`,
+  SET_SUMMARY_FIELD: `${values.reducerKey.toUpperCase()}_SET_SUMMARY_FIELD`,
 };
 
 /**
@@ -76,11 +66,11 @@ export const actionsTypes = {
  * }));
  */
 export const setInitialValues = (initialValues) => {
-	return { type: actionsTypes.SET_INITIAL_VALUES, payload: initialValues };
+  return { type: actionsTypes.SET_INITIAL_VALUES, payload: initialValues };
 };
 
 export const setTitle = (title) => {
-	return { type: actionsTypes.SET_TITLE, payload: title };
+  return { type: actionsTypes.SET_TITLE, payload: title };
 };
 
 /**
@@ -95,16 +85,16 @@ export const setTitle = (title) => {
  * dispatch(setDisplayName('Contact Information'));
  */
 export const setDisplayName = (displayName) => {
-	return (dispatch) => {
-		// Update menu string when display name changes (will be title cased in updateFeatureBuilderString)
-		dispatch(updateFeatureBuilderString(displayName));
+  return (dispatch) => {
+    // Update menu string when display name changes (will be title cased in updateFeatureBuilderString)
+    dispatch(updateFeatureBuilderString(displayName));
 
-		// Dispatch the actual action
-		dispatch({
-			type: actionsTypes.SET_DISPLAY_NAME,
-			payload: displayName,
-		});
-	};
+    // Dispatch the actual action
+    dispatch({
+      type: actionsTypes.SET_DISPLAY_NAME,
+      payload: displayName,
+    });
+  };
 };
 
 /**
@@ -118,16 +108,16 @@ export const setDisplayName = (displayName) => {
  * dispatch(setIcon('fa-user'));
  */
 export const setIcon = (icon) => {
-	return (dispatch) => {
-		// Update menu icon when icon changes
-		dispatch(updateFeatureBuilderIcon(icon));
+  return (dispatch) => {
+    // Update menu icon when icon changes
+    dispatch(updateFeatureBuilderIcon(icon));
 
-		// Dispatch the actual action
-		dispatch({
-			type: actionsTypes.SET_ICON,
-			payload: icon,
-		});
-	};
+    // Dispatch the actual action
+    dispatch({
+      type: actionsTypes.SET_ICON,
+      payload: icon,
+    });
+  };
 };
 
 /**
@@ -142,20 +132,20 @@ export const setIcon = (icon) => {
  * dispatch(addField('image'));
  */
 export const addField = (fieldType = "text") => {
-	// Generate a unique ID for the new field
-	const fieldId = `custom-field-${Date.now()}-${Math.random().toString(36).substr(2, 9)}`;
-	return {
-		type: actionsTypes.ADD_FIELD,
-		payload: { id: fieldId, type: fieldType },
-	};
+  // Generate a unique ID for the new field
+  const fieldId = `custom-field-${Date.now()}-${Math.random().toString(36).substr(2, 9)}`;
+  return {
+    type: actionsTypes.ADD_FIELD,
+    payload: { id: fieldId, type: fieldType },
+  };
 };
 
 export const deleteField = (id) => {
-	return { type: actionsTypes.DELETE_FIELD, payload: id };
+  return { type: actionsTypes.DELETE_FIELD, payload: id };
 };
 
 export const updateFieldById = (id, updatedField) => {
-	return { type: actionsTypes.UPDATE_FIELD, payload: { id, updatedField } };
+  return { type: actionsTypes.UPDATE_FIELD, payload: { id, updatedField } };
 };
 
 /**
@@ -170,31 +160,31 @@ export const updateFieldById = (id, updatedField) => {
  * dispatch(setSummaryField('field-description-123'));
  */
 export const setSummaryField = (fieldId) => {
-	return { type: actionsTypes.SET_SUMMARY_FIELD, payload: fieldId };
+  return { type: actionsTypes.SET_SUMMARY_FIELD, payload: fieldId };
 };
 
 export const setGridLayoutIcon = (imageUrl) => {
-	return { type: actionsTypes.SET_LAYOUT_GRID_ICON, payload: imageUrl };
+  return { type: actionsTypes.SET_LAYOUT_GRID_ICON, payload: imageUrl };
 };
 
 export const setLayoutType = (layoutType) => {
-	return { type: actionsTypes.SET_LAYOUT_TYPE, payload: layoutType };
+  return { type: actionsTypes.SET_LAYOUT_TYPE, payload: layoutType };
 };
 
 const submitFormRequest = () => {
-	return { type: actionsTypes.SUBMIT_FORM_REQUEST };
+  return { type: actionsTypes.SUBMIT_FORM_REQUEST };
 };
 
 const submitFormSuccess = () => {
-	return { type: actionsTypes.SUBMIT_FORM_SUCCESS };
+  return { type: actionsTypes.SUBMIT_FORM_SUCCESS };
 };
 
 const submitFormFailure = (error) => {
-	return { type: actionsTypes.SUBMIT_FORM_FAILURE, payload: error };
+  return { type: actionsTypes.SUBMIT_FORM_FAILURE, payload: error };
 };
 
 export const clearFormSubmissionState = () => {
-	return { type: actionsTypes.CLEAR_FORM_SUBMISSION_STATE };
+  return { type: actionsTypes.CLEAR_FORM_SUBMISSION_STATE };
 };
 
 /**
@@ -205,107 +195,107 @@ export const clearFormSubmissionState = () => {
  * @throws {Error} When form validation fails or API submission encounters error
  */
 export function submitForm() {
-	return async (dispatch, getState) => {
-		const state = getState()[values.reducerKey];
-		const form = state && state.form;
-		if (!form) {
-			dispatch(
-				submitFormFailure(
-					new Error(
-						"Form data is missing. Please refresh the page and try again.",
-					),
-				),
-			);
-			return;
-		}
+  return async (dispatch, getState) => {
+    const state = getState()[values.reducerKey];
+    const form = state?.form;
+    if (!form) {
+      dispatch(
+        submitFormFailure(
+          new Error(
+            "Form data is missing. Please refresh the page and try again.",
+          ),
+        ),
+      );
+      return;
+    }
 
-		dispatch(submitFormRequest());
-		try {
-			// Get site from auth store
-			const site = getState().auth.site;
-			if (!site) {
-				throw new Error(
-					"Authentication error: Site context not found. Please refresh and login again.",
-				);
-			}
+    dispatch(submitFormRequest());
+    try {
+      // Get site from auth store
+      const site = getState().auth.site;
+      if (!site) {
+        throw new Error(
+          "Authentication error: Site context not found. Please refresh and login again.",
+        );
+      }
 
-			// Use mode from fetch instead of checking definition ID
-			const definitionState = state && state.definition;
-			const mode = definitionState && definitionState.mode; // Use stored mode from fetch
-			const definitionId = definitionState && definitionState.id;
+      // Use mode from fetch instead of checking definition ID
+      const definitionState = state?.definition;
+      const mode = definitionState?.mode; // Use stored mode from fetch
+      const definitionId = definitionState?.id;
 
-			if (mode === "edit") {
-				// Always update when in edit mode
-				const updatedDefinition = {
-					id: definitionId,
-					site: site, // Include site from auth store
-					featureDefinition: {
-						// Wrap in expected structure for backend
-						title: form.title,
-						icon: form.icon,
-						displayName: form.displayName,
-						layout: form.layout,
-						fields: form.fields,
-					},
-				};
-				await featureDefinitionActions.edit(updatedDefinition, site);
-			} else {
-				// Always create when in create mode (or mode is undefined/null)
-				if (!values.featureId || !site) {
-					throw new Error(
-						"Authentication error: Missing required context (featureId or site).",
-					);
-				}
-				await featureDefinitionActions.create(
-					values.featureId,
-					site, // Use actual site from auth store
-					{
-						title: form.title,
-						icon: form.icon,
-						displayName: form.displayName,
-						layout: form.layout,
-						fields: form.fields,
-					},
-				);
-			}
+      if (mode === "edit") {
+        // Always update when in edit mode
+        const updatedDefinition = {
+          id: definitionId,
+          site: site, // Include site from auth store
+          featureDefinition: {
+            // Wrap in expected structure for backend
+            title: form.title,
+            icon: form.icon,
+            displayName: form.displayName,
+            layout: form.layout,
+            fields: form.fields,
+          },
+        };
+        await featureDefinitionActions.edit(updatedDefinition, site);
+      } else {
+        // Always create when in create mode (or mode is undefined/null)
+        if (!values.featureId || !site) {
+          throw new Error(
+            "Authentication error: Missing required context (featureId or site).",
+          );
+        }
+        await featureDefinitionActions.create(
+          values.featureId,
+          site, // Use actual site from auth store
+          {
+            title: form.title,
+            icon: form.icon,
+            displayName: form.displayName,
+            layout: form.layout,
+            fields: form.fields,
+          },
+        );
+      }
 
-			dispatch(submitFormSuccess());
-		} catch (err) {
-			// Handle different types of errors
-			let errorToDisplay = err;
+      dispatch(submitFormSuccess());
+    } catch (err) {
+      // Handle different types of errors
+      let errorToDisplay = err;
 
-			if (err.response) {
-				// API error (400, 401, 404, 500, etc.)
-				const { status, data } = err.response;
-				if (status === 400 && data && data.error) {
-					errorToDisplay = new Error(`Validation error: ${data.error}`);
-				} else if (status === 401) {
-					errorToDisplay = new Error(
-						"You are not authorized to perform this action",
-					);
-				} else if (status === 404) {
-					errorToDisplay = new Error("Feature definition not found");
-				} else if (status >= 500) {
-					errorToDisplay = new Error("Server error. Please try again later.");
-				} else {
-					errorToDisplay = new Error(
-						(data && data.error) || `Request failed with status ${status}`,
-					);
-				}
-			} else if (err.request) {
-				// Network error (no response received)
-				errorToDisplay = new Error(
-					"Network error. Please check your connection and try again.",
-				);
-			} else if (err.message) {
-				// Other JavaScript errors
-				errorToDisplay = err;
-			} else {
-				// Unknown error
-				errorToDisplay = new Error("An unexpected error occurred while saving");
-			}
+      if (err.response) {
+        // API error (400, 401, 404, 500, etc.)
+        const { status, data } = err.response;
+        if (status === 400 && data?.error) {
+          errorToDisplay = new Error(`Validation error: ${data.error}`);
+        } else if (status === 401) {
+          errorToDisplay = new Error(
+            "You are not authorized to perform this action",
+          );
+        } else if (status === 404) {
+          errorToDisplay = new Error("Feature definition not found");
+        } else if (status >= 500) {
+          errorToDisplay = new Error("Server error. Please try again later.");
+        } else {
+          errorToDisplay = new Error(
+            data?.error || `Request failed with status ${status}`,
+          );
+        }
+      } else if (err.request) {
+        // Network error (no response received)
+        errorToDisplay = new Error(
+          "Network error. Please check your connection and try again.",
+        );
+      } else if (err.message) {
+        // Other JavaScript errors
+        errorToDisplay = err;
+      } else {
+        // Unknown error
+        errorToDisplay = new Error("An unexpected error occurred while saving");
+      }
 
-			dispatch(submitFormFailure(errorToDisplay));
-		}
-	};
+      dispatch(submitFormFailure(errorToDisplay));
+    }
+  };
 }

package/src/actions/listingActions.js

@@ -1,38 +1,40 @@
 import { arrayMove } from "@dnd-kit/sortable";
 import { listingActions as webApiActions } from "../webapi/listingActions";
+import { values } from "../values.config";
 
 // Action Types
-export const FETCH_LISTING_REQUEST = "FETCH_LISTING_REQUEST";
-export const FETCH_LISTING_SUCCESS = "FETCH_LISTING_SUCCESS";
-export const FETCH_LISTING_FAILURE = "FETCH_LISTING_FAILURE";
-export const FETCH_LISTING_SILENT_SUCCESS = "FETCH_LISTING_SILENT_SUCCESS";
-export const FETCH_LISTING_SILENT_FAILURE = "FETCH_LISTING_SILENT_FAILURE";
+const REDUCER_PREFIX = values.reducerKey.toUpperCase();
+export const FETCH_LISTING_REQUEST = `${REDUCER_PREFIX}_FETCH_LISTING_REQUEST`;
+export const FETCH_LISTING_SUCCESS = `${REDUCER_PREFIX}_FETCH_LISTING_SUCCESS`;
+export const FETCH_LISTING_FAILURE = `${REDUCER_PREFIX}_FETCH_LISTING_FAILURE`;
+export const FETCH_LISTING_SILENT_SUCCESS = `${REDUCER_PREFIX}_FETCH_LISTING_SILENT_SUCCESS`;
+export const FETCH_LISTING_SILENT_FAILURE = `${REDUCER_PREFIX}_FETCH_LISTING_SILENT_FAILURE`;
 
-export const REORDER_LISTING_SUCCESS = "REORDER_LISTING_SUCCESS";
-export const REORDER_LISTING_FAILURE = "REORDER_LISTING_FAILURE";
+export const REORDER_LISTING_SUCCESS = `${REDUCER_PREFIX}_REORDER_LISTING_SUCCESS`;
+export const REORDER_LISTING_FAILURE = `${REDUCER_PREFIX}_REORDER_LISTING_FAILURE`;
 
-export const DELETE_LISTING_REQUEST = "DELETE_LISTING_REQUEST";
-export const DELETE_LISTING_SUCCESS = "DELETE_LISTING_SUCCESS";
-export const DELETE_LISTING_FAILURE = "DELETE_LISTING_FAILURE";
+export const DELETE_LISTING_REQUEST = `${REDUCER_PREFIX}_DELETE_LISTING_REQUEST`;
+export const DELETE_LISTING_SUCCESS = `${REDUCER_PREFIX}_DELETE_LISTING_SUCCESS`;
+export const DELETE_LISTING_FAILURE = `${REDUCER_PREFIX}_DELETE_LISTING_FAILURE`;
 
-export const UNDELETE_LISTING_REQUEST = "UNDELETE_LISTING_REQUEST";
-export const UNDELETE_LISTING_SUCCESS = "UNDELETE_LISTING_SUCCESS";
-export const UNDELETE_LISTING_FAILURE = "UNDELETE_LISTING_FAILURE";
+export const UNDELETE_LISTING_REQUEST = `${REDUCER_PREFIX}_UNDELETE_LISTING_REQUEST`;
+export const UNDELETE_LISTING_SUCCESS = `${REDUCER_PREFIX}_UNDELETE_LISTING_SUCCESS`;
+export const UNDELETE_LISTING_FAILURE = `${REDUCER_PREFIX}_UNDELETE_LISTING_FAILURE`;
 
-export const CREATE_LISTING_REQUEST = "CREATE_LISTING_REQUEST";
-export const CREATE_LISTING_SUCCESS = "CREATE_LISTING_SUCCESS";
-export const CREATE_LISTING_FAILURE = "CREATE_LISTING_FAILURE";
+export const CREATE_LISTING_REQUEST = `${REDUCER_PREFIX}_CREATE_LISTING_REQUEST`;
+export const CREATE_LISTING_SUCCESS = `${REDUCER_PREFIX}_CREATE_LISTING_SUCCESS`;
+export const CREATE_LISTING_FAILURE = `${REDUCER_PREFIX}_CREATE_LISTING_FAILURE`;
 
-export const EDIT_LISTING_REQUEST = "EDIT_LISTING_REQUEST";
-export const EDIT_LISTING_SUCCESS = "EDIT_LISTING_SUCCESS";
-export const EDIT_LISTING_FAILURE = "EDIT_LISTING_FAILURE";
+export const EDIT_LISTING_REQUEST = `${REDUCER_PREFIX}_EDIT_LISTING_REQUEST`;
+export const EDIT_LISTING_SUCCESS = `${REDUCER_PREFIX}_EDIT_LISTING_SUCCESS`;
+export const EDIT_LISTING_FAILURE = `${REDUCER_PREFIX}_EDIT_LISTING_FAILURE`;
 
-export const TOGGLE_LISTING_REQUEST = "TOGGLE_LISTING_REQUEST";
-export const TOGGLE_LISTING_SUCCESS = "TOGGLE_LISTING_SUCCESS";
-export const TOGGLE_LISTING_FAILURE = "TOGGLE_LISTING_FAILURE";
+export const TOGGLE_LISTING_REQUEST = `${REDUCER_PREFIX}_TOGGLE_LISTING_REQUEST`;
+export const TOGGLE_LISTING_SUCCESS = `${REDUCER_PREFIX}_TOGGLE_LISTING_SUCCESS`;
+export const TOGGLE_LISTING_FAILURE = `${REDUCER_PREFIX}_TOGGLE_LISTING_FAILURE`;
 
-export const SET_SORT_BY = "SET_SORT_BY";
-export const SET_SHOW_DELETED = "SET_SHOW_DELETED";
+export const SET_SORT_BY = `${REDUCER_PREFIX}_SET_SORT_BY`;
+export const SET_SHOW_DELETED = `${REDUCER_PREFIX}_SET_SHOW_DELETED`;
 
 export const fetchListingRequest = () => ({ type: FETCH_LISTING_REQUEST });
 export const fetchListingSuccess = (listings) => ({
@@ -210,7 +212,7 @@ export const undeleteListing = (id) => {
 			const response = await webApiActions.undelete(id, site);
 			// If API returns the restored listing, use it; otherwise we'll need to refetch
 			const restoredListing = response.data;
-			if (restoredListing && restoredListing.id) {
+			if (restoredListing?.id) {
 				dispatch(undeleteListingSuccess(restoredListing));
 			} else {
 				// Trigger a refetch by dispatching the success with just ID, then fetch updated listings