Merge pull request #30 from vtfk/preview

Only contactTeacher and groupAccess can view files

README.md

@@ -39,6 +39,9 @@ REACT_APP_SENTRY_DSN=%dsn-url%
 REACT_APP_SENTRY_ENV=production
 ```
 
+## Vercel preview
+If you need to test your code before deploying to Vercel, commit and push your changes to a branch named "preview". This will generate a preview deployment using the dev-api. The url for visiting the preview is [generated based on the branch name](https://vercel.com/docs/concepts/deployments/generated-urls#url-with-git-branch), and the branch-genereated url is also set up in the app registration for redirection, so you can test authentication as well.
+
 ## Sentry
 
 This application uses [Sentry](http://sentry.io/) to log FrontEnd erros. To activate Sentry logging, add correct *dsn* (Data Source Name) from your Sentry project into `REACT_APP_SENTRY_DSN` in your `.env` file. *Environment is used from `NODE_ENV`, or it can be specified in `REACT_APP_SENTRY_ENV`.

src/Layout/index.js

@@ -25,7 +25,7 @@ export function Layout () {
             <img alt='logo' src='/logo.png' height='40px' width='40px' />
           </div>
           <div className='text' onClick={() => navigate('/')}>
-            Elevmappa
+            Elevdok (tidligere Elevmappa)
           </div>
         </div>
 
@@ -62,7 +62,14 @@ export function Layout () {
           showInfoMsg &&
             <div className='infoBox'>
               <p>
-                <strong>Info: </strong>I forbindelse med splittingen av fylket er det satt i gang en jobb med å se på tilgangsstyringer i ulike digitale løsninger. Vi har i denne sammenhengen startet arbeidet med å se på tilgangsstyringen i blant annet Elevmappa. Som resultat av dette arbeidet vil det bli innført noen endringer i elevmappa fra og med <strong>fredag 31. mars kl. 15:00</strong>. Dere vil få mer informasjon om denne endringen ved egen skole. 
+                <strong>Informasjon om endring i tilgangsstyring</strong><br />
+                <ul>
+                  <li>Dette er en løsning som gir oversikt over eller lesetilgang til elevdokumenter som ligger arkivert i fylkets sak -og arkivsystem ("P360").</li>
+                  <li>Kontaktlærer har lesetilgang til elevdokumenter tilknyttet kontaktelever.</li>
+                  <li>Faglærer får se en oversikt over titlene på elevdokumentene, og må derfor henvende seg til sin nærmeste leder ved tjenstlig behov for å lese et dokument. Leder åpner dokumentet i "P360" og lar faglærer lese det.</li>
+                  <li>Alle oppslag i elevdokumenter loggføres av systemet.</li>
+                  <li>Løsningen har fått nytt navn, og heter nå "Elevdok".</li>
+                </ul>
               </p>
               <p class="butt" onClick={() => { setShowInfoMsg(false) }}>Lukk info</p>
             </div>

src/Pages/Student/index.js

@@ -14,15 +14,18 @@ import { useAPI } from '../../hooks/useAPI'
 import './style.scss'
 
 const PersonInfo = ({ person }) => {
+  const lastNameWithContactTeacher = person.contactTeacher ? `${person.lastName} (KL)` : person.lastName
+  const contactTeacherTxt = person.contactTeacher ? 'Du er kontaktlærer' : ''
   return (
     <PersonCard
       firstName={person.firstName}
-      lastName={person.lastName}
+      lastName={lastNameWithContactTeacher}
       largeName={person.fullname}
     >
       <div className='extra-info'>
         <span>{person.mail}</span>
         <span>{person.mainGroupName}</span>
+        <span>{contactTeacherTxt}</span>
       </div>
     </PersonCard>
   )
@@ -79,6 +82,10 @@ export function Student () {
     return file.title
   }
 
+  const canViewFile = (accessType, contactTeacher) => {
+    return accessType === 'groupAccess' || contactTeacher
+  }
+
   function onDocumentLoadSuccess ({ numPages }) {
     setNumPages(numPages)
   }
@@ -104,30 +111,55 @@ export function Student () {
                     return (
                       <div className='document' key={index}>
                         <div className='document-header'>{document.title}</div>
+                        <InfoProp header='Ansvarlig virksomhet' value={document.responsibleEnterprise || 'Ukjent'} />
                         <InfoProp header='Sendt dato' value={document.displayDate} />
                         <InfoProp header='Dok. nr.' value={`${document.documentNumber}${document.source ? ` (${document.source})` : ''}` || 'Ukjent'} />
+                        <InfoProp header='Dokumentarkiv' value={document.documentArchive || 'Ukjent'} />
                         <InfoProp header='Dokumentkategori' value={document.category || 'Ukjent'} />
                         <InfoProp header='Tilgangskode' value={document.accessCodeDescription || 'Ukjent'} />
                         <InfoProp header='Fra' value={getContactNames(document.contacts, 'Avsender')} />
                         <InfoProp header='Til' value={getContactNames(document.contacts, 'Mottaker')} />
                         <div className='toggle' onClick={() => toggleExpandedDocument(document.documentNumber)}>
                           <Icon name={expandedDocument === document.documentNumber ? 'chevronUp' : 'chevronDown'} size='small' /> Filer ({document.files.length})
                         </div>
+                        {
+                          expandedDocument === document.documentNumber && !canViewFile(items.accessType, items.contactTeacher) &&
+                          <div className="accessMessage">
+                            <br />
+                            <em><strong>OBS!</strong> {`Du er ikke kontaktlærer for ${items.firstName} ${items.lastName} og har derfor ikke tilgang til å åpne filer`}</em>
+                          </div>
+                        }
                         {
                           expandedDocument === document.documentNumber && document.files.map(file => {
-                            return (
-                              <IconButton
-                                className='file'
-                                key={file.recno}
-                                bordered
-                                icon={documentFileLoading.error ? 'error' : 'pdf'}
-                                spinner={documentFileLoading.recno === file.recno && !documentFileLoading.error}
-                                disabled={document.disableFiles || (documentFileLoading.recno === file.recno && !documentFileLoading.error)}
-                                onClick={() => { getDocumentFile(document, { file: document.documentNumber, recno: file.recno }) }}
-                                title={document.disableFiles ? 'Fil kun tilgjengelig i arkiv' : 'Klikk for å åpne filen'}
-                              >{getFileTitle(document, file)}
-                              </IconButton>
-                            )
+                            if (canViewFile(items.accessType, items.contactTeacher)) {
+                              return (
+                                <IconButton
+                                  className='file'
+                                  key={file.recno}
+                                  bordered
+                                  icon={documentFileLoading.error ? 'error' : 'pdf'}
+                                  spinner={documentFileLoading.recno === file.recno && !documentFileLoading.error}
+                                  disabled={document.disableFiles || (documentFileLoading.recno === file.recno && !documentFileLoading.error)}
+                                  onClick={() => { getDocumentFile(document, { file: document.documentNumber, recno: file.recno }) }}
+                                  title={document.disableFiles ? 'Fil kun tilgjengelig i arkiv' : 'Klikk for å åpne filen'}
+                                >{getFileTitle(document, file)}
+                                </IconButton>
+                              )
+                            } else {
+                              return (
+                                <IconButton
+                                  className='file'
+                                  key={file.recno}
+                                  bordered
+                                  icon={documentFileLoading.error ? 'error' : 'pdf'}
+                                  spinner={documentFileLoading.recno === file.recno && !documentFileLoading.error}
+                                  disabled={true}
+                                  onClick={() => { getDocumentFile(document, { file: document.documentNumber, recno: file.recno }) }}
+                                  title={`Du er ikke kontaktlærer for ${items.firstName} ${items.lastName} og har derfor ikke tilgang til å åpne filen`}
+                                >{getFileTitle(document, file)}
+                                </IconButton>
+                              )
+                            }
                           })
                         }
 

src/Pages/Students/index.js

@@ -57,7 +57,7 @@ export function Students () {
                   (students.length > config.studentsPrPage ? students.slice(paginationFrom, paginationTo) : students).map((student, index) => {
                     return (
                       <div className='student' key={index} onClick={() => handleStudentClick(student.userName)}>
-                        <div className='student-name' title='Navn'>{student.fullName}</div>
+                        <div className='student-name' title={student.contactTeacher ? 'Du er kontaktlærer for denne eleven' : 'Du er ikke kontaktlærer for denne eleven'}>{student.fullName} <strong>{student.contactTeacher ? '(KL)' : ''}</strong></div>
                         <div className='student-class' title='Klasse'>{student.mainGroupName}</div>
                       </div>
                     )