fix comments

mlrun · Dec 9, 2024 · 5351179 · 5351179
1 parent 985cc77
commit 5351179
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 22 deletions.
diff --git a/src/common/HistoryBackLink/historyBackLink.js b/src/common/HistoryBackLink/historyBackLink.js
@@ -28,31 +28,28 @@ import { ReactComponent as HistoryIcon } from 'igz-controls/images/history.svg'
 
 import './historyBackLink.scss'
 
-const HistoryBackLink = ({link, itemName}) => {
-    return (
-        <div className="history-back-link">
-        <Link
-          to={link}
-          className="history-back-link__icon"
-        >
-          <RoundedIcon id="history-back-link-btn" tooltipText="Back">
-            <Back />
-          </RoundedIcon>
-        </Link>
-        <div className="history-back-link__title">
-          <HistoryIcon />
-          <div className="history-back-link__title-version" data-testid="version-history">Version history: </div>
-          <Tooltip template={<TextTooltipTemplate text={itemName} />}>
-            {itemName}
-          </Tooltip>
+const HistoryBackLink = ({ itemName, link }) => {
+  return (
+    <div className="history-back-link">
+      <Link to={link} className="history-back-link__icon">
+        <RoundedIcon id="history-back-link-btn" tooltipText="Back">
+          <Back />
+        </RoundedIcon>
+      </Link>
+      <div className="history-back-link__title">
+        <HistoryIcon />
+        <div className="history-back-link__title-version" data-testid="version-history">
+          Version history:{' '}
         </div>
+        <Tooltip template={<TextTooltipTemplate text={itemName} />}>{itemName}</Tooltip>
       </div>
-    )
+    </div>
+  )
 }
 
 HistoryBackLink.propTypes = {
-  link: PropTypes.string.isRequired,
   itemName: PropTypes.string.isRequired,
+  link: PropTypes.string.isRequired
 }
 
 export default HistoryBackLink
diff --git a/src/components/ActionBar/ActionBar.js b/src/components/ActionBar/ActionBar.js
@@ -192,7 +192,7 @@ const ActionBar = ({
       const newFilters = { ...filters, ...formValues }
 
       if (filtersHelperResult) {
-        if (params.name || params.hash) {
+        if (params.name || params.id) {
           navigate(navigateLink)
         }
 
@@ -220,10 +220,10 @@ const ActionBar = ({
       filtersHelper,
       changes,
       dispatch,
-      saveFilters,
       params.name,
-      params.hash,
+      params.id,
       filtersStore.groupBy,
+      saveFilters,
       removeSelectedItem,
       setSelectedRowData,
       toggleAllRows,