Merge remote-tracking branch 'origin/feature/teacher' into feature/teacher

Author: DPKaragyozova21

My_path_team9/teacher/routes.py

@@ -12,6 +12,13 @@ def dashboard():
     if current_user.role != 'teacher':
         return "Access denied", 403
 
+    """
+        Teacher dashboard view.
+
+        Displays all video submissions to the teacher.
+        Only accessible to users with the 'teacher' role.
+        """
+
     submissions = VideoSubmission.query.all()
     return render_template('teacher_dashboard.html', submissions=submissions)
 
@@ -21,6 +28,14 @@ def dashboard():
 def approve_video(submission_id):
     if current_user.role != 'teacher':
         return "Access denied", 403
+    """
+       Approves a specific video submission.
+
+       Sets the 'confirmed' flag to True and commits the change to the database.
+       Only accessible to users with the 'teacher' role.
+
+       :param submission_id: ID of the video submission to approve
+       """
 
     submission = VideoSubmission.query.get_or_404(submission_id)
     submission.confirmed = True
@@ -35,6 +50,15 @@ def reject_video(submission_id):
     if current_user.role != 'teacher':
         return "Access denied", 403
 
+    """
+        Rejects and deletes a specific video submission.
+
+        Deletes the submission from the database.
+        Only accessible to users with the 'teacher' role.
+
+        :param submission_id: ID of the video submission to reject
+        """
+
     submission = VideoSubmission.query.get_or_404(submission_id)
     db.session.delete(submission)
     db.session.commit()
@@ -47,6 +71,14 @@ def settings_teacher():
     if current_user.role != 'teacher':
         return "Access denied", 403
 
+    """
+        Allows a teacher to update their profile and optionally change their password.
+
+        Performs validation for username and email uniqueness,
+        and validates the current password before allowing a password change.
+        Only accessible to users with the 'teacher' role.
+    """
+
     form = SettingsForm(obj=current_user)
 
     if form.validate_on_submit():
@@ -85,6 +117,12 @@ def settings_teacher():
 @teacher_bp.route('/students', methods=['GET', 'POST'])
 @login_required
 def students_list():
+    """
+        Displays a list of all student users and allows the teacher to create new student accounts.
+
+        On POST, processes the creation of a new student account after validating required fields.
+        Only accessible to users with the 'teacher' role.
+        """
     if current_user.role != 'teacher':
         flash('You do not have access to this page.', 'danger')
         return redirect(url_for('main.home'))
@@ -118,6 +156,13 @@ def students_list():
 @teacher_bp.route('/students/delete/<int:user_id>', methods=['POST'])
 @login_required
 def delete_student(user_id):
+    """
+       Deletes a student account based on the provided user ID.
+
+       Ensures only students can be deleted and that the current user has the 'teacher' role.
+
+       :param user_id: ID of the student user to delete
+       """
     if current_user.role != 'teacher':
         flash('You do not have permission to perform this action.', 'danger')
         return redirect(url_for('main.home'))