Merge pull request #4221 from systeminit/victor/bug-463-issue-with-mo…

…dule-sync-not-understanding-past-hashes Compare upgradeable modules to builtins list
systeminit · Jul 24, 2024 · daa29cb · daa29cb
2 parents bbe6071 + 981bed3
commit daa29cb
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/lib/module-index-client/src/lib.rs b/lib/module-index-client/src/lib.rs
@@ -290,6 +290,7 @@ impl ModuleIndexClient {
             .await?)
     }
 
+    // Will skip builtins
     pub async fn list_module_details(&self) -> ModuleIndexClientResult<ListModulesResponse> {
         let url = self.base_url.join("modules")?;
 

diff --git a/lib/sdf-server/src/server/service/v2/module/sync.rs b/lib/sdf-server/src/server/service/v2/module/sync.rs
@@ -34,7 +34,7 @@ pub async fn sync(
             ModuleIndexClient::new(module_index_url.try_into()?, &raw_access_token);
         (
             module_index_client.list_latest_modules().await?,
-            module_index_client.list_module_details().await?,
+            module_index_client.list_builtins().await?,
         )
     };