psp-9649 ensure that new map markers display immediately, and that ma…

…p refresh also triggers the advanced filter refresh. (#4476)
bcgov · Nov 14, 2024 · 5eca897 · 5eca897
1 parent 86d4198
commit 5eca897
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/source/frontend/src/components/common/mapFSM/machineDefinition/mapMachine.ts b/source/frontend/src/components/common/mapFSM/machineDefinition/mapMachine.ts
@@ -397,10 +397,9 @@ const advancedFilterSideBarStates = {
           actions: [
             send({ type: 'REFRESH_PROPERTIES' }),
             assign({
-              isFiltering: () => false,
               showDisposed: () => false,
               showRetired: () => false,
-              activePimsPropertyIds: () => [],
+              advancedSearchCriteria: () => new PropertyFilterFormModel(),
             }),
           ],
         },

diff --git a/source/frontend/src/features/properties/map/MapContainer.tsx b/source/frontend/src/features/properties/map/MapContainer.tsx
@@ -37,6 +37,7 @@ const MapContainer: React.FC<React.PropsWithChildren<MapContainerProps>> = () =>
     setVisiblePimsProperties,
     advancedSearchCriteria,
     isMapVisible,
+    isLoading,
   } = useMapStateMachine();
 
   const { getMatchingProperties } = usePimsPropertyRepository();
@@ -58,7 +59,7 @@ const MapContainer: React.FC<React.PropsWithChildren<MapContainerProps>> = () =>
 
   useEffect(() => {
     filterProperties(advancedSearchCriteria?.toApi());
-  }, [filterProperties, advancedSearchCriteria]);
+  }, [filterProperties, advancedSearchCriteria, isLoading]);
 
   const cursorClass = isSelecting
     ? MapCursors.DRAFT