197 display user icon name (#591)

* feat: add user profile and usernames * feat: changed updated usernames, and fixed studysets page, better base studies support * fix: removed unnecessary imports and updated openapi spec, fixed failing tests * fix: resolve failing cypress test * fix: added fixture to fix failing cypress test * fix: update cypress to v12 due to issue with chrome 117
neurostuff · Sep 21, 2023 · 76ec881 · 76ec881
1 parent de0d47d
commit 76ec881
Show file tree

Hide file tree

Showing 31 changed files with 599 additions and 339 deletions.
diff --git a/compose/neurosynth-frontend/cypress/e2e/pages/EditStudyPage.cy.tsx b/compose/neurosynth-frontend/cypress/e2e/pages/EditStudyPage.cy.tsx
@@ -2,7 +2,7 @@
 
 export {};
 
-const PATH = '/studies/mock-study-id/edit';
+const PATH = '/projects/abc123/extraction/studies/mock-study-id/edit';
 const PAGE_NAME = 'EditStudyPage';
 
 describe(PAGE_NAME, () => {
@@ -12,11 +12,11 @@ describe(PAGE_NAME, () => {
     });
 
     it('should load', () => {
-        cy.intercept('GET', `**/api/projects*`).as('realProjectsRequest');
+        cy.intercept('GET', `**/api/projects/*`, { fixture: 'project' }).as('projectFixture');
         cy.intercept('GET', `**/api/studies/mock-study-id*`, { fixture: 'study' }).as(
             'studyFixture'
         );
-        cy.visit(PATH).wait('@studyFixture');
+        cy.visit(PATH).wait('@projectFixture').wait('@studyFixture');
         // cy.login('mocked').wait('@realProjectsRequest').visit(PATH).wait('@studyFixture');
     });
 

diff --git a/compose/neurosynth-frontend/cypress/e2e/pages/PublicStudiesPage.cy.tsx b/compose/neurosynth-frontend/cypress/e2e/pages/PublicStudiesPage.cy.tsx
@@ -4,7 +4,7 @@ import { mockStudies } from 'testing/mockData';
 
 export {};
 
-const PATH = '/studies';
+const PATH = '/base-studies';
 const PAGE_NAME = 'StudiesPage';
 
 describe(PAGE_NAME, () => {

diff --git a/compose/neurosynth-frontend/cypress/e2e/pages/StudyPage.cy.tsx b/compose/neurosynth-frontend/cypress/e2e/pages/StudyPage.cy.tsx
@@ -22,14 +22,8 @@ describe(PAGE_NAME, () => {
         cy.intercept('GET', `**/api/studies/**`, {
             fixture: 'study',
         }).as('studyFixture');
-        cy.intercept('GET', `**/api/base-studies/**`, {
-            fixture: 'baseStudy',
-        }).as('baseStudyFixture');
-
-        cy.visit(PATH)
-            .wait('@semanticScholarFixture')
-            .wait('@studyFixture')
-            .wait('@baseStudyFixture');
+
+        cy.visit(PATH).wait('@semanticScholarFixture').wait('@studyFixture');
         // .get('tr')
         // .eq(2)
         // .click()

diff --git a/compose/neurosynth-frontend/cypress/fixtures/project.json b/compose/neurosynth-frontend/cypress/fixtures/project.json
@@ -0,0 +1,47 @@
+{
+    "created_at": "2023-09-14T18:04:28.079423+00:00",
+    "description": "",
+    "id": "4HxF4g4WC5zj",
+    "meta_analyses": [],
+    "name": "My project name",
+    "neurostore_study": {
+      "created_at": "2023-09-14T18:04:28.090521+00:00",
+      "exception": null,
+      "neurostore_id": "4PG9x8gYaoWd",
+      "status": "PENDING",
+      "traceback": null,
+      "updated_at": "2023-09-14T18:04:28.111166+00:00"
+    },
+    "neurostore_url": "https://neurostore.org/api/studies/4PG9x8gYaoWd",
+    "provenance": {
+      "curationMetadata": {
+        "columns": [],
+        "exclusionTags": [],
+        "identificationSources": [],
+        "infoTags": [],
+        "prismaConfig": {
+          "eligibility": {
+            "exclusionTags": []
+          },
+          "identification": {
+            "exclusionTags": []
+          },
+          "isPrisma": false,
+          "screening": {
+            "exclusionTags": []
+          }
+        }
+      },
+      "extractionMetadata": {
+        "studyStatusList": []
+      },
+      "metaAnalysisMetadata": {
+        "canEditMetaAnalyses": false
+      }
+    },
+    "public": false,
+    "updated_at": "2023-09-14T18:04:34.184976+00:00",
+    "user": "github|12387943",
+    "username": "Nicholas Lee"
+  }
+
diff --git a/compose/neurosynth-frontend/package-lock.json b/compose/neurosynth-frontend/package-lock.json
diff --git a/compose/neurosynth-frontend/package.json b/compose/neurosynth-frontend/package.json
@@ -90,7 +90,7 @@
         "@testing-library/user-event": "^12.8.3",
         "@types/body-scroll-lock": "^3.1.0",
         "@types/node": "^18.6.1",
-        "cypress": "^10.3.1",
+        "cypress": "^12.15.0",
         "env-cmd": "^10.1.0",
         "eslint-config-prettier": "^8.3.0",
         "eslint-plugin-cypress": "^2.12.1",

diff --git a/...urationComponents/CurationImport/CurationImport/ImportFromNeurostore/NeurostoreSearch.tsx b/...urationComponents/CurationImport/CurationImport/ImportFromNeurostore/NeurostoreSearch.tsx
@@ -190,7 +190,7 @@ const NeurostoreSearch: React.FC<IImportArgs> = (props) => {
                                 data-tour={index === 0 ? 'StudiesPage-4' : null}
                                 sx={NeurosynthTableStyles.tableRow}
                                 key={studyrow.id || index}
-                                onClick={() => history.push(`/studies/${studyrow.id}`)}
+                                onClick={() => history.push(`/base-studies/${studyrow.id}`)}
                             >
                                 <TableCell>
                                     {studyrow?.name || (
@@ -207,10 +207,7 @@ const NeurostoreSearch: React.FC<IImportArgs> = (props) => {
                                         <Box sx={{ color: 'warning.dark' }}>No Journal</Box>
                                     )}
                                 </TableCell>
-                                <TableCell>
-                                    {(studyrow?.user === user?.sub ? 'Me' : studyrow?.user) ||
-                                        'Neurosynth-Compose'}
-                                </TableCell>
+                                <TableCell>{studyrow?.username || 'Neurosynth-Compose'}</TableCell>
                             </TableRow>
                         ))}
                     />

diff --git a/compose/neurosynth-frontend/src/components/EditAnnotations/EditAnnotations.tsx b/compose/neurosynth-frontend/src/components/EditAnnotations/EditAnnotations.tsx
@@ -4,11 +4,6 @@ import { DetailedSettings as MergeCellsSettings } from 'handsontable/plugins/mer
 import { ColumnSettings } from 'handsontable/settings';
 import { useGetAnnotationById, useUpdateAnnotationById } from 'hooks';
 import { NoteCollectionReturn } from 'neurostore-typescript-sdk';
-import {
-    useInitProjectStoreIfRequired,
-    useProjectExtractionAnnotationId,
-} from 'pages/Projects/ProjectPage/ProjectStore';
-import { useInitStudyStoreIfRequired } from 'pages/Studies/StudyStore';
 import { useCallback, useEffect, useRef, useState } from 'react';
 import AnnotationsHotTable from './AnnotationsHotTable/AnnotationsHotTable';
 import {
@@ -21,17 +16,24 @@ import {
     noteKeyArrToObj,
     noteKeyObjToArr,
 } from './helpers/utils';
+import { useAuth0 } from '@auth0/auth0-react';
+import { useSnackbar } from 'notistack';
 
 const hardCodedColumns = ['Study', 'Analysis'];
 
-const EditAnnotations: React.FC = (props) => {
-    const annotationId = useProjectExtractionAnnotationId();
-
-    useInitProjectStoreIfRequired();
-    useInitStudyStoreIfRequired();
+const EditAnnotations: React.FC<{ annotationId: string }> = (props) => {
+    const { user } = useAuth0();
+    const { enqueueSnackbar } = useSnackbar();
+    const { mutate, isLoading: updateAnnotationIsLoading } = useUpdateAnnotationById(
+        props.annotationId
+    );
+    const {
+        data,
+        isLoading: getAnnotationIsLoading,
+        isError,
+    } = useGetAnnotationById(props.annotationId);
 
-    const { mutate, isLoading: updateAnnotationIsLoading } = useUpdateAnnotationById(annotationId);
-    const { data, isLoading: getAnnotationIsLoading, isError } = useGetAnnotationById(annotationId);
+    const theLoggedInUserOwnsThisAnnotation = (user?.sub || null) === (data?.user || undefined);
 
     // tracks the changes made to hot table
     const hotTableDataUpdatesRef = useRef<{
@@ -90,7 +92,13 @@ const EditAnnotations: React.FC = (props) => {
     }, [data]);
 
     const handleClickSave = () => {
-        if (!annotationId) return;
+        if (!props.annotationId) return;
+        if (!theLoggedInUserOwnsThisAnnotation) {
+            enqueueSnackbar('You do not have permission to edit this annotation', {
+                variant: 'error',
+            });
+            return;
+        }
 
         const { hotData, noteKeys } = hotTableDataUpdatesRef.current;
 
@@ -103,7 +111,7 @@ const EditAnnotations: React.FC = (props) => {
 
         mutate(
             {
-                argAnnotationId: annotationId,
+                argAnnotationId: props.annotationId,
                 annotation: {
                     notes: updatedAnnotationNotes.map((annotationNote) => ({
                         note: annotationNote.note,

diff --git a/compose/neurosynth-frontend/src/components/Navbar/NavDrawer/NavDrawer.tsx b/compose/neurosynth-frontend/src/components/Navbar/NavDrawer/NavDrawer.tsx
@@ -89,7 +89,7 @@ const NavDrawer: React.FC<INav> = (props) => {
                     <DrawerToggleMenu labelText="EXPLORE">
                         <List>
                             <ListItem>
-                                <ListItemButton onClick={() => history.push('/studies')}>
+                                <ListItemButton onClick={() => history.push('/base-studies')}>
                                     <ListItemIcon />
                                     <ListItemText primary="studies" />
                                 </ListItemButton>