Hide versions with no changes in changelog (#4086)

Signed-off-by: Cintia Sánchez García <cynthiasg@icloud.com>
artifacthub · Oct 8, 2024 · cec84f9 · cec84f9
1 parent 088b92b
commit cec84f9
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 5 deletions.
diff --git a/web/src/layout/package/changelog/Content.module.css b/web/src/layout/package/changelog/Content.module.css
@@ -65,8 +65,8 @@
   background-color: #df2a29;
 }
 
-.badgeIcon {
-  top: -1px;
+.badgeContent {
+  margin-top: 1px;
 }
 
 .lastVersion {

diff --git a/web/src/layout/package/changelog/Content.tsx b/web/src/layout/package/changelog/Content.tsx
@@ -143,7 +143,7 @@ const Content = (props: Props) => {
                               styles[`${change.kind.toString()}ChangeBadge`]
                             )}
                           >
-                            <span className={`position-relative ${styles.badgeIcon}`}>
+                            <span>
                               {(() => {
                                 switch (change.kind) {
                                   case ChangeKind.added:
@@ -163,7 +163,9 @@ const Content = (props: Props) => {
                                 }
                               })()}
                             </span>
-                            <span className="d-none d-md-block ms-1">{change.kind.toString()}</span>
+                            <span className={`d-none d-md-block ms-1 ${styles.badgeContent}`}>
+                              {change.kind.toString()}
+                            </span>
                           </div>
                         </div>
                       ) : (

diff --git a/web/src/layout/package/changelog/Modal.tsx b/web/src/layout/package/changelog/Modal.tsx
@@ -1,4 +1,5 @@
 import classnames from 'classnames';
+import isEmpty from 'lodash/isEmpty';
 import isNull from 'lodash/isNull';
 import isUndefined from 'lodash/isUndefined';
 import moment from 'moment';
@@ -74,6 +75,12 @@ const ChangelogModal = (props: Props) => {
     }
   }, [activeVersionIndex]);
 
+  const removeEmptyVersions = (data: ChangeLog[]): ChangeLog[] => {
+    return data.filter(
+      (item: ChangeLog) => !isNull(item.changes) && !isUndefined(item.changes) && !isEmpty(item.changes)
+    );
+  };
+
   useEffect(() => {
     // We load correct active version after rendering modal
     if (openStatus && changelog && isUndefined(activeVersionIndex)) {
@@ -109,7 +116,7 @@ const ChangelogModal = (props: Props) => {
     try {
       setIsLoading(true);
       setCurrentPkgId(props.packageId);
-      setChangelog(await API.getChangelog(props.packageId));
+      setChangelog(removeEmptyVersions(await API.getChangelog(props.packageId)));
       setIsLoading(false);
       setOpenStatus(true);
     } catch {