feat: more info for imported releases under review

also fix some potential type errors in importing

Author: sgfost

django/library/github_integration.py

@@ -548,7 +548,7 @@ def __init__(self, remote: CodebaseGitRemote, github_release_id: str | int):
         self._reimporting = False
 
         if not self.sync_state.download_url:
-            return self.log_failure("No zipball found in the github release")
+            raise ValueError("No zipball found in the github release")
 
     @property
     def installation_token(self):

django/library/metadata.py

@@ -448,11 +448,19 @@ def _extract_os_from_codemeta(self) -> str:
         if not codemeta_os:
             return ""
         if isinstance(codemeta_os, list):
-            os_str = codemeta_os[0]
-        elif isinstance(codemeta_os, str):
-            os_str = codemeta_os
+            if not codemeta_os:
+                return ""
+            os_value = codemeta_os[0]
         else:
-            return ""
+            os_value = codemeta_os
+
+        if isinstance(os_value, str):
+            os_str = os_value
+        else:
+            # try to extract a name from a structured object (e.g., {"name": "Linux"})
+            os_name = self._get_field(os_value, "name")
+            os_str = os_name if isinstance(os_name, str) else ""
+
         if os_str:
             normalized = os_str.lower()
             # attempt to match the given os string to a known platform

django/library/tasks.py

@@ -71,9 +71,7 @@ def import_github_release_task(
                 remote=remote, github_release_id=str(github_release_id)
             ).first()
             if sync_state:
-                sync_state.status = ImportedReleaseSyncState.Status.ERROR
-                sync_state.error_message = str(e)
-                sync_state.save()
+                sync_state.log_failure(str(e))
         except Exception:
             logger.exception("Failed to update sync state after import error")
         logger.exception("Failed to import GitHub release asynchronously: %s", e)

frontend/src/components/githubIntegration/LocalReleaseItem.vue

@@ -9,21 +9,31 @@
             <i class="fas fa-lock"></i> Private
           </span>
         </h6>
-        <span v-if="release.status === 'UNDER_REVIEW'" class="badge bg-danger">Under Review</span>
+        <span v-if="release.status.toLowerCase() === 'under_review'" class="badge bg-danger"
+          >Under Review</span
+        >
       </div>
       <div class="text-muted small">
         <span v-if="!release.live">
-          <span v-if="release.importedReleaseSyncState" class="float-end">
-            <a :href="editHref"><i class="fas fa-edit"></i> Verify import</a>
+          <span class="float-end">
+            <a :href="editHref">
+              <span v-if="release.reviewStatus?.toLowerCase() === 'awaiting_author_changes'">
+                <i class="fas fa-bell"></i> Verify import and notify reviewers
+              </span>
+              <span v-else-if="release.importedReleaseSyncState"
+                ><i class="fas fa-edit"></i> Verify import</span
+              >
+            </a>
             <BootstrapTooltip
+              v-if="release.importedReleaseSyncState"
               class="ms-1"
-              title="Releases imported from GitHub will be unpublished until you verify and correct metadata and file categorization. Afterwards, you can publish OR request peer review."
+              :title="tooltipTitle"
               icon-class="fas fa-question-circle text-muted"
               placement="bottom"
             >
             </BootstrapTooltip>
+            <span v-else class="small">Only published releases are included in git repos</span>
           </span>
-          <span v-else class="small">Only published releases are included in git repos</span>
         </span>
         <span v-else>Published {{ release.lastPublishedOn }}</span>
       </div>
@@ -51,6 +61,13 @@ const editHref = computed(() =>
   editUrl(props.release.codebase.identifier, props.release.versionNumber)
 );
 
+const tooltipTitle = computed(() => {
+  if (props.release.status.toLowerCase() === "under_review") {
+    return "Releases being reviewed must remain unpublished until complete. If revisions are requested, you will need to update the GitHub release tag (see tutorial for help), reimport, and then notify reviewers.";
+  }
+  return "Releases imported from GitHub will be unpublished until you verify and correct metadata and file categorization. Afterwards, you can publish OR request peer review.";
+});
+
 const isOriginal = computed(() => !props.release.importedReleaseSyncState);
 const itemClass = computed(() => {
   return [