change to a single checkbox filter for all modules

This commit is contained in:
Joao Correia
2025-02-26 19:23:21 +00:00
parent ba8cc342b1
commit 4e3cfec9e8

View File

@@ -101,14 +101,6 @@ interface Props {
const SIZE = 9; const SIZE = 9;
type ExamApprovalFilterMap = {
writing: boolean;
reading: boolean;
listening: boolean;
speaking: boolean;
level: boolean;
};
export default function AssignmentsPage({ export default function AssignmentsPage({
user, user,
users, users,
@@ -151,20 +143,8 @@ export default function AssignmentsPage({
const [useRandomExams, setUseRandomExams] = useState(true); const [useRandomExams, setUseRandomExams] = useState(true);
const [examIDs, setExamIDs] = useState<{ id: string; module: Module }[]>([]); const [examIDs, setExamIDs] = useState<{ id: string; module: Module }[]>([]);
const [showExamsThatRequiredApproval, setShowExamsThatRequiredApproval] = useState<ExamApprovalFilterMap>({ const [showApprovedExams, setShowApprovedExams] = useState<boolean>(true);
writing: true, const [showNonApprovedExams, setShowNonApprovedExams] = useState<boolean>(true);
reading: true,
listening: true,
speaking: true,
level: true,
});
const [showExamsThatDidntRequireApproval, setShowExamsThatDidntRequireApproval] = useState<ExamApprovalFilterMap>({
writing: true,
reading: true,
listening: true,
speaking: true,
level: true,
});
const { exams } = useExams(); const { exams } = useExams();
const router = useRouter(); const router = useRouter();
@@ -524,36 +504,29 @@ export default function AssignmentsPage({
Random Exams Random Exams
</Checkbox> </Checkbox>
{!useRandomExams && ( {!useRandomExams && (
<>
<Checkbox
isChecked={showApprovedExams}
onChange={() => {
setShowApprovedExams((prev) => !prev)
}}
>
Show approved exams
</Checkbox>
<Checkbox
isChecked={showNonApprovedExams}
onChange={() => {
setShowNonApprovedExams((prev) => !prev)
}}
>
Show non-approved exams
</Checkbox>
<div className="grid md:grid-cols-2 w-full gap-4"> <div className="grid md:grid-cols-2 w-full gap-4">
{selectedModules.map((module) => ( {selectedModules.map((module) => (
<div key={module} className="flex flex-col gap-3 w-full"> <div key={module} className="flex flex-col gap-3 w-full">
<div className="flex flex-row gap-6">
<label className="font-normal text-base text-mti-gray-dim"> <label className="font-normal text-base text-mti-gray-dim">
{capitalize(module)} Exam {capitalize(module)} Exam
</label> </label>
<Checkbox
isChecked={showExamsThatRequiredApproval[module]}
onChange={() => {
setShowExamsThatRequiredApproval(prev => ({
...prev,
[module]: !prev[module],
}))
}}
>
Show exams that required approval
</Checkbox>
<Checkbox
isChecked={showExamsThatDidntRequireApproval[module]}
onChange={() => {
setShowExamsThatDidntRequireApproval(prev => ({
...prev,
[module]: !prev[module],
}))
}}
>
Show exams that did not require approval
</Checkbox>
</div>
<Select <Select
isClearable isClearable
value={{ value={{
@@ -577,9 +550,10 @@ export default function AssignmentsPage({
.filter((x) => .filter((x) =>
!x.isDiagnostic && !x.isDiagnostic &&
x.module === module && x.module === module &&
x.access !== "confidential" &&
( (
(x.requiresApproval && showExamsThatRequiredApproval[module]) || (x.requiresApproval && showApprovedExams) ||
(!x.requiresApproval && showExamsThatDidntRequireApproval[module]) (!x.requiresApproval && showNonApprovedExams)
) )
) )
.map((x) => ({ value: x.id, label: x.id }))} .map((x) => ({ value: x.id, label: x.id }))}
@@ -587,6 +561,7 @@ export default function AssignmentsPage({
</div> </div>
))} ))}
</div> </div>
</>
)} )}
</div> </div>
)} )}