@openneuro/app 5.1.1 → 5.1.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@openneuro/app",
-  "version": "5.1.1",
+  "version": "5.1.3",
   "description": "React JS web frontend for the OpenNeuro platform.",
   "license": "MIT",
   "main": "public/client.js",
@@ -79,5 +79,5 @@
   "publishConfig": {
     "access": "public"
   },
-  "gitHead": "035b0b4544d4b287506ea00aff3b424ed816611a"
+  "gitHead": "b1059b286940adb3b60150217e691fe75a895d94"
 }

package/src/scripts/types/user-types.ts

@@ -68,6 +68,7 @@ export interface DatasetSnapshot {
     DatasetDOI?: string | null
     Name: string
   }
+  readme?: string
   summary?: {
     primaryModality?: string
   }

package/src/scripts/users/__tests__/dataset-card.spec.tsx

@@ -81,7 +81,7 @@ const mockDataset = {
 describe("DatasetCard", () => {
   it("should render dataset information correctly", () => {
     render(<DatasetCard dataset={mockDataset} hasEdit={false} />)
-    expect(screen.getByText("Test Dataset")).toBeInTheDocument()
+    expect(screen.getByText("Test Dataset Description")).toBeInTheDocument()
     expect(screen.getByText("ds000001")).toBeInTheDocument()
   })
 
@@ -96,7 +96,7 @@ describe("DatasetCard", () => {
   it("should show the dataset if not public but hasEdit is true", () => {
     const privateDataset = { ...mockDataset, public: false }
     render(<DatasetCard dataset={privateDataset} hasEdit={true} />)
-    expect(screen.getByText("Test Dataset")).toBeInTheDocument()
+    expect(screen.getByText("Test Dataset Description")).toBeInTheDocument()
   })
 
   it("should render activity details correctly", () => {

package/src/scripts/users/dataset-card.tsx

@@ -98,7 +98,8 @@ export const DatasetCard: React.FC<DatasetCardProps> = (
     >
       <h4>
         <a href={`/datasets/${dataset.id}`}>
-          {dataset.name ? dataset.name : dataset.id}
+          {dataset.latestSnapshot?.description?.Name || dataset.name ||
+            dataset.id}
         </a>
       </h4>
       <div className={styles.userDsBody}>

package/src/scripts/utils/user-datasets.tsx

@@ -10,9 +10,19 @@ export function filterAndSortDatasets(
   { searchQuery, publicFilter, sortOrder }: FilterOptions,
 ): Dataset[] {
   const filteredDatasets = datasets.filter((dataset) => {
+    const datasetName = dataset.latestSnapshot?.description?.Name ||
+      dataset.name
+    const datasetAuthors = dataset.latestSnapshot?.description?.Authors || []
+    const datasetReadme = dataset.latestSnapshot?.readme || ""
+
     const matchesSearch = searchQuery === "" ||
-      (dataset.name &&
-        dataset.name.toLowerCase().includes(searchQuery.toLowerCase())) ||
+      (datasetName &&
+        datasetName.toLowerCase().includes(searchQuery.toLowerCase())) ||
+      datasetAuthors.some((author: string) =>
+        author.toLowerCase().includes(searchQuery.toLowerCase())
+      ) ||
+      (datasetReadme &&
+        datasetReadme.toLowerCase().includes(searchQuery.toLowerCase())) ||
       (dataset.id &&
         dataset.id.toLowerCase().includes(searchQuery.toLowerCase()))
 
@@ -30,13 +40,19 @@ export function filterAndSortDatasets(
     // Move declarations outside the switch block
     let aUpdated: Date | string
     let bUpdated: Date | string
+    let aName: string
+    let bName: string
 
     switch (sortOrder) {
       case "name-asc":
-        comparisonResult = (a.name || "").localeCompare(b.name || "")
+        aName = a.latestSnapshot?.description?.Name || a.name || ""
+        bName = b.latestSnapshot?.description?.Name || b.name || ""
+        comparisonResult = aName.localeCompare(bName)
         break
       case "name-desc":
-        comparisonResult = (b.name || "").localeCompare(a.name || "")
+        aName = a.latestSnapshot?.description?.Name || a.name || ""
+        bName = b.latestSnapshot?.description?.Name || b.name || ""
+        comparisonResult = bName.localeCompare(aName)
         break
       case "date-newest":
         comparisonResult =