Improving visualization of locked students on the Exam page.

Author: krulis-martin

src/components/Groups/LockedStudentsTable/LockedStudentsTable.js

@@ -17,17 +17,18 @@ const LockedStudentsTable = ({ groupId, lockedStudents, currentUser, intl: { loc
   lockedStudents && lockedStudents.length > 0 ? (
     <Table className="m-0" hover>
       <tbody>
-        {sortStudents(lockedStudents).map(student => (
+        {sortStudents(lockedStudents, locale).map(student => (
           <tr key={student.id}>
             <td>
               <UsersName
                 {...student}
                 currentUserId={currentUser.id}
-                showEmail="full"
+                showEmail="icon"
                 showExternalIdentifiers
                 listItem
               />
             </td>
+            <td>{student.privateData?.ipLock && <code>{student.privateData.ipLock}</code>}</td>
             <td className="text-end">
               <ExamUnlockButtonContainer groupId={groupId} userId={student.id} size="xs" />
             </td>

src/components/Groups/NotLockedStudentsTable/NotLockedStudentsTable.js

@@ -0,0 +1,74 @@
+import React from 'react';
+import PropTypes from 'prop-types';
+import { FormattedMessage, injectIntl } from 'react-intl';
+import { Table } from 'react-bootstrap';
+import { lruMemoize } from 'reselect';
+
+import UsersName from '../../Users/UsersName';
+import DateTime from '../../widgets/DateTime/DateTime';
+import Icon from '../../icons';
+import { createUserNameComparator } from '../../helpers/users.js';
+
+const sortStudents = lruMemoize((notLockedStudents, locale) => {
+  const sorted = [...notLockedStudents];
+  return sorted.sort(createUserNameComparator(locale));
+});
+
+const NotLockedStudentsTable = ({ notLockedStudents, currentUser, intl: { locale } }) =>
+  notLockedStudents && notLockedStudents.length > 0 ? (
+    <Table className="m-0" hover>
+      <tbody>
+        {sortStudents(notLockedStudents, locale).map(student => (
+          <tr key={student.id}>
+            <td>
+              <UsersName
+                {...student}
+                currentUserId={currentUser.id}
+                showEmail="icon"
+                showExternalIdentifiers
+                listItem
+              />
+            </td>
+            <td className="text-nowrap shrink-col small text-body-secondary align-middle">
+              {student.privateData?.lastAuthenticationAt && (
+                <>
+                  <Icon
+                    icon="right-to-bracket"
+                    gapRight
+                    tooltipId={student.id}
+                    tooltipPlacement="bottom"
+                    tooltip={
+                      <FormattedMessage
+                        id="app.notLockedStudentsTable.lastAuthentication"
+                        defaultMessage="Last time the user was authenticated"
+                      />
+                    }
+                  />
+                  <DateTime unixts={student.privateData.lastAuthenticationAt} showSeconds />
+                </>
+              )}
+            </td>
+          </tr>
+        ))}
+      </tbody>
+    </Table>
+  ) : (
+    <div className="text-center text-body-secondary p-4">
+      <em>
+        <FormattedMessage
+          id="app.notLockedStudentsTable.noNotLockedStudents"
+          defaultMessage="There are no remaining unlocked students."
+        />
+      </em>
+    </div>
+  );
+
+NotLockedStudentsTable.propTypes = {
+  notLockedStudents: PropTypes.array,
+  currentUser: PropTypes.shape({
+    id: PropTypes.string.isRequired,
+  }),
+  intl: PropTypes.object.isRequired,
+};
+
+export default injectIntl(NotLockedStudentsTable);

src/components/Groups/NotLockedStudentsTable/index.js

@@ -0,0 +1,2 @@
+import NotLockedStudentsTable from './NotLockedStudentsTable.js';
+export default NotLockedStudentsTable;

src/components/buttons/ExamUnlockButton/ExamUnlockButton.js

@@ -4,7 +4,7 @@ import { FormattedMessage } from 'react-intl';
 import { UnlockIcon, LoadingIcon } from '../../icons';
 import Button from '../../widgets/TheButton';
 
-const ExamUnlockButton = ({ pending, disabled = false, unlockUserForExam, ...props }) => (
+const ExamUnlockButton = ({ pending = false, disabled = false, unlockUserForExam, ...props }) => (
   <Button
     {...props}
     variant={disabled ? 'secondary' : 'warning'}
@@ -16,7 +16,7 @@ const ExamUnlockButton = ({ pending, disabled = false, unlockUserForExam, ...pro
 );
 
 ExamUnlockButton.propTypes = {
-  pending: PropTypes.bool.isRequired,
+  pending: PropTypes.bool,
   disabled: PropTypes.bool,
   unlockUserForExam: PropTypes.func.isRequired,
 };

src/containers/ExamUnlockButtonContainer/ExamUnlockButtonContainer.js

@@ -7,7 +7,6 @@ import { lruMemoize } from 'reselect';
 import ExamUnlockButton from '../../components/buttons/ExamUnlockButton';
 import { unlockStudentFromExam } from '../../redux/modules/groups.js';
 import { groupPendingUserUnlock } from '../../redux/selectors/groups.js';
-import { loggedInUserSelector } from '../../redux/selectors/users.js';
 import { getErrorMessage } from '../../locales/apiErrorMessages.js';
 import { addNotification } from '../../redux/modules/notifications.js';
 
@@ -45,7 +44,6 @@ ExamUnlockButtonContainer.propTypes = {
 
 const mapStateToProps = (state, { groupId }) => ({
   pending: groupPendingUserUnlock(state, groupId),
-  currentUser: loggedInUserSelector(state),
 });
 
 const mapDispatchToProps = (dispatch, { groupId, userId }) => ({

src/locales/cs.json

@@ -106,11 +106,6 @@
   "app.assignExerciseButton.isBroken": "Rozbitá",
   "app.assignExerciseButton.isLocked": "Zamčená",
   "app.assignExerciseButton.noRefSolutions": "Neověřená",
-  "app.assignmentStatusIcon.failed": "Žádné správné řešení nebylo zatím odevzdáno.",
-  "app.assignmentStatusIcon.inProgress": "Řešení úlohy je právě vyhodnocováno.",
-  "app.assignmentStatusIcon.isBestSolution": "Toto je nejlepší řešení, které autor dosud odevzdal.",
-  "app.assignmentStatusIcon.ok": "Úloha byla úspěšně splněna.",
-  "app.assignmentStatusIcon.solutionMissingSubmission": "Řešení nebylo odevzdáno k vyhodnocení pravděpodobně díky chybě. Budete ho muset odevzdat znovu.",
   "app.assignment.assignedAtExplanation": "Čas zadání úlohy ve skupině. Tento čas se nutně nemusí shodovat s časem, kdy byla úloha nastavena jako viditelná studentům.",
   "app.assignment.correctnessThreshold": "Hranice správnosti",
   "app.assignment.correctnessThresholdExplanation": "Minimální požadovaná správnost řešení, za kterou je možné udělit body. Řešení, která nedosahují této hranice, nedostanou žádné body.",
@@ -184,6 +179,11 @@
   "app.assignmentSolutions.viewModes.reviewRequests": "Žádosti o revize",
   "app.assignmentSolutions.viewModes.reviewed": "Pouze revidovaná řešení",
   "app.assignmentSolutions.viewModesTitle": "Vyberte způsob zobrazení řešení",
+  "app.assignmentStatusIcon.failed": "Žádné správné řešení nebylo zatím odevzdáno.",
+  "app.assignmentStatusIcon.inProgress": "Řešení úlohy je právě vyhodnocováno.",
+  "app.assignmentStatusIcon.isBestSolution": "Toto je nejlepší řešení, které autor dosud odevzdal.",
+  "app.assignmentStatusIcon.ok": "Úloha byla úspěšně splněna.",
+  "app.assignmentStatusIcon.solutionMissingSubmission": "Řešení nebylo odevzdáno k vyhodnocení pravděpodobně díky chybě. Budete ho muset odevzdat znovu.",
   "app.assignments.deadline": "Termín odevzdání",
   "app.assignments.deleteAllButton": "Smazat",
   "app.assignments.deleteAllButtonConfirm": "Opravdu si přejete smazat všechny vybrané zadané úlohy?",
@@ -374,14 +374,14 @@
   "app.editAssignmentForm.visibility.visible": "Viditelná studentům",
   "app.editAssignmentForm.visibility.visibleFrom": "Bude viditelná od daného data",
   "app.editAssignmentForm.visibleFrom": "Datum zveřejnění:",
+  "app.editAssignmentForm.warnings.alreadyAssigned": "Úloha je v této skupině již zadaná.",
+  "app.editAssignmentForm.warnings.alreadyAssignedGlobal": "Úloha již byla zadána do některých vybraných skupin. Po potvrzení formuláře bude tedy v některých skupinách úloha zadána vícekrát.",
+  "app.editAssignmentForm.warnings.canViewJudgeLogs": "Povolení zveřejnění logů sudího studentům sebou nese určitá rizika. U jednoduchých úloh může např. dojít k tomu, že studenti využijí tento kanál, aby získali vstupní data a očekávané výstupy jednotlivých testů a vyrobili triviální řešení, které má správné odpovědi přímo v kódu. Používejte tuto volbu uváživě.",
   "app.editAssignmentForm.warnings.chooseFirstDeadlineBeforeSecondDeadline": "Před nastavením druhého termínu odevzdání je nutné zvolit první termín.",
   "app.editAssignmentForm.warnings.interpolationGoesToZero": "Limit dosáhne nuly ještě před druhým deadline (viz graf).",
   "app.editAssignmentForm.warnings.pointsAreTheSame": "Oba bodové limity jsou stejné, takže není důvod používat nastavení se dvěma termíny odevzdání.",
   "app.editAssignmentForm.warnings.secondLimitIsGreaterThanFirstLimit": "Limit je větší než první limit, což je poněkud neobvyklé.",
   "app.editAssignmentForm.warnings.secondLimitIsZero": "Limit je nula, takže není důvod používat nastavení se dvěma termíny odevzdání.",
-  "app.editAssignmentForm.warnings.alreadyAssigned": "Úloha je v této skupině již zadaná.",
-  "app.editAssignmentForm.warnings.alreadyAssignedGlobal": "Úloha již byla zadána do některých vybraných skupin. Po potvrzení formuláře bude tedy v některých skupinách úloha zadána vícekrát.",
-  "app.editAssignmentForm.warnings.canViewJudgeLogs": "Povolení zveřejnění logů sudího studentům sebou nese určitá rizika. U jednoduchých úloh může např. dojít k tomu, že studenti využijí tento kanál, aby získali vstupní data a očekávané výstupy jednotlivých testů a vyrobili triviální řešení, které má správné odpovědi přímo v kódu. Používejte tuto volbu uváživě.",
   "app.editEnvironmentConfig.duplicateVariable": "Duplicitní jméno proměnné.",
   "app.editEnvironmentConfig.noRuntimeSelected": "Aby bylo možné pokračovat se zbytkem konfigurace, nejprve je třeba nastavit běhové prostředí.",
   "app.editEnvironmentConfig.noVariables": "Zatím nejsou nastavené žádné proměnné...",
@@ -1121,10 +1121,12 @@
   "app.groupExams.noExam": "V tuto chvíli není naplánovaná žádná zkouška",
   "app.groupExams.pending.studentLockedTitle": "Jste uzamčeném režimu pro probíhající zkoušku",
   "app.groupExams.pending.teacherInfo": "V tuto chvíli jsou zkouškové úlohy viditelné pouze studentům, kteří se uzamkli ve skupině.",
+  "app.groupExams.remainingStudentsBoxTitle": "Zbývající studenti (dosud nezamčení)",
   "app.groupExams.studentInfo": "Musíte se uzamknout ve skupině, abyste mohl(a) vidět zkouškové úlohy. Po čas uzamčení je komunikace s vámi omezena na vaši aktuální IP adresu.",
   "app.groupExams.studentInfoRegular": "Navíc budete moct přistupovat k ostatním skupinám pouze v režimu pro čtení dokud budete v uzamčeném režimu.",
   "app.groupExams.studentInfoStrict": "Navíc nebudete moct přistupovat k ostatním skupinám dokud budete v uzamčeném režimu.",
   "app.groupExams.studentsBoxTitle": "Studenti účastnící se zkoušky",
+  "app.groupExams.studentsCount": "({count, plural, one {# student} =2 {# studenti} =3 {# studenti} =4 {# studenti} other {# studentů}})",
   "app.groupExams.timeAccuracyWarning": "Lokální hodiny na vašem systému musí být dostatečně seřízené, jinak nemusí tato komponenta fungovat zcela správně.",
   "app.groupExams.title": "Zkouškové termíny skupiny",
   "app.groupExams.unlockStudentButton": "Odemknout",
@@ -1325,6 +1327,8 @@
   "app.notFound.description": "Oops, tohle jste pravděpodobně nehledali.",
   "app.notFound.text": "Buď jste do adresního řádku dostali špatné (případně staré) URL, nebo máme chybu v generátoru odkazů.",
   "app.notFound.title": "Stránka nenalezena",
+  "app.notLockedStudentsTable.lastAuthentication": "Kdy byl uživatel naposledy autenitzován",
+  "app.notLockedStudentsTable.noNotLockedStudents": "Nezbývají žádní nezamčení studenti.",
   "app.notifications.hideAll": "Pouze nové notifikace",
   "app.notifications.showAll": "Zobrazit {count, plural, one {jednu starou notifikaci} two {dvě staré notifikace} other {# starých notifikací}}",
   "app.notifications.title": "{count, plural, =0 {Nemáte žádnou novou notifikaci} one {Máte novou notifikaci} =2 {Máte dvě nové notifikace} =3 {Máte tři nové notifikace} =4 {Máte čtyři nové notifikace}  other {Máte # nových notifikací}}",
@@ -1763,6 +1767,7 @@
   "app.sidebar.menu.admin.messages": "Systémové zprávy",
   "app.sidebar.menu.admin.server": "Správa serveru",
   "app.sidebar.menu.admin.sis": "SIS integrace [stará]",
+  "app.sidebar.menu.admin.title": "Administrátor",
   "app.sidebar.menu.admin.users": "Uživatelé",
   "app.sidebar.menu.archive": "Archiv",
   "app.sidebar.menu.createAccount": "Zaregistrovat se",
@@ -2022,26 +2027,25 @@
   "app.submissions.testResultsTable.statusSkipped": "Přeskočeno",
   "app.submissions.testResultsTable.timeExceeded": "Naměřený čas spuštění",
   "app.submissions.testResultsTable.wallTimeExplain": "Reálný čas (tzv. 'wall-time' měřený systémovými hodinami)",
-  "app.submitSolution.instructions": "Musíte připojit alespoň jeden soubor se zdrojovým kódem a počkat, než jsou všechny soubory nahrány na server. Pokud nastane problém při uploadu některých soborů, nahrajte je znovu nebo soubory odeberte. Jméno souboru NESMÍ OBSAHOVAT žádné nestandardní znaky (například v kódování UTF-8).",
-  "app.submitSolution.submitFailed": "Odevzdané řešení bylo aktivně odmítnuto. Toto je obvykle způsobeno nahráním souborů se jménem obsahujícím nestandardní znaky nebo špatnou koncovkou. Pokud nemůžete odevzdat řešení bez zjevného důvodu, kontaktujte prosím svého vyučujícího.",
   "app.submitButton.invalid": "Nějaký vstup není validní",
   "app.submitButton.lastError.title": "Nastala chyba",
   "app.submitRefSolution.noteLabel": "Popis referenčního řešení",
   "app.submitRefSolution.title": "Vytvořit referenční řešení",
   "app.submitSolution.emptyNoteSubmitConfirm": "Popis referenčního řešení je prázdný. Je nanejvýš vhodné, aby referenční řešení byla opatřena relevantním popiskem. Opravdu si přejete pokračovat v odevzdání?",
   "app.submitSolution.emptyNoteWarning": "Popis referenčního řešení je prázdný. Je nanejvýš vhodné, aby referenční řešení byla opatřena relevantním popiskem.",
   "app.submitSolution.entryPoint": "Vyberte vstupní bod (zaváděcí soubor vaší aplikace):",
+  "app.submitSolution.instructions": "Musíte připojit alespoň jeden soubor se zdrojovým kódem a počkat, než jsou všechny soubory nahrány na server. Pokud nastane problém při uploadu některých soborů, nahrajte je znovu nebo soubory odeberte. Jméno souboru NESMÍ OBSAHOVAT žádné nestandardní znaky (například v kódování UTF-8).",
   "app.submitSolution.limitsExceeded": "Byly překročeny povolené limity souborů",
   "app.submitSolution.limitsExceededCount": "Nemůžete odevzdat více než {limit} {limit, plural, one {soubor} =2 {soubory} =3 {soubory} =4 {soubory} other {souborů}}.",
   "app.submitSolution.limitsExceededSize": "Celková velikost všech odevzdaných souborů nesmí překročit {limit} KiB.",
   "app.submitSolution.linkToWiki": "Vyberte běhové prostředí, pro které je určeno vaše řešení. Více informací o běhových prostředích naleznete na naší <a>wiki stránce</a>.",
   "app.submitSolution.noEnvironments": "Nahrané soubory neodpovídají žádnému povolenému běhovému prostředí.",
   "app.submitSolution.noteLabel": "Poznámka pro vás a vašeho vyučujícího:",
   "app.submitSolution.runtimeEnvironment": "Vyberte běhové prostředí (programovací jazyk):",
+  "app.submitSolution.submitFailed": "Odevzdané řešení bylo aktivně odmítnuto. Toto je obvykle způsobeno nahráním souborů se jménem obsahujícím nestandardní znaky nebo špatnou koncovkou. Pokud nemůžete odevzdat řešení bez zjevného důvodu, kontaktujte prosím svého vyučujícího.",
   "app.submitSolution.title": "Odevzdat nové řešení",
   "app.submitSolution.uploadFilesFirst": "Nejprve nahrajte vaše soubory...",
   "app.submitSolution.validating": "Předběžná kontrola nahraných souborů...",
-  "app.sidebar.menu.admin.title": "Administrátor",
   "app.supplementaryFiles.cannotDeleteExplain": "Soubor nelze smazat protože je použit v konfiguraci.",
   "app.supplementaryFiles.deleteConfirm": "Opravdu si přejete tento soubor smazat? Tato operace nemůže být vrácena.",
   "app.supplementaryFilesTable.description": "Soubory úlohy jsou soubory, které mohou být použity v nastavení úlohy (jako vstupní soubory, vzorové výstupní soubory, přídavné soubory pro kompilaci, vlastní sudí, ...).",
@@ -2050,10 +2054,10 @@
   "app.systemMessages.failed": "Nepodařilo se načíst seznam systémových zpráv",
   "app.systemMessages.listTitle": "Systémové zprávy",
   "app.systemMessages.loading": "Načítám seznam systémových zpráv...",
+  "app.systemMessages.markUnread": "Zobrazit všechny zprávy (označit jako nepřečtené)",
   "app.systemMessages.newSystemMessage": "Nová systémová zpráva",
   "app.systemMessages.title": "Systémové zprávy",
   "app.systemMessages.titleLong": "Máte {totalMessagesCount, number} {totalMessagesCount, plural, one {aktivní zprávu} =2 {aktivní zprávy} =3 {aktivní zprávy} =4 {aktivní zprávy} other {aktivních zpráv}} (z toho {unreadCount, number} {unreadCount, plural, one {nepřečtenou} =2 {nepřečtené} =3 {nepřečtené} =4 {nepřečtené} other {nepřečtených}})",
-  "app.systemMessages.markUnread": "Zobrazit všechny zprávy (označit jako nepřečtené)",
   "app.systemMessagesList.noMessages": "Nyní zde nejsou žádné systémové zprávy.",
   "app.systemMessagesList.showAll": "Zobrazit všechny zprávy (včetně expirovaných)",
   "app.systemMessagesList.text": "Text",
@@ -2110,8 +2114,8 @@
   "app.users.takeOver": "Přihlásit jako",
   "app.users.title": "Seznam všech uživatelů",
   "app.users.userCreatedAt": "Uživatel vytvořen",
-  "app.usersName.notVerified.title": "Tento účet nemá ověřenou e-mailovou adresu.",
   "app.usersName.notVerified.description": "Tento uživatel si neověřil svou e-mailovou adresu přes aktivační odkaz, který mu byl na tuto adresu zaslán.",
+  "app.usersName.notVerified.title": "Tento účet nemá ověřenou e-mailovou adresu.",
   "diff": "Sudí binárních dat",
   "generic.accept": "Akceptovat",
   "generic.accessDenied": "Nemáte oprávnění vidět tuto stránku. Pokud došlo k přesměrování na tuto stránku po kliknutí na zdánlivě legitimní odkaz nebo tlačítko, prosíme nahlaste chybu.",