Fixed the sorting and versioning on the file versions

CenterForOpenScience · Dec 6, 2023 · 24c59df · 24c59df
1 parent 317eb91
commit 24c59df
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 3 deletions.
diff --git a/app/preprints/-components/preprint-file-render/component.ts b/app/preprints/-components/preprint-file-render/component.ts
@@ -43,7 +43,9 @@ export default class PreprintFileRender extends Component<InputArgs> {
     @task
     @waitFor
     private async loadPrimaryFileVersions()  {
-        const primaryFileVersions = (await this.primaryFile.versions).toArray();
+        const primaryFileVersions = (await this.primaryFile.queryHasMany('versions', {
+            sort: '-id', 'page[size]': 50,
+        })).toArray();
         this.serializeVersions(primaryFileVersions);
         this.primaryFileHasVersions = primaryFileVersions.length > 0;
     }

diff --git a/mirage/factories/file-version.ts b/mirage/factories/file-version.ts
@@ -9,6 +9,7 @@ export default Factory.extend<FileVersionModel>({
     id: guid('file - version'),
     afterCreate: (fileVersion/* , server: Server */) => {
         fileVersion.update({
+            id : faker.random.number(1000),
             size: faker.random.number(1000),
             contentType: 'application/octet-stream',
             dateCreated: faker.date.past(2, new Date(2018, 0, 0)),

diff --git a/mirage/factories/preprint.ts b/mirage/factories/preprint.ts
@@ -87,8 +87,7 @@ export default Factory.extend<PreprintMirageModel & PreprintTraits>({
                 delete: 'http://localhost:4200/assets/osf-assets/mfr-test.pdf',
                 html: 'http://localhost:4200/assets/osf-assets/mfr-test.pdf',
                 upload: 'http://localhost:4200/assets/osf-assets/mfr-test.pdf',
-                // download: 'http://localhost:4200/assets/osf-assets/mfr-test.pdf',
-                download: 'http://localhost:4200/assets/osf-assets/amazon.png',
+                download: 'http://localhost:4200/assets/osf-assets/mfr-test.pdf',
             },
         });