- );
- }
+ if (isLoading) {
+ return ;
+ }
- return children;
+ return children;
};
function App() {
- return (
-
-
-
-
-
-
- }>
-
- } />
- } />
- } />
-
- {/* Student Routes */}
-
-
-
- }
- />
-
-
-
- }
- />
-
-
-
- }
- />
-
-
-
- }
- />
-
-
-
- }
- />
-
-
-
- }
- />
-
-
-
- }
- />
-
-
-
- }
- />
-
- {/* Instructor Routes */}
-
-
-
- }
- />
-
-
-
- }
- />
-
-
-
- }
- />
-
-
-
- }
- />
-
-
-
- }
- />
-
-
-
- }
- />
-
-
-
- }
- />
-
-
-
- }
- />
-
- {/* Admin routes — nested so /admin/* all go to AdminPage's internal router */}
-
-
-
- }
- />
-
- {/* Catch all */}
- } />
-
-
-
-
-
- )
+ return (
+
+
+
+ }>
+
+ } />
+ } />
+
+
+
+ }
+ />
+
+ {/* Student Routes */}
+
+
+
+ }
+ />
+
+
+
+ }
+ />
+
+
+
+ }
+ />
+
+
+
+ }
+ />
+
+
+
+ }
+ />
+
+
+
+ }
+ />
+
+
+
+ }
+ />
+
+
+
+ }
+ />
+
+ {/* Instructor Routes */}
+
+
+
+ }
+ />
+
+
+
+ }
+ />
+
+
+
+ }
+ />
+
+
+
+ }
+ />
+
+
+
+ }
+ />
+
+
+
+ }
+ />
+
+
+
+ }
+ />
+
+
+
+ }
+ />
+
+ {/* Admin routes — nested so /admin/* all go to AdminPage's internal router */}
+
+
+
+ }
+ />
+
+ {/* Catch all */}
+ } />
+
+
+
+
+
+ );
}
export default App;
diff --git a/frontend/src/assets/quiz_data.json b/frontend/src/assets/quiz_data.json
index 14764ec..c894f6d 100644
--- a/frontend/src/assets/quiz_data.json
+++ b/frontend/src/assets/quiz_data.json
@@ -1,35 +1,35 @@
[
- {
- "question": "What is 'Authentication' (AuthN)?",
- "options": [
- "The process that checks what you are allowed to do.",
- "The process where the user *proves* their identity to the system (e.g., via password or fingerprint 🧬).",
- "The name of the permissions model in Linux.",
- "The ability to access files."
- ],
- "correct_answer_index": 1,
- "explanation": "Authentication (AuthN) always comes first and answers the question 'Who are you?' 👤. For example, the Windows login screen."
- },
- {
- "question": "What is 'Authorization' (AuthZ)?",
- "options": [
- "The process of entering a password.",
- "The process that determines *what* an authenticated user is allowed to do (e.g., 'Can they delete this file?').",
- "The name of the system update process.",
- "Proving the user's identity."
- ],
- "correct_answer_index": 1,
- "explanation": "Authorization (AuthZ) always comes after authentication and answers the question 'What are you allowed to do?' 🔑. For example, checking if you are an 'Admin' or a 'User'."
- },
- {
- "question": "What is the 'Principle of Least Privilege'?",
- "options": [
- "Giving all users Admin privileges for convenience.",
- "Giving every process and user the *minimum* necessary permissions needed to work, and no more.",
- "Ensuring every user has at least one permission.",
- "Deleting permissions from users who haven't logged in recently."
- ],
- "correct_answer_index": 1,
- "explanation": "This is a basic security principle. 🛡️ If a program only needs to read a file, do not give it write permission. That way, if the program is 'hijacked' by a virus, the damage it can cause is limited."
- }
-]
\ No newline at end of file
+ {
+ "question": "What is 'Authentication' (AuthN)?",
+ "options": [
+ "The process that checks what you are allowed to do.",
+ "The process where the user *proves* their identity to the system (e.g., via password or fingerprint 🧬).",
+ "The name of the permissions model in Linux.",
+ "The ability to access files."
+ ],
+ "correct_answer_index": 1,
+ "explanation": "Authentication (AuthN) always comes first and answers the question 'Who are you?' 👤. For example, the Windows login screen."
+ },
+ {
+ "question": "What is 'Authorization' (AuthZ)?",
+ "options": [
+ "The process of entering a password.",
+ "The process that determines *what* an authenticated user is allowed to do (e.g., 'Can they delete this file?').",
+ "The name of the system update process.",
+ "Proving the user's identity."
+ ],
+ "correct_answer_index": 1,
+ "explanation": "Authorization (AuthZ) always comes after authentication and answers the question 'What are you allowed to do?' 🔑. For example, checking if you are an 'Admin' or a 'User'."
+ },
+ {
+ "question": "What is the 'Principle of Least Privilege'?",
+ "options": [
+ "Giving all users Admin privileges for convenience.",
+ "Giving every process and user the *minimum* necessary permissions needed to work, and no more.",
+ "Ensuring every user has at least one permission.",
+ "Deleting permissions from users who haven't logged in recently."
+ ],
+ "correct_answer_index": 1,
+ "explanation": "This is a basic security principle. 🛡️ If a program only needs to read a file, do not give it write permission. That way, if the program is 'hijacked' by a virus, the damage it can cause is limited."
+ }
+]
diff --git a/frontend/src/components/ErrorBoundary.jsx b/frontend/src/components/ErrorBoundary.jsx
index 53b66a5..d230b85 100644
--- a/frontend/src/components/ErrorBoundary.jsx
+++ b/frontend/src/components/ErrorBoundary.jsx
@@ -1,4 +1,5 @@
import React from 'react';
+import logger from '../utils/logger';
class ErrorBoundary extends React.Component {
constructor(props) {
@@ -12,7 +13,7 @@ class ErrorBoundary extends React.Component {
componentDidCatch(error, errorInfo) {
this.setState({ error, errorInfo });
- console.error("Uncaught error:", error, errorInfo);
+ logger.error('Uncaught error:', error, errorInfo);
}
render() {
diff --git a/frontend/src/components/analytics/AtRiskStudentsWidget.jsx b/frontend/src/components/analytics/AtRiskStudentsWidget.jsx
index fa64135..0ddd8dd 100644
--- a/frontend/src/components/analytics/AtRiskStudentsWidget.jsx
+++ b/frontend/src/components/analytics/AtRiskStudentsWidget.jsx
@@ -9,7 +9,7 @@ const AtRiskStudentsWidget = ({ courses = [] }) => {
const [selectedCourseId, setSelectedCourseId] = useState(null);
const navigate = useNavigate();
- const readyCourses = courses.filter(c => c.generationStatus === 'ready').slice(0, 5);
+ const readyCourses = courses.filter((c) => c.generationStatus === 'ready').slice(0, 5);
useEffect(() => {
if (readyCourses.length > 0 && !selectedCourseId) {
@@ -36,7 +36,10 @@ const AtRiskStudentsWidget = ({ courses = [] }) => {
if (readyCourses.length === 0) return null;
return (
-