Merge pull request #206 from devtron-labs/fix/plugin-learn-more

fix: add stopPropagation for learn more link in plugin list

Author: AbhishekA1509

src/Shared/Components/Plugin/PluginCard.tsx

@@ -1,4 +1,4 @@
-import { Checkbox, CHECKBOX_VALUE } from '../../../Common'
+import { Checkbox, CHECKBOX_VALUE, stopPropagation } from '../../../Common'
 import PluginImageContainer from './PluginImageContainer'
 import PluginTagsContainer from './PluginTagsContainer'
 import { PluginCardProps } from './types'
@@ -74,6 +74,7 @@ const PluginCard = ({
                                         className="anchor fs-12 fw-6 lh-20"
                                         target="_blank"
                                         rel="noopener noreferrer"
+                                        onClick={stopPropagation}
                                     >
                                         Learn more
                                     </a>

src/Shared/Components/Plugin/PluginListContainer.tsx

@@ -79,7 +79,8 @@ const PluginListContainer = ({
 
     const [isLoadingPluginData, pluginData, pluginDataError, reloadPluginData] = useAsync(
         getPluginStoreDataWrapper,
-        persistFilters ? [pluginList] : [searchKey, appId, selectedTags],
+        // In case of persistFilters with change of searchKey or selectedTags we anyways clear the pluginList so no need to add dependency
+        [searchKey, appId, selectedTags],
         persistFilters ? !pluginList.length : true,
     )