Merge pull request #140 from nikomatsakis/progress-bars

small fixes to hopefully make it work

Author: nikomatsakis

mdbook-goals/src/json.rs

@@ -157,8 +157,6 @@ fn checkboxes(issue: &ExistingGithubIssue) -> anyhow::Result<Progress> {
         }
     }
 
-    eprintln!("#{}: {checkboxes:?}, {issues:?}", issue.number);
-
     match (checkboxes, issues) {
         (Some((completed, total)), None) | (None, Some((completed, total))) => {
             Ok(Progress::Tracked { completed, total })

src/update-progress-bars.js

@@ -41,11 +41,8 @@
             }
 
             if (!(milestone in this.#dataMap)) {
-                // Construct the URL using the dirName
-                const url = `/api/${milestone}.json`;
-
                 try {
-                    const response = await fetch(url);
+                    const response = await fetch(getApiUrl(milestone));
                     if (!response.ok) {
                         throw new Error(`HTTP error! status: ${response.status}`);
                     }
@@ -144,4 +141,15 @@
         }
         return element;
     }
+
+    function getApiUrl(milestone) {
+        // When you are doing mdbook serve, you don't want /rust-project/goals.
+        // But on github pages, you do.
+        // There is probably a more elegant way to do this.
+        const currentPath = document.location.pathname;
+        const basePath = currentPath.startsWith('/rust-project-goals/')
+            ? '/rust-project-goals/'
+            : '/';
+        return `${basePath}api/${milestone}.json`;
+    }
 })();