Merge pull request #31 from saadbruno/latest-release-date

we now consider the date of the last release on github
saadbruno · Sep 12, 2022 · 7d460f1 · 7d460f1
2 parents 770ca59 + bb2165b
commit 7d460f1
Showing 1 changed file with 11 additions and 2 deletions.
diff --git a/code/lib/db/submit.php b/code/lib/db/submit.php
@@ -58,6 +58,9 @@ function savePlugin($githubUrl)
         avatarUrl
         url
       }
+      latestRelease {
+        publishedAt
+      }
       readme1: object(expression: "master:readme.md") {
         ... on Blob {
           text
@@ -125,8 +128,14 @@ function savePlugin($githubUrl)
     return json_encode($result);
   }
 
-  // converts the timestamp
-  $updatedAt =  date("U", strtotime($result['data']['repository']['updatedAt']));
+  // Let's get the latest update timestamp for the repo.
+  // If the repo has a release on GitHub, we use the release date. Otherwise we get the time of the latest commit
+  if (isset($result['data']['repository']['latestRelease']['publishedAt'])) {
+    $updatedAt =  date("U", strtotime($result['data']['repository']['latestRelease']['publishedAt']));
+  } else {
+    $updatedAt =  date("U", strtotime($result['data']['repository']['pushedAt']));
+  }
+
   $submittedAt = time();
 
   // converts OG image to boolean