Merge pull request #20 from randombenj/bugfix/show-version

fix(web): do not show invalid versions
docat-org · Nov 12, 2019 · 1ade830 · 1ade830
2 parents cb7b868 + 7042e39
commit 1ade830
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 4 deletions.
diff --git a/web/src/components/Project.vue b/web/src/components/Project.vue
@@ -30,7 +30,7 @@ export default {
   },
   async created() {
     this.logoURL = ProjectRepository.getProjectLogoURL(this.project)
-    this.versions = (await ProjectRepository.getVersions(this.project)).data
+    this.versions = await ProjectRepository.getVersions(this.project)
     this.latestVersion = (this.versions.find((version) => version.name == 'latest') || this.versions[0]).name
   }
 }

diff --git a/web/src/pages/Docs.vue b/web/src/pages/Docs.vue
@@ -45,7 +45,7 @@ export default {
   async created() {
     this.versions = (await ProjectRepository.getVersions(
       this.$route.params.project
-    )).data.map((version) => version.name)
+    )).map((version) => version.name)
     this.docURL = this.$route.params.location || ProjectRepository.getProjectDocsURL(
       this.$route.params.project,
       this.$route.params.version

diff --git a/web/src/repositories/ProjectRepository.js b/web/src/repositories/ProjectRepository.js
@@ -32,7 +32,9 @@ export default {
    * this includes mainly the existing versions
    * @param {string} projectName Name of the project
    */
-  getVersions(projectName) {
-    return Repository.get(`${Repository.defaults.baseURL}/${resource}/${projectName}/`)
+  async getVersions(projectName) {
+    return (await Repository.get(`${Repository.defaults.baseURL}/${resource}/${projectName}/`))
+      .data
+      .filter((version) => version.type == 'directory')
   }
 }