From 5e91e820433bbef4e9f9c20d41eae884f8c42b99 Mon Sep 17 00:00:00 2001
From: b0ink <40929320+b0ink@users.noreply.github.com>
Date: Wed, 7 Jan 2026 13:40:54 +1100
Subject: [PATCH 01/38] feat: init tutor notes and mentorship

---
 src/app/api/models/tutor-note.ts              |  50 ++++++
 src/app/api/models/unit-role.ts               |  21 ++-
 src/app/api/services/tutor-note.service.ts    | 123 +++++++++++++
 src/app/api/services/unit-role.service.ts     |   3 +-
 src/app/common/footer/footer.component.html   |  14 ++
 src/app/common/footer/footer.component.ts     |  50 ++++++
 src/app/doubtfire-angular.module.ts           |   6 +
 src/app/doubtfire-angularjs.module.ts         |   6 +
 .../tutor-notes-view.component.html           |   8 +
 .../tutor-notes-view.component.scss           |   0
 .../tutor-notes-view.component.ts             |  21 +++
 .../task-dashboard.component.html             |   4 +
 .../states/dashboard/selected-task.service.ts |   5 +
 .../tutor-notes/tutor-notes.component.html    | 127 +++++++++++++
 .../tutor-notes/tutor-notes.component.scss    |  27 +++
 .../tutor-notes/tutor-notes.component.ts      | 167 ++++++++++++++++++
 .../unit-staff-editor.component.html          |  19 ++
 .../unit-staff-editor.component.ts            |  24 ++-
 18 files changed, 667 insertions(+), 8 deletions(-)
 create mode 100644 src/app/api/models/tutor-note.ts
 create mode 100644 src/app/api/services/tutor-note.service.ts
 create mode 100644 src/app/projects/states/dashboard/directives/task-dashboard/directives/tutor-notes-view/tutor-notes-view.component.html
 create mode 100644 src/app/projects/states/dashboard/directives/task-dashboard/directives/tutor-notes-view/tutor-notes-view.component.scss
 create mode 100644 src/app/projects/states/dashboard/directives/task-dashboard/directives/tutor-notes-view/tutor-notes-view.component.ts
 create mode 100644 src/app/projects/states/tutor-notes/tutor-notes.component.html
 create mode 100644 src/app/projects/states/tutor-notes/tutor-notes.component.scss
 create mode 100644 src/app/projects/states/tutor-notes/tutor-notes.component.ts

diff --git a/src/app/api/models/tutor-note.ts b/src/app/api/models/tutor-note.ts
new file mode 100644
index 0000000000..d585c1f5f5
--- /dev/null
+++ b/src/app/api/models/tutor-note.ts
@@ -0,0 +1,50 @@
+import {Entity} from 'ngx-entity-service';
+import {Project, Unit, User, UserService, Task, UnitRole} from './doubtfire-model';
+import {AppInjector} from 'src/app/app-injector';
+import {AlertService} from 'src/app/common/services/alert.service';
+import {StaffNoteService} from '../services/staff-note.service';
+import {TutorNoteService} from '../services/tutor-note.service';
+
+export class TutorNote extends Entity {
+  id: number;
+
+  // project: Project;
+  unitRole: UnitRole;
+  task: Task;
+  user: User;
+  note: string;
+  replyTo?: TutorNote;
+  replyToId: number;
+  createdAt: Date;
+  updatedAt: Date;
+
+  constructor(data?: UnitRole) {
+    super();
+    if (data) {
+      this.unitRole = data;
+    } else {
+      console.error('Failed to get project');
+    }
+  }
+
+  public get authorIsMe(): boolean {
+    const userService: UserService = AppInjector.get(UserService);
+    return this.user.id === userService.currentUser.id;
+  }
+
+  public delete() {
+    const tutorNoteService: TutorNoteService = AppInjector.get(TutorNoteService);
+    tutorNoteService
+      .delete({unitRoleId: this.unitRole.id, id: this.id}, {cache: this.unitRole.tutorNotesCache})
+      .subscribe({
+        next: (response: object) => {
+          AppInjector.get(AlertService).error('Successfully deleted tutor note', 4000);
+          // this.project.staffNoteCount--;
+          // staffNoteService.updateStaffNoteReplies(this.project.staffNoteCache.currentValues);
+        },
+        error: (error) => {
+          AppInjector.get(AlertService).error(error?.message || error || 'Unknown error', 2000);
+        },
+      });
+  }
+}
diff --git a/src/app/api/models/unit-role.ts b/src/app/api/models/unit-role.ts
index 63bf9852de..982f4fd600 100644
--- a/src/app/api/models/unit-role.ts
+++ b/src/app/api/models/unit-role.ts
@@ -1,18 +1,25 @@
-import { Entity, EntityMapping } from 'ngx-entity-service';
-import { User, Unit } from './doubtfire-model';
+import {Entity, EntityCache, EntityMapping} from 'ngx-entity-service';
+import {User, Unit} from './doubtfire-model';
+import {TutorNote} from './tutor-note';
 
 /**
  * A unit role represents a academic teaching role within a unit. Linking a user
  * to their role within the unit.
  */
 export class UnitRole extends Entity {
-
   id: number;
   role: string;
   user: User;
   unit: Unit;
   observerOnly: boolean;
+  // mentor: UnitRole;
+  mentorId: number;
+
+  public readonly tutorNotesCache: EntityCache<TutorNote> = new EntityCache<TutorNote>();
 
+  public get mentor(): UnitRole {
+    return this.unit?.staff.find((ur) => ur.id === this.mentorId);
+  }
   /**
    * The id for updated roles - but we need to move away from this to the role string...
    * @deprecated
@@ -29,10 +36,12 @@ export class UnitRole extends Entity {
     return this.user.matches(text) || this.unit.matches(text);
   }
 
-  public override toJson<T extends Entity>(mappingData: EntityMapping<T>, ignoreKeys?: string[]): object {
+  public override toJson<T extends Entity>(
+    mappingData: EntityMapping<T>,
+    ignoreKeys?: string[],
+  ): object {
     return {
       unit_role: super.toJson(mappingData, ignoreKeys),
-    }
+    };
   }
-
 }
diff --git a/src/app/api/services/tutor-note.service.ts b/src/app/api/services/tutor-note.service.ts
new file mode 100644
index 0000000000..39e08b98e3
--- /dev/null
+++ b/src/app/api/services/tutor-note.service.ts
@@ -0,0 +1,123 @@
+import {HttpClient} from '@angular/common/http';
+import {Injectable} from '@angular/core';
+import {CachedEntityService, RequestOptions} from 'ngx-entity-service';
+import {Observable, tap} from 'rxjs';
+import {ProjectService, UnitRole, UserService} from 'src/app/api/models/doubtfire-model';
+import API_URL from 'src/app/config/constants/apiUrl';
+import {TutorNote} from '../models/tutor-note';
+
+@Injectable()
+export class TutorNoteService extends CachedEntityService<TutorNote> {
+  // public readonly staffNoteAdded$: EventEmitter<StaffNote> = new EventEmitter();
+
+  protected readonly endpointFormat = 'unit_roles/:unitRoleId:/tutor_notes/:id:';
+
+  constructor(
+    httpClient: HttpClient,
+    private userService: UserService,
+    private projectService: ProjectService,
+  ) {
+    super(httpClient, API_URL);
+
+    this.mapping.addKeys('id', 'note', 'createdAt', 'updatedAt', 'replyToId', {
+      keys: ['user', 'user_id'],
+      toEntityFn: (data: object, key: string, tutorNote: TutorNote) => {
+        const userRole = tutorNote.unitRole.unit.staff.find((s) => s.user.id === data['user_id']);
+        // const user = this.userService.cache.getOrCreate(data[key]?.id, userService, data[key]);
+        // If the user is not a staff in the unit it will be null
+        return userRole?.user;
+      },
+    });
+
+    this.mapping.addJsonKey('note', 'createdAt', 'updatedAt');
+  }
+
+  public createInstanceFrom(json: object, other?: UnitRole): TutorNote {
+    return new TutorNote(other);
+  }
+
+  public addNote(unitRole: UnitRole, text: string, originalNote: TutorNote): Observable<TutorNote> {
+    const pathId = {
+      unitRoleId: unitRole.id,
+    };
+
+    const body: FormData = new FormData();
+    if (originalNote) {
+      body.append('reply_to_id', originalNote?.id.toString());
+    }
+
+    body.append('note', text);
+
+    const opts: RequestOptions<TutorNote> = {endpointFormat: this.endpointFormat};
+    opts.cache = unitRole.tutorNotesCache;
+    opts.body = body;
+    opts.constructorParams = unitRole;
+
+    return this.create(pathId, opts).pipe(
+      tap((note: TutorNote) => {
+        // this.staffNoteAdded$.emit(note);
+      }),
+    );
+  }
+
+  public updateTutorNoteReplies(tutorNotes: readonly TutorNote[]) {
+    for (const note of tutorNotes) {
+      if (note.replyToId) {
+        const repliedTo = tutorNotes.find((n) => n.id === note.replyToId);
+        if (repliedTo) {
+          note.replyTo = repliedTo;
+        } else {
+          // Remove deleted replies
+          note.replyTo = null;
+        }
+      }
+    }
+  }
+
+  public updateNote(unitRole: UnitRole, note: TutorNote, text: string): Observable<TutorNote> {
+    const pathId = {
+      unitRoleId: unitRole.id,
+      id: note.id,
+    };
+
+    const body: FormData = new FormData();
+    body.append('note', text);
+
+    const opts: RequestOptions<TutorNote> = {endpointFormat: this.endpointFormat};
+    opts.cache = unitRole.tutorNotesCache;
+    opts.body = body;
+    opts.constructorParams = unitRole;
+
+    return this.put(pathId, opts).pipe(
+      tap((_note: TutorNote) => {
+        note.note = text;
+      }),
+    );
+  }
+
+  public loadTutorNotes(unitRole: UnitRole, useFetch: boolean = false): Observable<TutorNote[]> {
+    const options: RequestOptions<TutorNote> = {
+      endpointFormat: this.endpointFormat,
+      cache: unitRole.tutorNotesCache,
+      sourceCache: unitRole.tutorNotesCache,
+      cacheBehaviourOnGet: 'cacheQuery',
+      constructorParams: unitRole,
+    };
+
+    if (useFetch) {
+      return super.fetchAll(
+        {
+          unitRoleId: unitRole.id,
+        },
+        options,
+      );
+    } else {
+      return super.query(
+        {
+          unitRoleId: unitRole.id,
+        },
+        options,
+      );
+    }
+  }
+}
diff --git a/src/app/api/services/unit-role.service.ts b/src/app/api/services/unit-role.service.ts
index 71cb194422..00102dd37a 100644
--- a/src/app/api/services/unit-role.service.ts
+++ b/src/app/api/services/unit-role.service.ts
@@ -63,9 +63,10 @@ export class UnitRoleService extends CachedEntityService<UnitRole> {
         },
       },
       'observerOnly',
+      'mentorId',
     );
 
-    this.mapping.addJsonKey('roleId', 'userId', 'unitId', 'role', 'observerOnly');
+    this.mapping.addJsonKey('roleId', 'userId', 'unitId', 'role', 'observerOnly', 'mentorId');
   }
 
   public createInstanceFrom(json: any, other?: any): UnitRole {
diff --git a/src/app/common/footer/footer.component.html b/src/app/common/footer/footer.component.html
index a31bf9fac8..033beedc92 100644
--- a/src/app/common/footer/footer.component.html
+++ b/src/app/common/footer/footer.component.html
@@ -156,6 +156,20 @@
           </button>
         }
 
+        @if (canAccessTutorNotes(selectedTask)) {
+          <div class="relative inline-block">
+            <button
+              mat-icon-button
+              matTooltipPosition="above"
+              class="button small-button"
+              [disabled]="!markingTutor(selectedTask)"
+              (click)="viewTutorNotes()"
+            >
+              <mat-icon class="small-icon">local_library</mat-icon>
+            </button>
+          </div>
+        }
+
         <div class="relative inline-block">
           @if (selectedTask.project.staffNoteCount > 0) {
             <div [matBadge]="selectedTask.project.staffNoteCount"></div>
diff --git a/src/app/common/footer/footer.component.ts b/src/app/common/footer/footer.component.ts
index fa44839024..e058cedd68 100644
--- a/src/app/common/footer/footer.component.ts
+++ b/src/app/common/footer/footer.component.ts
@@ -5,6 +5,8 @@ import {SelectedTaskService} from 'src/app/projects/states/dashboard/selected-ta
 import {TaskService} from 'src/app/api/services/task.service';
 import {FileDownloaderService} from '../file-downloader/file-downloader.service';
 import {TaskAssessmentModalService} from '../modals/task-assessment-modal/task-assessment-modal.service';
+import {UnitRole} from 'src/app/api/models/unit-role';
+import {UserService} from 'src/app/api/services/user.service';
 
 @Component({
   selector: 'f-footer',
@@ -17,6 +19,7 @@ export class FooterComponent implements OnInit {
     public taskService: TaskService,
     private fileDownloader: FileDownloaderService,
     private taskAssessmentModal: TaskAssessmentModalService,
+    private userService: UserService,
   ) {}
 
   selectedTask$: Observable<Task>;
@@ -48,12 +51,59 @@ export class FooterComponent implements OnInit {
       (this.warningText?.nativeElement.getBoundingClientRect().width + totalPaddingOffset) / 2;
   }
 
+  public markingTutor(task: Task): UnitRole {
+    const enrolments = task.project.tutorialEnrolmentsCache.currentValues.filter(
+      (t) => t.tutorialStream.name === task.definition.tutorialStream.name,
+    );
+    // TODO: is checking for just the one tutorial enrolment correct? should be..
+    if (enrolments.length === 1) {
+      const user = enrolments[0].tutor;
+      return task.unit.staff.find((ur) => ur.user.id === user.id);
+    }
+    return null;
+  }
+
+  public canAccessTutorNotes(task: Task): boolean {
+    // TODO: if the access is truthy because current user == marking tutor
+    // TODO: we should only reveal the icon if a note has been left by their mentor
+
+    const tutor = this.markingTutor(task);
+    if (!tutor) {
+      console.log('tutor invalid!');
+      return false;
+    }
+
+    const currentUser = this.userService.currentUser;
+    const currentUserRole = task.unit.staff.find((ur) => ur.user.id === currentUser.id);
+
+    if (!currentUserRole) {
+      return false;
+    }
+
+    // Ensure the unit is mapped correctly to access the mentor
+    tutor.unit = task.unit;
+
+    const canAccess =
+      currentUserRole.role === 'Convenor' ||
+      currentUserRole.role === 'Admin' ||
+      (tutor.mentor && tutor.mentor.id === currentUserRole.id) ||
+      tutor.id === currentUserRole.id;
+
+    return canAccess;
+  }
+
+  public viewTutorNotes() {
+    console.log(`viewing tutor notes for `, this.markingTutor(this.selectedTask));
+    this.selectedTaskService.showTutorNotes();
+  }
+
   ngOnInit(): void {
     // watch for changes to the selected task
     this.selectedTask$ = this.selectedTaskService.selectedTask$;
 
     this.selectedTask$.subscribe((task) => {
       this.selectedTask = task;
+
       // We need to timeout to give the DOM a chance to place the elements
       setTimeout(() => {
         this.findSimilaritiesButton();
diff --git a/src/app/doubtfire-angular.module.ts b/src/app/doubtfire-angular.module.ts
index 33c8db369b..11220a1da3 100644
--- a/src/app/doubtfire-angular.module.ts
+++ b/src/app/doubtfire-angular.module.ts
@@ -312,6 +312,9 @@ import {TaskPlannerCardComponent} from './projects/states/dashboard/directives/p
 import {OverseerStepService} from './api/services/overseer-step.service';
 import {TaskOverseerReportComponent} from './projects/states/dashboard/directives/task-dashboard/directives/task-overseer-report/task-overseer-report.component';
 import {OverseerStepResultService} from './api/services/overseer-step-result.service';
+import {TutorNotesComponent} from './projects/states/tutor-notes/tutor-notes.component';
+import {TutorNotesViewComponent} from './projects/states/dashboard/directives/task-dashboard/directives/tutor-notes-view/tutor-notes-view.component';
+import {TutorNoteService} from './api/services/tutor-note.service';
 
 // See https://stackoverflow.com/questions/55721254/how-to-change-mat-datepicker-date-format-to-dd-mm-yyyy-in-simplest-way/58189036#58189036
 const MY_DATE_FORMAT = {
@@ -500,6 +503,8 @@ const GANTT_CHART_CONFIG = {
     TaskPlannerCardComponent,
     TaskPlannerPrerequisitesModalComponent,
     TaskOverseerReportComponent,
+    TutorNotesComponent,
+    TutorNotesViewComponent,
   ],
   providers: [
     // Services we provide
@@ -594,6 +599,7 @@ const GANTT_CHART_CONFIG = {
     TaskPlannerPrerequisitesModalService,
     OverseerStepService,
     OverseerStepResultService,
+    TutorNoteService,
   ],
   imports: [
     FlexLayoutModule,
diff --git a/src/app/doubtfire-angularjs.module.ts b/src/app/doubtfire-angularjs.module.ts
index 40e8930490..69f7180a11 100644
--- a/src/app/doubtfire-angularjs.module.ts
+++ b/src/app/doubtfire-angularjs.module.ts
@@ -237,6 +237,7 @@ import {ProjectPlanComponent} from './projects/states/plan/project-plan.componen
 import {TaskPlannerComponent} from './projects/states/plan/task-planner/task-planner.component';
 import {TaskPlannerCardComponent} from './projects/states/dashboard/directives/progress-dashboard/task-planner-card/task-planner-card.component';
 import {TaskOverseerReportComponent} from './projects/states/dashboard/directives/task-dashboard/directives/task-overseer-report/task-overseer-report.component';
+import {TutorNotesComponent} from './projects/states/tutor-notes/tutor-notes.component';
 
 export const DoubtfireAngularJSModule = angular
   .module('doubtfire', [
@@ -599,3 +600,8 @@ DoubtfireAngularJSModule.directive(
   'fTaskOverseerReport',
   downgradeComponent({component: TaskOverseerReportComponent}),
 );
+
+DoubtfireAngularJSModule.directive(
+  'fTutorNotes',
+  downgradeComponent({component: TutorNotesComponent}),
+);
diff --git a/src/app/projects/states/dashboard/directives/task-dashboard/directives/tutor-notes-view/tutor-notes-view.component.html b/src/app/projects/states/dashboard/directives/task-dashboard/directives/tutor-notes-view/tutor-notes-view.component.html
new file mode 100644
index 0000000000..2c8698b7b9
--- /dev/null
+++ b/src/app/projects/states/dashboard/directives/task-dashboard/directives/tutor-notes-view/tutor-notes-view.component.html
@@ -0,0 +1,8 @@
+<div class="p-4 h-[calc(100%-50px)]">
+  <div class="flex items-center">
+    <mat-icon class="mr-2">comment</mat-icon>
+    <h1 class="mat-headline m-0">Tutor Notes for {{ unitRole?.user?.name }}</h1>
+  </div>
+
+  <f-tutor-notes [task]="task"></f-tutor-notes>
+</div>
diff --git a/src/app/projects/states/dashboard/directives/task-dashboard/directives/tutor-notes-view/tutor-notes-view.component.scss b/src/app/projects/states/dashboard/directives/task-dashboard/directives/tutor-notes-view/tutor-notes-view.component.scss
new file mode 100644
index 0000000000..e69de29bb2
diff --git a/src/app/projects/states/dashboard/directives/task-dashboard/directives/tutor-notes-view/tutor-notes-view.component.ts b/src/app/projects/states/dashboard/directives/task-dashboard/directives/tutor-notes-view/tutor-notes-view.component.ts
new file mode 100644
index 0000000000..99c0ea1297
--- /dev/null
+++ b/src/app/projects/states/dashboard/directives/task-dashboard/directives/tutor-notes-view/tutor-notes-view.component.ts
@@ -0,0 +1,21 @@
+import {Component, Input} from '@angular/core';
+
+@Component({
+  selector: 'f-tutor-notes-view',
+  templateUrl: './tutor-notes-view.component.html',
+  styleUrls: ['./tutor-notes-view.component.scss'],
+})
+export class TutorNotesViewComponent {
+  @Input() task?;
+  // @Input() project;
+  public get unitRole() {
+    const enrolments = this.task.project.tutorialEnrolmentsCache.currentValues.filter(
+      (t) => t.tutorialStream.name === this.task.definition.tutorialStream.name,
+    );
+    // TODO: is checking for just the one tutorial enrolment correct? should be..
+    if (enrolments.length === 1) {
+      const user = enrolments[0].tutor;
+      return this.task.unit.staff.find((ur) => ur.user.id === user.id);
+    }
+  }
+}
diff --git a/src/app/projects/states/dashboard/directives/task-dashboard/task-dashboard.component.html b/src/app/projects/states/dashboard/directives/task-dashboard/task-dashboard.component.html
index fc861e5b34..7d9aa48482 100644
--- a/src/app/projects/states/dashboard/directives/task-dashboard/task-dashboard.component.html
+++ b/src/app/projects/states/dashboard/directives/task-dashboard/task-dashboard.component.html
@@ -24,6 +24,10 @@
       <f-staff-notes-view [project]="task.project"></f-staff-notes-view>
     </ng-template>
 
+    <ng-template [ngSwitchCase]="DashboardViews.tutor_notes">
+      <f-tutor-notes-view [task]="task"></f-tutor-notes-view>
+    </ng-template>
+
     <ng-template [ngSwitchCase]="DashboardViews.discussion_prompts">
       <f-discussion-prompts-view
         [project]="task.project"
diff --git a/src/app/projects/states/dashboard/selected-task.service.ts b/src/app/projects/states/dashboard/selected-task.service.ts
index 3385019a77..94730262da 100644
--- a/src/app/projects/states/dashboard/selected-task.service.ts
+++ b/src/app/projects/states/dashboard/selected-task.service.ts
@@ -9,6 +9,7 @@ export enum DashboardViews {
   task,
   similarity,
   staff_notes,
+  tutor_notes,
   discussion_prompts,
   overseer,
 }
@@ -68,6 +69,10 @@ export class SelectedTaskService {
     this.currentView$.next(DashboardViews.staff_notes);
   }
 
+  public showTutorNotes() {
+    this.currentView$.next(DashboardViews.tutor_notes);
+  }
+
   public showOverseerReports() {
     this.currentView$.next(DashboardViews.overseer);
   }
diff --git a/src/app/projects/states/tutor-notes/tutor-notes.component.html b/src/app/projects/states/tutor-notes/tutor-notes.component.html
new file mode 100644
index 0000000000..f3d87a9981
--- /dev/null
+++ b/src/app/projects/states/tutor-notes/tutor-notes.component.html
@@ -0,0 +1,127 @@
+<div class="flex flex-col h-full">
+  <!-- @if (!loadingStaffNotes && project?.staffNoteCount === 0) {
+    <div class="flex justify-center items-center w-full h-full text-gray-300">
+      No staff notes for {{ project.student.preferredName }} {{ project.student.lastName }}
+    </div>
+  } -->
+  <div #staffNotesContainer class="flex-1 overflow-auto p-4">
+    @if (!loadingStaffNotes) {
+      @for (note of unitRole?.tutorNotesCache?.currentValues; track note) {
+        @if (note.replyToId) {
+          <div
+            class="text-md px-2 py-2 select-none rounded-t-md border border-gray-300 flex items-center justify-between"
+          >
+            <div
+              class="flex items-center"
+              [ngClass]="note.replyTo && 'hover:text-blue-500 hover:underline cursor-pointer'"
+              (click)="scrollToNote(note.replyTo)"
+            >
+              <mat-icon class="mr-3">reply</mat-icon>
+              <div class="flex flex-col">
+                @if (note.replyTo) {
+                  <span class="font-medium">
+                    Replying to {{ note.replyTo.user.preferredName }}
+                    {{ note.replyTo.user.lastName }} ({{ note.replyTo.user.nickname }})
+                  </span>
+                  <span class="text-base text-gray-600">{{ note.replyTo.note }}</span>
+                } @else {
+                  <span class="text-base text-gray-400 italic">Replying to: Deleted note</span>
+                }
+              </div>
+            </div>
+          </div>
+        }
+        <mat-card
+          [id]="'note-' + note.id"
+          class="mb-4 p-5"
+          (mouseover)="hoveredNoteId = note.id"
+          (mouseout)="hoveredNoteId = null"
+        >
+          <div [hidden]="hoveredNoteId !== note.id" class="flex absolute right-5 bottom-5">
+            @if (note.authorIsMe) {
+              <mat-icon type="button" (click)="editNote(note)">edit</mat-icon>
+            }
+            <mat-icon type="button" (click)="replyToNote(note)">reply</mat-icon>
+            <mat-icon type="button" (click)="deleteNote(note)">delete</mat-icon>
+          </div>
+          <div class="flex justify-between">
+            <mat-card-title class="flex items-center">
+              <user-icon size="25" [user]="note.user" style="margin-right: 8px"></user-icon>
+              {{ note.user?.firstName }} {{ note.user?.lastName }}</mat-card-title
+            >
+            <div class="flex items-center">
+              <mat-card-subtitle
+                [matTooltip]="note.createdAt | localizedDateTime"
+                matTooltipPosition="above"
+                matTooltipShowDelay="1000"
+                >{{ note.createdAt | humanizedDate }}</mat-card-subtitle
+              >
+            </div>
+          </div>
+
+          <mat-card-content class="py-4">
+            @if (editingNote && editingNote.id === note.id) {
+              <mat-form-field appearance="outline" class="flex items-center w-full">
+                <mat-label>Update Note</mat-label>
+                <textarea
+                  (input)="autoResizeStaffNoteEditor()"
+                  #staffNoteEditor
+                  matInput
+                  [(ngModel)]="editingNoteText"
+                  class="resize-none w-full px-2 py-1 placeholder:px-2 placeholder:py-1 overflow-hidden"
+                ></textarea>
+                <div class="flex justify-end gap-2">
+                  <button mat-raised-button color="" (click)="cancelEditingNote()">Cancel</button>
+                  <button mat-raised-button color="primary" (click)="updateNote(note)">
+                    Update Note
+                  </button>
+                </div>
+              </mat-form-field>
+            } @else {
+              <div class="markdown-to-html" [innerHTML]="note.note | marked"></div>
+            }
+          </mat-card-content>
+        </mat-card>
+      }
+    } @else {
+      <!-- <div id="spinner">
+          <mat-spinner
+            class="absolute top-1/2 left-1/2 transform -translate-x-1/2 -translate-y-1/2 z-50"
+          ></mat-spinner>
+        </div> -->
+    }
+  </div>
+
+  <div class="h-[150px] border-t border-gray-300 p-4">
+    @if (replyingToNote) {
+      <div
+        class="text-md px-2 py-2 select-none rounded-t-md border border-b-0 border-gray-300 flex items-center justify-between"
+      >
+        <div class="flex items-center">
+          <mat-icon class="mr-3 mt-1">reply</mat-icon>
+          <div class="flex flex-col">
+            <span class="font-medium">
+              Replying to {{ replyingToNote.user.firstName }} {{ replyingToNote.user.lastName }} ({{
+                replyingToNote.user.nickname
+              }})
+            </span>
+            <span class="text-base text-gray-600">{{ replyingToNote.note }}</span>
+          </div>
+        </div>
+        <mat-icon class="cursor-pointer" (click)="cancelReplyingToNote()">close</mat-icon>
+      </div>
+    }
+
+    <mat-form-field appearance="outline" class="flex items-center w-full">
+      <mat-label>Staff Note</mat-label>
+      <textarea
+        matInput
+        [(ngModel)]="noteText"
+        class="resize-none w-full px-2 py-1 placeholder:px-2 placeholder:py-1"
+      ></textarea>
+      <div class="flex justify-end">
+        <button mat-raised-button color="primary" (click)="submitNote()">Submit</button>
+      </div>
+    </mat-form-field>
+  </div>
+</div>
diff --git a/src/app/projects/states/tutor-notes/tutor-notes.component.scss b/src/app/projects/states/tutor-notes/tutor-notes.component.scss
new file mode 100644
index 0000000000..a6d41ed2d2
--- /dev/null
+++ b/src/app/projects/states/tutor-notes/tutor-notes.component.scss
@@ -0,0 +1,27 @@
+.mat-icon {
+  color: #9696969d;
+  font-size: 20px;
+  width: 20px;
+  height: 20px;
+  cursor: pointer;
+  vertical-align: middle;
+  text-align: center;
+  margin-left: 0.3em;
+}
+
+.mat-icon:hover {
+  color: black;
+}
+
+@keyframes blueGlowFade {
+  0% {
+    background-color: rgba(66, 133, 244, 0.6);
+  }
+  100% {
+    background-color: transparent;
+  }
+}
+
+.flash-highlight {
+  animation: blueGlowFade 1s ease-out;
+}
diff --git a/src/app/projects/states/tutor-notes/tutor-notes.component.ts b/src/app/projects/states/tutor-notes/tutor-notes.component.ts
new file mode 100644
index 0000000000..9b2b2c4eff
--- /dev/null
+++ b/src/app/projects/states/tutor-notes/tutor-notes.component.ts
@@ -0,0 +1,167 @@
+import {Component, ElementRef, Input, OnInit, ViewChild} from '@angular/core';
+import {Task, UserService} from 'src/app/api/models/doubtfire-model';
+import {TutorNote} from 'src/app/api/models/tutor-note';
+import {TutorNoteService} from 'src/app/api/services/tutor-note.service';
+import {ConfirmationModalService} from 'src/app/common/modals/confirmation-modal/confirmation-modal.service';
+import {AlertService} from 'src/app/common/services/alert.service';
+
+@Component({
+  selector: 'f-tutor-notes',
+  templateUrl: './tutor-notes.component.html',
+  styleUrl: './tutor-notes.component.scss',
+})
+export class TutorNotesComponent implements OnInit {
+  @ViewChild('staffNotesContainer') staffNotesContainer!: ElementRef;
+  @ViewChild('staffNoteEditor', {static: false}) staffNoteEditor!: ElementRef<HTMLTextAreaElement>;
+
+  // @Input() unitRole: UnitRole;
+  @Input() task: Task;
+  // @Input() project: Project;
+
+  // TODO: allow unitRole Input(), but if its nil, we need to set it during OnInit
+  // TODO: otherwise, if task is null and unitRole exists (When current users access their own tutor notes, here we can load tutor notes with a unitRole and no task)
+
+  public get unitRole() {
+    const enrolments = this.task.project.tutorialEnrolmentsCache.currentValues.filter(
+      (t) => t.tutorialStream.name === this.task.definition.tutorialStream.name,
+    );
+    // TODO: is checking for just the one tutorial enrolment correct? should be..
+    if (enrolments.length === 1) {
+      const user = enrolments[0].tutor;
+      return this.task.unit.staff.find((ur) => ur.user.id === user.id);
+    }
+  }
+
+  loadingStaffNotes: boolean = true;
+
+  noteText: string = '';
+
+  editingNote?: TutorNote;
+  editingNoteText?: string = '';
+
+  replyingToNote?: TutorNote;
+
+  hoveredNoteId: number | null = null;
+
+  constructor(
+    private userService: UserService,
+    private tutorNoteService: TutorNoteService,
+    private alertService: AlertService,
+    private confirmationModalService: ConfirmationModalService,
+  ) {}
+  ngOnInit(): void {
+    this.loadingStaffNotes = true;
+    this.tutorNoteService.loadTutorNotes(this.unitRole).subscribe((notes) => {
+      this.loadingStaffNotes = false;
+      this.tutorNoteService.updateTutorNoteReplies(this.unitRole?.tutorNotesCache.currentValues);
+      this.scrollDown();
+    });
+  }
+
+  scrollToComment(commentID: number) {
+    document.querySelector(`#comment-${commentID}`).scrollIntoView();
+  }
+
+  scrollDown() {
+    setTimeout(() => {
+      const el = this.staffNotesContainer.nativeElement;
+      el.scrollTop = el.scrollHeight;
+    }, 50);
+  }
+
+  public submitNote() {
+    const noteText = this.noteText.trim();
+    if (noteText === '') {
+      return;
+    }
+
+    this.noteText = '';
+
+    this.tutorNoteService.addNote(this.unitRole, noteText, this.replyingToNote).subscribe({
+      next: (note) => {
+        this.alertService.success('Succesfully submitted note', 4000);
+        this.scrollDown();
+        // TODO: maybe we do export a tutorNoteCount? but then we dont know for which tasks they are for
+        // TODO: itll only be helpful in the header notifications icon...
+        // TODO: otherwise, we could just load all the notes for a mentors mentee when loading the audt page?
+
+        // this.project.staffNoteCount++;
+        this.replyingToNote = null;
+        this.tutorNoteService.updateTutorNoteReplies(this.unitRole?.tutorNotesCache.currentValues);
+      },
+      error: (error) => {
+        this.alertService.error(`Failed to create note: ${error}`, 4000);
+        this.noteText = noteText;
+      },
+    });
+  }
+
+  public updateNote() {
+    const noteText = this.editingNoteText.trim();
+    if (noteText === '' || !this.editingNote) {
+      return;
+    }
+
+    this.tutorNoteService.updateNote(this.unitRole, this.editingNote, noteText).subscribe({
+      next: (note) => {
+        this.alertService.success('Succesfully updated note', 4000);
+        this.editingNote = null;
+        this.editingNoteText = '';
+      },
+      error: (error) => {
+        this.alertService.error(`Failed to update note: ${error}`, 4000);
+      },
+    });
+  }
+
+  public deleteNote(note: TutorNote) {
+    this.confirmationModalService.show(
+      'Delete note',
+      'Are you sure want to delete this tutor note?',
+      () => {
+        note.delete();
+      },
+    );
+  }
+
+  public replyToNote(note: TutorNote) {
+    this.replyingToNote = note;
+  }
+  public cancelReplyingToNote() {
+    this.replyingToNote = null;
+  }
+
+  public editNote(note: TutorNote) {
+    if (!note.authorIsMe) {
+      return;
+    }
+
+    this.editingNote = note;
+    this.editingNoteText = note.note;
+    setTimeout(() => {
+      this.autoResizeStaffNoteEditor();
+      this.staffNoteEditor?.nativeElement.focus();
+    });
+  }
+
+  public cancelEditingNote() {
+    this.editingNote = null;
+    this.editingNoteText = '';
+  }
+
+  public autoResizeStaffNoteEditor() {
+    const el = this.staffNoteEditor.nativeElement;
+    el.style.height = 'auto';
+    el.offsetHeight;
+    el.style.height = el.scrollHeight + 'px';
+  }
+
+  scrollToNote(note: TutorNote): void {
+    const el = document.getElementById(`note-${note.id}`);
+    if (el) {
+      el.scrollIntoView({behavior: 'smooth', block: 'center'});
+      el.classList.add('flash-highlight');
+      setTimeout(() => el.classList.remove('flash-highlight'), 1000);
+    }
+  }
+}
diff --git a/src/app/units/states/edit/directives/unit-staff-editor/unit-staff-editor.component.html b/src/app/units/states/edit/directives/unit-staff-editor/unit-staff-editor.component.html
index ae0ca0114d..431b09dfc3 100644
--- a/src/app/units/states/edit/directives/unit-staff-editor/unit-staff-editor.component.html
+++ b/src/app/units/states/edit/directives/unit-staff-editor/unit-staff-editor.component.html
@@ -70,6 +70,25 @@ <h3>Unit Staff</h3>
         ></mat-slide-toggle>
       </td>
     </ng-container>
+
+    <ng-container matColumnDef="mentor">
+      <th mat-header-cell *matHeaderCellDef>Mentor</th>
+      <td mat-cell *matCellDef="let unitRole">
+        <mat-select
+          class="w-full min-w-fit"
+          (selectionChange)="selectMentor(unitRole, $event)"
+          [ngModel]="unitRole.mentorId"
+        >
+          <mat-option value="">(None)</mat-option>
+          @for (unitRole of unitStaff; track unitRole) {
+            <mat-option class="w-full min-w-fit" [value]="unitRole.id">
+              {{ unitRole.user.name }}</mat-option
+            >
+          }
+        </mat-select>
+      </td>
+    </ng-container>
+
     <ng-container matColumnDef="actions">
       <th mat-header-cell *matHeaderCellDef>Actions</th>
       <td mat-cell *matCellDef="let unitRole">
diff --git a/src/app/units/states/edit/directives/unit-staff-editor/unit-staff-editor.component.ts b/src/app/units/states/edit/directives/unit-staff-editor/unit-staff-editor.component.ts
index f161c384aa..bf55d7b45d 100644
--- a/src/app/units/states/edit/directives/unit-staff-editor/unit-staff-editor.component.ts
+++ b/src/app/units/states/edit/directives/unit-staff-editor/unit-staff-editor.component.ts
@@ -7,6 +7,7 @@ import {UnitRole} from 'src/app/api/models/unit-role';
 import {MatTableDataSource} from '@angular/material/table';
 import {MatButtonToggleChange} from '@angular/material/button-toggle';
 import {ConfirmationModalService} from 'src/app/common/modals/confirmation-modal/confirmation-modal.service';
+import {MatSelectChange} from '@angular/material/select';
 
 @Component({
   selector: 'unit-staff-editor',
@@ -22,7 +23,14 @@ export class UnitStaffEditorComponent implements OnInit {
   filteredStaff: User[] = []; // Filtered staff members
   searchTerm: string = ''; // Search term entered by the user
 
-  displayedColumns: string[] = ['name', 'role', 'main-convenor', 'observer-only', 'actions'];
+  displayedColumns: string[] = [
+    'name',
+    'role',
+    'main-convenor',
+    'observer-only',
+    'mentor',
+    'actions',
+  ];
   dataSource = new MatTableDataSource<UnitRole>();
 
   // Inject services here
@@ -87,6 +95,20 @@ export class UnitStaffEditorComponent implements OnInit {
     });
   }
 
+  selectMentor(unitRole: UnitRole, event: MatSelectChange) {
+    const previousValue = unitRole.mentorId;
+    unitRole.mentorId = event.value;
+    unitRole.roleId = unitRole.role === 'Tutor' ? 2 : 3;
+
+    this.unitRoleService.update(unitRole).subscribe({
+      next: () => this.alertService.success('Mentor updated', 2000),
+      error: (response) => {
+        // Revert changes on error
+        unitRole.mentorId = previousValue;
+        this.alertService.error(response, 6000);
+      },
+    });
+  }
   /**
    * Changes who the `Main Convenor` of the unit is.
    *

From a74742a7059276863259112abfc483a6811a6346 Mon Sep 17 00:00:00 2001
From: b0ink <40929320+b0ink@users.noreply.github.com>
Date: Thu, 8 Jan 2026 13:16:07 +1100
Subject: [PATCH 02/38] feat: tutor moderation route

---
 src/app/api/services/task.service.ts          | 19 +++++++++++
 src/app/common/footer/footer.component.ts     |  2 +-
 .../task-dropdown.component.html              |  7 ++++
 .../task-dropdown/task-dropdown.component.ts  |  1 +
 src/app/doubtfire-angularjs.module.ts         |  1 +
 .../states/tasks/inbox/inbox.component.html   | 32 +++++++++----------
 .../states/tasks/inbox/inbox.component.ts     | 25 ++++++++-------
 .../units/states/tasks/inbox/inbox.tpl.html   | 10 +++++-
 .../states/tasks/moderation/moderation.coffee | 31 ++++++++++++++++++
 src/app/units/states/tasks/tasks.coffee       |  1 +
 10 files changed, 100 insertions(+), 29 deletions(-)
 create mode 100644 src/app/units/states/tasks/moderation/moderation.coffee

diff --git a/src/app/api/services/task.service.ts b/src/app/api/services/task.service.ts
index e8e51ace8b..54d92d49c1 100644
--- a/src/app/api/services/task.service.ts
+++ b/src/app/api/services/task.service.ts
@@ -22,6 +22,7 @@ export class TaskService extends CachedEntityService<Task> {
 
   private readonly taskInboxEndpoint = '/units/:id:/tasks/inbox';
   private readonly taskExplorerEndpoint = '/units/:id:/task_definitions/:task_def_id:/tasks';
+  private readonly taskModerationEndpoint = '/units/:id:/tasks/moderation';
   private readonly refreshTaskEndpoint = 'projects/:projectId:/refresh_tasks/:taskDefinitionId:';
 
   constructor(httpClient: HttpClient) {
@@ -155,6 +156,24 @@ export class TaskService extends CachedEntityService<Task> {
     );
   }
 
+  public queryTasksForMentorModeration(unit: Unit): Observable<Task[]> {
+    const cache: EntityCache<Task> = new EntityCache<Task>();
+    return this.query(
+      {
+        id: unit.id,
+      },
+      {
+        endpointFormat: this.taskModerationEndpoint,
+        cache: cache,
+        constructorParams: unit,
+      },
+    ).pipe(
+      tap((tasks: Task[]) => {
+        unit.incorporateTasks(tasks);
+      }),
+    );
+  }
+
   public refreshExtensionDetails(task: Task): void {
     const pathIds = {
       projectId: task.project.id,
diff --git a/src/app/common/footer/footer.component.ts b/src/app/common/footer/footer.component.ts
index e058cedd68..f7c420eae4 100644
--- a/src/app/common/footer/footer.component.ts
+++ b/src/app/common/footer/footer.component.ts
@@ -53,7 +53,7 @@ export class FooterComponent implements OnInit {
 
   public markingTutor(task: Task): UnitRole {
     const enrolments = task.project.tutorialEnrolmentsCache.currentValues.filter(
-      (t) => t.tutorialStream.name === task.definition.tutorialStream.name,
+      (t) => t.tutorialStream.abbreviation === task.definition.tutorialStream.abbreviation,
     );
     // TODO: is checking for just the one tutorial enrolment correct? should be..
     if (enrolments.length === 1) {
diff --git a/src/app/common/header/task-dropdown/task-dropdown.component.html b/src/app/common/header/task-dropdown/task-dropdown.component.html
index cc8cec968f..451a3abc74 100644
--- a/src/app/common/header/task-dropdown/task-dropdown.component.html
+++ b/src/app/common/header/task-dropdown/task-dropdown.component.html
@@ -106,6 +106,13 @@
         >
           <mat-icon aria-label="Task Explorer Icon" fontIcon="pageview"></mat-icon> Explorer
         </button>
+        <button
+          uiSref="units/tasks/moderation"
+          [uiParams]="{unitId: unitRole.unit.id}"
+          mat-menu-item
+        >
+          <mat-icon aria-label="Audit Icon">rate_review</mat-icon> Moderation
+        </button>
         <button uiSref="units/tasks/viewer" [uiParams]="{unitId: unitRole.unit.id}" mat-menu-item>
           <mat-icon aria-label="Task list icon" fontIcon="list_alt"></mat-icon> Tasks
         </button>
diff --git a/src/app/common/header/task-dropdown/task-dropdown.component.ts b/src/app/common/header/task-dropdown/task-dropdown.component.ts
index b67f220b03..cd733052ec 100644
--- a/src/app/common/header/task-dropdown/task-dropdown.component.ts
+++ b/src/app/common/header/task-dropdown/task-dropdown.component.ts
@@ -24,6 +24,7 @@ export class TaskDropdownComponent {
     'Student List': 'Students',
     'Student Portfolios': 'Portfolios',
     'Task Explorer': 'Task Explorer',
+    'Task Moderation': 'Task Moderation',
     'Task Inbox': 'Inbox',
     'Task Lists': 'Tasks',
     'Tutorial List': 'Tutorials',
diff --git a/src/app/doubtfire-angularjs.module.ts b/src/app/doubtfire-angularjs.module.ts
index 69f7180a11..abf7f6d5ab 100644
--- a/src/app/doubtfire-angularjs.module.ts
+++ b/src/app/doubtfire-angularjs.module.ts
@@ -95,6 +95,7 @@ import 'build/src/app/units/states/tasks/tasks.js';
 import 'build/src/app/units/states/tasks/viewer/directives/directives.js';
 import 'build/src/app/units/states/tasks/viewer/viewer.js';
 import 'build/src/app/units/states/tasks/definition/definition.js';
+import 'build/src/app/units/states/tasks/moderation/moderation.js';
 import 'build/src/app/units/states/portfolios/portfolios.js';
 import 'build/src/app/units/states/groups/groups.js';
 import 'build/src/app/units/states/states.js';
diff --git a/src/app/units/states/tasks/inbox/inbox.component.html b/src/app/units/states/tasks/inbox/inbox.component.html
index 84edcabe13..328ca94ed2 100644
--- a/src/app/units/states/tasks/inbox/inbox.component.html
+++ b/src/app/units/states/tasks/inbox/inbox.component.html
@@ -2,14 +2,14 @@
   <div fxLayout="row" fxLayoutAlign="center stretch" fxFill>
     <div id="inboxpanel" #inboxpanel class="staff-task-list inbox-panel">
       @if (taskData) {
-      <df-staff-task-list
-        [isNarrow]="narrowTaskInbox"
-        [showSearchOptions]="showSearchOptions"
-        [filters]="filters"
-        [taskData]="taskData"
-        [unit]="unit"
-        [unitRole]="unitRole"
-      ></df-staff-task-list>
+        <df-staff-task-list
+          [isNarrow]="narrowTaskInbox"
+          [showSearchOptions]="showSearchOptions"
+          [filters]="filters"
+          [taskData]="taskData"
+          [unit]="unit"
+          [unitRole]="unitRole"
+        ></df-staff-task-list>
       }
     </div>
     @if (subs$ | async) {
@@ -48,14 +48,14 @@
 <div #mobile fxLayout="row" fxFill fxHide fxShow.lt-md>
   <div id="inboxpanel" #inboxpanel class="inbox-panel mobile" [hidden]="taskSelected">
     @if (taskData) {
-    <df-staff-task-list
-      [isNarrow]="false"
-      [showSearchOptions]="showSearchOptions"
-      [filters]="filters"
-      [taskData]="taskData"
-      [unit]="unit"
-      [unitRole]="unitRole"
-    ></df-staff-task-list>
+      <df-staff-task-list
+        [isNarrow]="false"
+        [showSearchOptions]="showSearchOptions"
+        [filters]="filters"
+        [taskData]="taskData"
+        [unit]="unit"
+        [unitRole]="unitRole"
+      ></df-staff-task-list>
     }
   </div>
   <div [hidden]="!taskSelected" fxLayout="column" fxFill fxLayoutAlign="space-between stretch">
diff --git a/src/app/units/states/tasks/inbox/inbox.component.ts b/src/app/units/states/tasks/inbox/inbox.component.ts
index 7e73fcccb1..4df47dc500 100644
--- a/src/app/units/states/tasks/inbox/inbox.component.ts
+++ b/src/app/units/states/tasks/inbox/inbox.component.ts
@@ -1,12 +1,5 @@
 import {CdkDragEnd, CdkDragStart, CdkDragMove} from '@angular/cdk/drag-drop';
-import {
-  Component,
-  ElementRef,
-  Input,
-  OnDestroy,
-  OnInit,
-  ViewChild,
-} from '@angular/core';
+import {Component, ElementRef, Input, OnDestroy, OnInit, ViewChild} from '@angular/core';
 import {MediaObserver} from 'ng-flex-layout';
 import {UIRouter} from '@uirouter/angular';
 import {auditTime, merge, Observable, of, Subject, tap, withLatestFrom} from 'rxjs';
@@ -19,6 +12,8 @@ import {HotkeysService, HotkeysHelpComponent} from '@ngneat/hotkeys';
 import {MatDialog} from '@angular/material/dialog';
 import {UserService} from 'src/app/api/services/user.service';
 import {DoubtfireConstants} from 'src/app/config/constants/doubtfire-constants';
+import {Tutorial} from 'src/app/api/models/doubtfire-model';
+import {TaskDefinition} from 'src/app/api/models/task-definition';
 
 @Component({
   selector: 'f-inbox',
@@ -29,7 +24,15 @@ export class InboxComponent implements OnInit, OnDestroy {
   @Input() unit: Unit;
   @Input() unitRole: UnitRole;
   @Input() taskData: {selectedTask: Task; any};
-
+  @Input() filters: {
+    taskDefinition: TaskDefinition;
+    tutorials: Tutorial[];
+    forceStream: boolean;
+    studentName: string;
+    tutorialIdSelected: any;
+    taskDefinitionIdSelected: number | TaskDefinition;
+  };
+  @Input() showSearchOptions: boolean;
   @ViewChild('inboxpanel') inboxPanel: ElementRef;
   @ViewChild('commentspanel') commentspanel: ElementRef;
 
@@ -39,8 +42,8 @@ export class InboxComponent implements OnInit, OnDestroy {
   private dragMove$ = new Subject<{event: CdkDragMove; div: HTMLDivElement}>();
   private dragMoveAudited$;
 
-  protected filters;
-  protected showSearchOptions;
+  // protected filters;
+  // protected showSearchOptions;
 
   public taskSelected = false;
 
diff --git a/src/app/units/states/tasks/inbox/inbox.tpl.html b/src/app/units/states/tasks/inbox/inbox.tpl.html
index d77bb14cc2..d1769779ea 100644
--- a/src/app/units/states/tasks/inbox/inbox.tpl.html
+++ b/src/app/units/states/tasks/inbox/inbox.tpl.html
@@ -1 +1,9 @@
-<f-inbox [unit]="unit" [task-data]="taskData" [unit-role]="unitRole" [task]="task" [project]="project"></f-inbox>
+<f-inbox
+  [unit]="unit"
+  [task-data]="taskData"
+  [unit-role]="unitRole"
+  [task]="task"
+  [project]="project"
+  [filters]="filters"
+  [showSearchOptions]="showSearchOptions"
+></f-inbox>
diff --git a/src/app/units/states/tasks/moderation/moderation.coffee b/src/app/units/states/tasks/moderation/moderation.coffee
new file mode 100644
index 0000000000..446df9a389
--- /dev/null
+++ b/src/app/units/states/tasks/moderation/moderation.coffee
@@ -0,0 +1,31 @@
+angular.module('doubtfire.units.states.tasks.moderation', [
+])
+
+#
+# Get tasks for mentor moderation
+#
+.config(($stateProvider) ->
+  $stateProvider.state 'units/tasks/moderation', {
+    parent: 'units/tasks'
+    url: '/moderation/{taskKey:any}'
+    # We can recycle the task inbox, switching the data source scope variable
+    templateUrl: "units/states/tasks/inbox/inbox.tpl.html"
+    controller: "TaskModerationStateCtrl"
+    params:
+      taskKey: dynamic: true
+    data:
+      task: "Task Moderation"
+      pageTitle: "_Home_"
+      roleWhitelist: ['Tutor', 'Convenor', 'Admin', 'Auditor']
+   }
+)
+
+.controller('TaskModerationStateCtrl', ($scope, newTaskService) ->
+  $scope.taskData.source = newTaskService.queryTasksForMentorModeration.bind(newTaskService)
+
+  $scope.taskData.taskDefMode = false
+  $scope.showSearchOptions = false
+  $scope.filters = {
+    tutorialIdSelected: 'all'
+  }
+)
diff --git a/src/app/units/states/tasks/tasks.coffee b/src/app/units/states/tasks/tasks.coffee
index 141e0364c6..c9fbef5cb4 100644
--- a/src/app/units/states/tasks/tasks.coffee
+++ b/src/app/units/states/tasks/tasks.coffee
@@ -1,6 +1,7 @@
 angular.module('doubtfire.units.states.tasks', [
   'doubtfire.units.states.tasks.inbox'
   'doubtfire.units.states.tasks.definition'
+  'doubtfire.units.states.tasks.moderation'
   'doubtfire.units.states.tasks.viewer'
 ])
 

From 569c829d22c0b1b0a2802fd74c8ad23a64e603c3 Mon Sep 17 00:00:00 2001
From: b0ink <40929320+b0ink@users.noreply.github.com>
Date: Mon, 12 Jan 2026 13:54:49 +1100
Subject: [PATCH 03/38] feat: add moderator actions

---
 src/app/api/models/task.ts                    | 34 ++++++++++
 src/app/doubtfire-angular.module.ts           |  2 +
 .../states/tasks/definition/definition.coffee |  1 +
 .../moderation/moderation.component.html      | 28 ++++++++
 .../moderation/moderation.component.scss      |  0
 .../moderation/moderation.component.ts        | 67 +++++++++++++++++++
 src/app/units/states/tasks/inbox/inbox.coffee |  1 +
 .../states/tasks/inbox/inbox.component.html   |  3 +
 .../states/tasks/inbox/inbox.component.ts     |  2 +
 .../units/states/tasks/inbox/inbox.tpl.html   |  3 +-
 .../states/tasks/moderation/moderation.coffee |  3 +-
 11 files changed, 141 insertions(+), 3 deletions(-)
 create mode 100644 src/app/units/states/tasks/inbox/directives/moderation/moderation.component.html
 create mode 100644 src/app/units/states/tasks/inbox/directives/moderation/moderation.component.scss
 create mode 100644 src/app/units/states/tasks/inbox/directives/moderation/moderation.component.ts

diff --git a/src/app/api/models/task.ts b/src/app/api/models/task.ts
index e2d85bf61b..e8f71eed5c 100644
--- a/src/app/api/models/task.ts
+++ b/src/app/api/models/task.ts
@@ -18,6 +18,8 @@ import {
   TestAttempt,
   TestAttemptService,
   ScormComment,
+  UnitRoleService,
+  UnitRole,
 } from './doubtfire-model';
 import {Grade} from './grade';
 import {LOCALE_ID} from '@angular/core';
@@ -93,6 +95,16 @@ export class Task extends Entity {
     return this.commentCache.currentValues;
   }
 
+  public get tutor(): UnitRole {
+    const enrolments = this.project.tutorialEnrolmentsCache.currentValues.filter(
+      (t) => t.tutorialStream.name === this.definition.tutorialStream.name,
+    );
+    if (enrolments.length === 1) {
+      const user = enrolments[0].tutor;
+      return this.unit.staff.find((ur) => ur.user.id === user.id);
+    }
+  }
+
   public addComment(textString): void {
     AppInjector.get(TaskCommentService)
       .addComment(this, textString, 'text')
@@ -982,4 +994,26 @@ export class Task extends Entity {
 
     return false;
   }
+
+  public moderateFeedback(score: -1 | 0 | 1): Observable<boolean> {
+    const unitRoleService: UnitRoleService = AppInjector.get(UnitRoleService);
+
+    const tutor = this.tutor;
+    if (!tutor) {
+      return;
+    }
+
+    return unitRoleService.post(
+      {
+        id: tutor.id,
+        taskId: this.id,
+      },
+      {
+        endpointFormat: '/unit_roles/:id:/moderation/:taskId:',
+        body: {
+          score: score,
+        },
+      },
+    );
+  }
 }
diff --git a/src/app/doubtfire-angular.module.ts b/src/app/doubtfire-angular.module.ts
index 11220a1da3..d456484b19 100644
--- a/src/app/doubtfire-angular.module.ts
+++ b/src/app/doubtfire-angular.module.ts
@@ -315,6 +315,7 @@ import {OverseerStepResultService} from './api/services/overseer-step-result.ser
 import {TutorNotesComponent} from './projects/states/tutor-notes/tutor-notes.component';
 import {TutorNotesViewComponent} from './projects/states/dashboard/directives/task-dashboard/directives/tutor-notes-view/tutor-notes-view.component';
 import {TutorNoteService} from './api/services/tutor-note.service';
+import {ModerationComponent} from './units/states/tasks/inbox/directives/moderation/moderation.component';
 
 // See https://stackoverflow.com/questions/55721254/how-to-change-mat-datepicker-date-format-to-dd-mm-yyyy-in-simplest-way/58189036#58189036
 const MY_DATE_FORMAT = {
@@ -505,6 +506,7 @@ const GANTT_CHART_CONFIG = {
     TaskOverseerReportComponent,
     TutorNotesComponent,
     TutorNotesViewComponent,
+    ModerationComponent,
   ],
   providers: [
     // Services we provide
diff --git a/src/app/units/states/tasks/definition/definition.coffee b/src/app/units/states/tasks/definition/definition.coffee
index bd0bad8241..9dbdc249ea 100644
--- a/src/app/units/states/tasks/definition/definition.coffee
+++ b/src/app/units/states/tasks/definition/definition.coffee
@@ -22,6 +22,7 @@ angular.module('doubtfire.units.states.tasks.definition', [
 
 .controller('TaskDefinitionStateCtrl', ($scope, newTaskService) ->
   $scope.taskData.source = newTaskService.queryTasksForTaskExplorer.bind(newTaskService)
+  $scope.viewType = 'explorer'
   $scope.taskData.taskDefMode = true
   $scope.showSearchOptions = true
   $scope.filters = {
diff --git a/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.html b/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.html
new file mode 100644
index 0000000000..6d5f05b073
--- /dev/null
+++ b/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.html
@@ -0,0 +1,28 @@
+<div class="w-full p-5 flex justify-center gap-5">
+  <!-- TODO: Show the name of the tutor -->
+  <button
+    mat-stroked-button
+    (click)="showMore()"
+    [disabled]="isModerated(task)"
+    matTooltip="See more tasks from this tutor for moderation due to feedback concerns"
+  >
+    Show more from this tutor
+  </button>
+  <button
+    mat-stroked-button
+    color="primary"
+    (click)="showLess()"
+    [disabled]="isModerated(task)"
+    matTooltip="Remove this task from moderation without changing how many tasks from this tutor are shown"
+  >
+    Dismiss task
+  </button>
+  <button
+    mat-stroked-button
+    (click)="dismiss()"
+    [disabled]="isModerated(task)"
+    matTooltip="Reduce moderation for this tutor based on satisfactory feedback"
+  >
+    Show less from this tutor
+  </button>
+</div>
diff --git a/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.scss b/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.scss
new file mode 100644
index 0000000000..e69de29bb2
diff --git a/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.ts b/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.ts
new file mode 100644
index 0000000000..7ed48aa8c1
--- /dev/null
+++ b/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.ts
@@ -0,0 +1,67 @@
+import {Component, Input} from '@angular/core';
+import {Task} from 'src/app/api/models/task';
+import {AlertService} from 'src/app/common/services/alert.service';
+
+@Component({
+  selector: 'f-moderation',
+  templateUrl: './moderation.component.html',
+  styleUrl: './moderation.component.scss',
+})
+export class ModerationComponent {
+  @Input() task: Task;
+
+  public moderated: Map<Task, boolean> = new Map<Task, boolean>();
+
+  constructor(private alertService: AlertService) {}
+
+  showMore() {
+    console.log('bad!');
+
+    this.task.moderateFeedback(-1).subscribe({
+      next: (response) => {
+        console.log(response);
+        this.alertService.success(`Task moderated`, 2000);
+        this.setModerated(this.task);
+      },
+      error: (error) => {
+        this.alertService.error(`Failed to moderate task: ${error}`, 6000);
+      },
+    });
+  }
+
+  showLess() {
+    console.log('good!');
+
+    this.task.moderateFeedback(1).subscribe({
+      next: (response) => {
+        console.log(response);
+        this.alertService.success(`Task moderated`, 2000);
+        this.setModerated(this.task);
+      },
+      error: (error) => {
+        this.alertService.error(`Failed to moderate task: ${error}`, 6000);
+      },
+    });
+  }
+
+  dismiss() {
+    this.task.moderateFeedback(0).subscribe({
+      next: (response) => {
+        console.log(response);
+        this.alertService.success(`Task moderated`, 2000);
+        this.setModerated(this.task);
+      },
+      error: (error) => {
+        this.alertService.error(`Failed to moderate task: ${error}`, 6000);
+      },
+    });
+  }
+
+  private setModerated(task: Task) {
+    this.moderated.set(task, true);
+  }
+
+  public isModerated(task: Task): boolean {
+    return this.moderated.get(task);
+  }
+}
diff --git a/src/app/units/states/tasks/inbox/inbox.coffee b/src/app/units/states/tasks/inbox/inbox.coffee
index 9f04950ca0..77afab0740 100644
--- a/src/app/units/states/tasks/inbox/inbox.coffee
+++ b/src/app/units/states/tasks/inbox/inbox.coffee
@@ -20,5 +20,6 @@ angular.module('doubtfire.units.states.tasks.inbox', [])
 
 .controller('TaskInboxStateCtrl', ($scope, newTaskService) ->
   $scope.taskData.source = newTaskService.queryTasksForTaskInbox.bind(newTaskService)
+  $scope.viewType = 'inbox'
   $scope.taskData.taskDefMode = false
 )
diff --git a/src/app/units/states/tasks/inbox/inbox.component.html b/src/app/units/states/tasks/inbox/inbox.component.html
index 328ca94ed2..f28f83e243 100644
--- a/src/app/units/states/tasks/inbox/inbox.component.html
+++ b/src/app/units/states/tasks/inbox/inbox.component.html
@@ -26,6 +26,9 @@
     }
 
     <div fxFlex class="inbox-panel">
+      @if (viewType === 'moderation' && taskData.selectedTask) {
+        <f-moderation [task]="taskData.selectedTask"></f-moderation>
+      }
       <f-task-dashboard [task]="taskData.selectedTask" [pdfUrl]="visiblePdfUrl"></f-task-dashboard>
     </div>
     <div
diff --git a/src/app/units/states/tasks/inbox/inbox.component.ts b/src/app/units/states/tasks/inbox/inbox.component.ts
index 4df47dc500..9f2bd164ab 100644
--- a/src/app/units/states/tasks/inbox/inbox.component.ts
+++ b/src/app/units/states/tasks/inbox/inbox.component.ts
@@ -36,6 +36,8 @@ export class InboxComponent implements OnInit, OnDestroy {
   @ViewChild('inboxpanel') inboxPanel: ElementRef;
   @ViewChild('commentspanel') commentspanel: ElementRef;
 
+  @Input() viewType: 'inbox' | 'explorer' | 'moderation';
+
   subs$: Observable<unknown>;
 
   private inboxStartSize$ = new Subject<number>();
diff --git a/src/app/units/states/tasks/inbox/inbox.tpl.html b/src/app/units/states/tasks/inbox/inbox.tpl.html
index d1769779ea..9adb06b9ad 100644
--- a/src/app/units/states/tasks/inbox/inbox.tpl.html
+++ b/src/app/units/states/tasks/inbox/inbox.tpl.html
@@ -5,5 +5,6 @@
   [task]="task"
   [project]="project"
   [filters]="filters"
-  [showSearchOptions]="showSearchOptions"
+  [show-search-options]="showSearchOptions"
+  [view-type]="viewType"
 ></f-inbox>
diff --git a/src/app/units/states/tasks/moderation/moderation.coffee b/src/app/units/states/tasks/moderation/moderation.coffee
index 446df9a389..cc61515008 100644
--- a/src/app/units/states/tasks/moderation/moderation.coffee
+++ b/src/app/units/states/tasks/moderation/moderation.coffee
@@ -22,8 +22,7 @@ angular.module('doubtfire.units.states.tasks.moderation', [
 
 .controller('TaskModerationStateCtrl', ($scope, newTaskService) ->
   $scope.taskData.source = newTaskService.queryTasksForMentorModeration.bind(newTaskService)
-
-  $scope.taskData.taskDefMode = false
+  $scope.viewType = 'moderation'
   $scope.showSearchOptions = false
   $scope.filters = {
     tutorialIdSelected: 'all'

From 9f15954c7ce37f9aa65279c56b94c47d902a3c8b Mon Sep 17 00:00:00 2001
From: b0ink <40929320+b0ink@users.noreply.github.com>
Date: Mon, 12 Jan 2026 14:16:47 +1100
Subject: [PATCH 04/38] refactor: add confirmation modals

---
 .../moderation/moderation.component.html      |  5 +-
 .../moderation/moderation.component.ts        | 51 ++++++++++---------
 2 files changed, 31 insertions(+), 25 deletions(-)

diff --git a/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.html b/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.html
index 6d5f05b073..11d955afeb 100644
--- a/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.html
+++ b/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.html
@@ -1,3 +1,4 @@
+<div class="flex justify-center py-2">Tutor: {{ task.tutor?.user.name }}</div>
 <div class="w-full p-5 flex justify-center gap-5">
   <!-- TODO: Show the name of the tutor -->
   <button
@@ -11,7 +12,7 @@
   <button
     mat-stroked-button
     color="primary"
-    (click)="showLess()"
+    (click)="dismiss()"
     [disabled]="isModerated(task)"
     matTooltip="Remove this task from moderation without changing how many tasks from this tutor are shown"
   >
@@ -19,7 +20,7 @@
   </button>
   <button
     mat-stroked-button
-    (click)="dismiss()"
+    (click)="showLess()"
     [disabled]="isModerated(task)"
     matTooltip="Reduce moderation for this tutor based on satisfactory feedback"
   >
diff --git a/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.ts b/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.ts
index 7ed48aa8c1..7ae8b2745c 100644
--- a/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.ts
+++ b/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.ts
@@ -1,5 +1,6 @@
 import {Component, Input} from '@angular/core';
 import {Task} from 'src/app/api/models/task';
+import {ConfirmationModalService} from 'src/app/common/modals/confirmation-modal/confirmation-modal.service';
 import {AlertService} from 'src/app/common/services/alert.service';
 
 @Component({
@@ -12,40 +13,44 @@ export class ModerationComponent {
 
   public moderated: Map<Task, boolean> = new Map<Task, boolean>();
 
-  constructor(private alertService: AlertService) {}
+  constructor(
+    private alertService: AlertService,
+    private confirmationModal: ConfirmationModalService,
+  ) {}
 
   showMore() {
     console.log('bad!');
-
-    this.task.moderateFeedback(-1).subscribe({
-      next: (response) => {
-        console.log(response);
-        this.alertService.success(`Task moderated`, 2000);
-        this.setModerated(this.task);
+    this.confirmationModal.show(
+      'Show more from this tutor',
+      `There were concerns with feedback and you would like to see more tasks from this tutor. You will review this task again if further feedback is provided.`,
+      () => {
+        this.moderateTask(-1);
       },
-      error: (error) => {
-        this.alertService.error(`Failed to moderate task: ${error}`, 6000);
-      },
-    });
+    );
   }
 
   showLess() {
-    console.log('good!');
-
-    this.task.moderateFeedback(1).subscribe({
-      next: (response) => {
-        console.log(response);
-        this.alertService.success(`Task moderated`, 2000);
-        this.setModerated(this.task);
-      },
-      error: (error) => {
-        this.alertService.error(`Failed to moderate task: ${error}`, 6000);
+    this.confirmationModal.show(
+      'Show less from this tutor',
+      `The feedback provided by the tutor is satisfactory and you would like to see less of their feedback for moderation. You won't see this task again.`,
+      () => {
+        this.moderateTask(1);
       },
-    });
+    );
   }
 
   dismiss() {
-    this.task.moderateFeedback(0).subscribe({
+    this.confirmationModal.show(
+      'Dismiss from moderation',
+      'This task will be removed from moderation and will not get a follow up. This will not affect how many tasks from this tutor are selected for moderation.',
+      () => {
+        this.moderateTask(0);
+      },
+    );
+  }
+
+  private moderateTask(score: -1 | 0 | 1) {
+    this.task.moderateFeedback(score).subscribe({
       next: (response) => {
         console.log(response);
         this.alertService.success(`Task moderated`, 2000);

From c380be70c3a7d0aacd18a5fa77afe1c56ca6ceea Mon Sep 17 00:00:00 2001
From: b0ink <40929320+b0ink@users.noreply.github.com>
Date: Tue, 13 Jan 2026 11:19:49 +1100
Subject: [PATCH 05/38] refactor: move moderation buttons to footer

---
 src/app/common/footer/footer.component.html   | 181 +++++++++---------
 src/app/common/footer/footer.component.ts     |   4 +-
 .../tutor-notes-view.component.html           |   5 +
 .../moderation/moderation.component.html      |  41 ++--
 .../moderation/moderation.component.scss      |  27 +++
 .../states/tasks/inbox/inbox.component.html   |   5 +-
 6 files changed, 153 insertions(+), 110 deletions(-)

diff --git a/src/app/common/footer/footer.component.html b/src/app/common/footer/footer.component.html
index 033beedc92..71b483c52f 100644
--- a/src/app/common/footer/footer.component.html
+++ b/src/app/common/footer/footer.component.html
@@ -13,114 +13,119 @@
 
   <div class="flex w-full">
     <f-user-badge fxFlex fxFlexAlign="start start" [selectedTask]="selectedTask"></f-user-badge>
+    <div class="flex justify-between">
+      <span class="center-buttons flex justify-around gap-5">
+        <div>
+          <button
+            [disabled]="selectedTask === undefined"
+            mat-icon-button
+            matTooltip="Mark as Redo"
+            matTooltipPosition="above"
+            aria-label=""
+            class="button"
+            (click)="selectedTask?.updateTaskStatus('redo')"
+          >
+            <mat-icon>undo</mat-icon>
+          </button>
 
-    <span class="center-buttons flex justify-around gap-5">
-      <div>
+          <button
+            [disabled]="selectedTask === undefined"
+            mat-icon-button
+            matTooltip="Mark as Attention Required"
+            matTooltipPosition="above"
+            aria-label=""
+            class="button"
+            (click)="selectedTask?.updateTaskStatus('attention_required')"
+          >
+            <mat-icon>sms_failed</mat-icon>
+          </button>
+        </div>
         <button
           [disabled]="selectedTask === undefined"
           mat-icon-button
-          matTooltip="Mark as Redo"
+          matTooltip="Mark as Resubmit"
           matTooltipPosition="above"
           aria-label=""
-          class="button"
-          (click)="selectedTask?.updateTaskStatus('redo')"
+          class="button large-button"
+          (click)="selectedTask?.updateTaskStatus('fix')"
         >
-          <mat-icon>undo</mat-icon>
+          <mat-icon>construction</mat-icon>
         </button>
 
+        @if (selectedTask && selectedTask.suggestedTaskStatus) {
+          <button
+            mat-icon-button
+            matTooltip="Recommended: Mark as {{
+              taskService.statusData(selectedTask.suggestedTaskStatus).label
+            }}"
+            matTooltipPosition="above"
+            aria-label=""
+            class="button extra-large-button status-chip"
+            (click)="selectedTask?.updateTaskStatus(selectedTask.suggestedTaskStatus)"
+            [ngClass]="taskService.statusData(selectedTask.suggestedTaskStatus).class"
+          >
+            <i class="fa {{ taskService.statusData(selectedTask.suggestedTaskStatus).icon }}"></i>
+          </button>
+        }
+        @if (selectedTask?.definition?.assessInPortfolioOnly) {
+          <button
+            [disabled]="selectedTask === undefined"
+            mat-icon-button
+            matTooltip="Mark as Working On It"
+            matTooltipPosition="above"
+            aria-label="Mark as Working On It"
+            class="button large-button"
+            (click)="markTaskWorkingOnIt(selectedTask)"
+          >
+            <mat-icon>bolt</mat-icon>
+          </button>
+        } @else {
+          <button
+            [disabled]="selectedTask === undefined"
+            mat-icon-button
+            matTooltip="Mark as Complete"
+            matTooltipPosition="above"
+            aria-label="Mark as Complete"
+            class="button large-button"
+            (click)="selectedTask?.updateTaskStatus('complete')"
+          >
+            <mat-icon>done</mat-icon>
+          </button>
+        }
+
         <button
           [disabled]="selectedTask === undefined"
           mat-icon-button
-          matTooltip="Mark as Attention Required"
+          matTooltip="Mark as Discuss"
           matTooltipPosition="above"
           aria-label=""
           class="button"
-          (click)="selectedTask?.updateTaskStatus('attention_required')"
+          (click)="selectedTask?.updateTaskStatus('discuss')"
         >
-          <mat-icon>sms_failed</mat-icon>
+          <mat-icon>question_answer</mat-icon>
         </button>
-      </div>
-      <button
-        [disabled]="selectedTask === undefined"
-        mat-icon-button
-        matTooltip="Mark as Resubmit"
-        matTooltipPosition="above"
-        aria-label=""
-        class="button large-button"
-        (click)="selectedTask?.updateTaskStatus('fix')"
-      >
-        <mat-icon>construction</mat-icon>
-      </button>
 
-      @if (selectedTask && selectedTask.suggestedTaskStatus) {
-        <button
-          mat-icon-button
-          matTooltip="Recommended: Mark as {{
-            taskService.statusData(selectedTask.suggestedTaskStatus).label
-          }}"
-          matTooltipPosition="above"
-          aria-label=""
-          class="button extra-large-button status-chip"
-          (click)="selectedTask?.updateTaskStatus(selectedTask.suggestedTaskStatus)"
-          [ngClass]="taskService.statusData(selectedTask.suggestedTaskStatus).class"
-        >
-          <i class="fa {{ taskService.statusData(selectedTask.suggestedTaskStatus).icon }}"></i>
-        </button>
-      }
-      @if (selectedTask?.definition?.assessInPortfolioOnly) {
-        <button
-          [disabled]="selectedTask === undefined"
-          mat-icon-button
-          matTooltip="Mark as Working On It"
-          matTooltipPosition="above"
-          aria-label="Mark as Working On It"
-          class="button large-button"
-          (click)="markTaskWorkingOnIt(selectedTask)"
-        >
-          <mat-icon>bolt</mat-icon>
-        </button>
-      } @else {
-        <button
-          [disabled]="selectedTask === undefined"
-          mat-icon-button
-          matTooltip="Mark as Complete"
-          matTooltipPosition="above"
-          aria-label="Mark as Complete"
-          class="button large-button"
-          (click)="selectedTask?.updateTaskStatus('complete')"
-        >
-          <mat-icon>done</mat-icon>
-        </button>
-      }
+        <mat-divider [vertical]="true"></mat-divider>
 
-      <button
-        [disabled]="selectedTask === undefined"
-        mat-icon-button
-        matTooltip="Mark as Discuss"
-        matTooltipPosition="above"
-        aria-label=""
-        class="button"
-        (click)="selectedTask?.updateTaskStatus('discuss')"
-      >
-        <mat-icon>question_answer</mat-icon>
-      </button>
-
-      <mat-divider [vertical]="true"></mat-divider>
+        <div class="w-[120px] flex justify-start">
+          <button
+            [disabled]="selectedTask === undefined"
+            mat-icon-button
+            matTooltip="Discussed in class"
+            matTooltipPosition="above"
+            aria-label=""
+            class="button ml-5"
+            (click)="selectedTask?.markAsDiscussed()"
+          >
+            <mat-icon>group</mat-icon>
+          </button>
+        </div>
+      </span>
 
-      <div class="w-[120px] flex justify-start">
-        <button
-          [disabled]="selectedTask === undefined"
-          mat-icon-button
-          matTooltip="Discussed in class"
-          matTooltipPosition="above"
-          aria-label=""
-          class="button ml-5"
-          (click)="selectedTask?.markAsDiscussed()"
-        >
-          <mat-icon>group</mat-icon>
-        </button>
-      </div>
-    </span>
+      @if (viewType === 'moderation' && selectedTask) {
+        <f-moderation [task]="selectedTask"></f-moderation>
+      }
+    </div>
 
     <div fxFlex fxFlexAlign="end start" fxLayout="row" fxLayoutAlign="end center">
       @if (selectedTask?.similaritiesDetected) {
diff --git a/src/app/common/footer/footer.component.ts b/src/app/common/footer/footer.component.ts
index f7c420eae4..7c562b3d33 100644
--- a/src/app/common/footer/footer.component.ts
+++ b/src/app/common/footer/footer.component.ts
@@ -1,4 +1,4 @@
-import {Component, ElementRef, HostListener, OnInit, ViewChild} from '@angular/core';
+import {Component, ElementRef, HostListener, Input, OnInit, ViewChild} from '@angular/core';
 import {Observable} from 'rxjs';
 import {Task} from 'src/app/api/models/task';
 import {SelectedTaskService} from 'src/app/projects/states/dashboard/selected-task.service';
@@ -22,6 +22,8 @@ export class FooterComponent implements OnInit {
     private userService: UserService,
   ) {}
 
+  @Input() viewType: 'inbox' | 'explorer' | 'moderation';
+
   selectedTask$: Observable<Task>;
   selectedTask: Task;
 
diff --git a/src/app/projects/states/dashboard/directives/task-dashboard/directives/tutor-notes-view/tutor-notes-view.component.html b/src/app/projects/states/dashboard/directives/task-dashboard/directives/tutor-notes-view/tutor-notes-view.component.html
index 2c8698b7b9..c9f9bf872f 100644
--- a/src/app/projects/states/dashboard/directives/task-dashboard/directives/tutor-notes-view/tutor-notes-view.component.html
+++ b/src/app/projects/states/dashboard/directives/task-dashboard/directives/tutor-notes-view/tutor-notes-view.component.html
@@ -3,6 +3,11 @@
     <mat-icon class="mr-2">comment</mat-icon>
     <h1 class="mat-headline m-0">Tutor Notes for {{ unitRole?.user?.name }}</h1>
   </div>
+  <p>
+    Notes left here are for staff-only discussion about the feedback given to the student. Use this
+    space to question, clarify, or discuss the tutor’s feedback. These notes are visible to
+    {{ unitRole?.user?.name }} and other convenors.
+  </p>
 
   <f-tutor-notes [task]="task"></f-tutor-notes>
 </div>
diff --git a/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.html b/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.html
index 11d955afeb..2260d2346a 100644
--- a/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.html
+++ b/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.html
@@ -1,29 +1,36 @@
-<div class="flex justify-center py-2">Tutor: {{ task.tutor?.user.name }}</div>
-<div class="w-full p-5 flex justify-center gap-5">
-  <!-- TODO: Show the name of the tutor -->
+<!-- <div class="flex justify-center py-2">Tutor: {{ task.tutor?.user.name }}</div> -->
+<span class="center-buttons flex justify-around gap-2">
   <button
-    mat-stroked-button
+    [disabled]="task === undefined || isModerated(task)"
+    mat-icon-button
+    matTooltip="Show more from this tutor"
+    matTooltipPosition="above"
+    aria-label=""
+    class="button"
     (click)="showMore()"
-    [disabled]="isModerated(task)"
-    matTooltip="See more tasks from this tutor for moderation due to feedback concerns"
   >
-    Show more from this tutor
+    <mat-icon>flag</mat-icon>
   </button>
   <button
-    mat-stroked-button
-    color="primary"
+    [disabled]="task === undefined || isModerated(task)"
+    mat-icon-button
+    matTooltip="Dismiss moderation task"
+    matTooltipPosition="above"
+    aria-label=""
+    class="button large-button"
     (click)="dismiss()"
-    [disabled]="isModerated(task)"
-    matTooltip="Remove this task from moderation without changing how many tasks from this tutor are shown"
   >
-    Dismiss task
+    <mat-icon>archive</mat-icon>
   </button>
   <button
-    mat-stroked-button
+    [disabled]="task === undefined || isModerated(task)"
+    mat-icon-button
+    matTooltip="Show less from this tutor"
+    matTooltipPosition="above"
+    aria-label=""
+    class="button"
     (click)="showLess()"
-    [disabled]="isModerated(task)"
-    matTooltip="Reduce moderation for this tutor based on satisfactory feedback"
   >
-    Show less from this tutor
+    <mat-icon>thumb_up</mat-icon>
   </button>
-</div>
+</span>
diff --git a/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.scss b/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.scss
index e69de29bb2..9c9f0165e1 100644
--- a/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.scss
+++ b/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.scss
@@ -0,0 +1,27 @@
+.center-buttons {
+  text-align: center;
+
+  .button {
+    margin: 0 6px;
+    transform: scale(0.8);
+    border-radius: 100%;
+  }
+
+  .large-button {
+    transform: scale(1.15);
+    background-color: white;
+  }
+
+  .large-button:hover {
+    background-color: #f5f5f5;
+  }
+
+  .extra-large-button {
+    padding-top: 7px;
+    transform: scale(1.25);
+  }
+
+  .extra-large-button:hover {
+    filter: brightness(0.85);
+  }
+}
diff --git a/src/app/units/states/tasks/inbox/inbox.component.html b/src/app/units/states/tasks/inbox/inbox.component.html
index f28f83e243..cdaee8d1bd 100644
--- a/src/app/units/states/tasks/inbox/inbox.component.html
+++ b/src/app/units/states/tasks/inbox/inbox.component.html
@@ -26,9 +26,6 @@
     }
 
     <div fxFlex class="inbox-panel">
-      @if (viewType === 'moderation' && taskData.selectedTask) {
-        <f-moderation [task]="taskData.selectedTask"></f-moderation>
-      }
       <f-task-dashboard [task]="taskData.selectedTask" [pdfUrl]="visiblePdfUrl"></f-task-dashboard>
     </div>
     <div
@@ -45,7 +42,7 @@
       <task-comments-viewer [task]="taskData.selectedTask"> </task-comments-viewer>
     </div>
   </div>
-  <f-footer></f-footer>
+  <f-footer class="z-10" [viewType]="viewType"></f-footer>
 </div>
 
 <div #mobile fxLayout="row" fxFill fxHide fxShow.lt-md>

From 9843bbb333575e6f57913dc7ccac23564a100b91 Mon Sep 17 00:00:00 2001
From: b0ink <40929320+b0ink@users.noreply.github.com>
Date: Tue, 13 Jan 2026 13:32:36 +1100
Subject: [PATCH 06/38] feat: filter tutor notes by task

---
 src/app/api/models/tutor-note.ts              |  6 +-
 src/app/api/services/tutor-note.service.ts    | 52 +++++++++---
 src/app/doubtfire-angular.module.ts           |  3 +-
 .../tutor-notes-view.component.html           |  9 ++-
 .../tutor-notes/tutor-notes.component.html    | 35 +++++++-
 .../tutor-notes/tutor-notes.component.ts      | 81 +++++++++++++++----
 6 files changed, 153 insertions(+), 33 deletions(-)

diff --git a/src/app/api/models/tutor-note.ts b/src/app/api/models/tutor-note.ts
index d585c1f5f5..3eb16acae3 100644
--- a/src/app/api/models/tutor-note.ts
+++ b/src/app/api/models/tutor-note.ts
@@ -1,5 +1,5 @@
 import {Entity} from 'ngx-entity-service';
-import {Project, Unit, User, UserService, Task, UnitRole} from './doubtfire-model';
+import {Project, Unit, User, UserService, Task, UnitRole, TaskDefinition} from './doubtfire-model';
 import {AppInjector} from 'src/app/app-injector';
 import {AlertService} from 'src/app/common/services/alert.service';
 import {StaffNoteService} from '../services/staff-note.service';
@@ -10,7 +10,9 @@ export class TutorNote extends Entity {
 
   // project: Project;
   unitRole: UnitRole;
-  task: Task;
+  task?: Task;
+  taskDefinition?: TaskDefinition;
+  project?: Project;
   user: User;
   note: string;
   replyTo?: TutorNote;
diff --git a/src/app/api/services/tutor-note.service.ts b/src/app/api/services/tutor-note.service.ts
index 39e08b98e3..a78db5dd01 100644
--- a/src/app/api/services/tutor-note.service.ts
+++ b/src/app/api/services/tutor-note.service.ts
@@ -2,7 +2,7 @@ import {HttpClient} from '@angular/common/http';
 import {Injectable} from '@angular/core';
 import {CachedEntityService, RequestOptions} from 'ngx-entity-service';
 import {Observable, tap} from 'rxjs';
-import {ProjectService, UnitRole, UserService} from 'src/app/api/models/doubtfire-model';
+import {ProjectService, Task, UnitRole, UserService} from 'src/app/api/models/doubtfire-model';
 import API_URL from 'src/app/config/constants/apiUrl';
 import {TutorNote} from '../models/tutor-note';
 
@@ -19,15 +19,38 @@ export class TutorNoteService extends CachedEntityService<TutorNote> {
   ) {
     super(httpClient, API_URL);
 
-    this.mapping.addKeys('id', 'note', 'createdAt', 'updatedAt', 'replyToId', {
-      keys: ['user', 'user_id'],
-      toEntityFn: (data: object, key: string, tutorNote: TutorNote) => {
-        const userRole = tutorNote.unitRole.unit.staff.find((s) => s.user.id === data['user_id']);
-        // const user = this.userService.cache.getOrCreate(data[key]?.id, userService, data[key]);
-        // If the user is not a staff in the unit it will be null
-        return userRole?.user;
+    this.mapping.addKeys(
+      'id',
+      'note',
+      'createdAt',
+      'updatedAt',
+      'replyToId',
+      {
+        keys: ['user', 'user_id'],
+        toEntityFn: (data: object, key: string, tutorNote: TutorNote) => {
+          const userRole = tutorNote.unitRole.unit.staff.find((s) => s.user.id === data['user_id']);
+          // const user = this.userService.cache.getOrCreate(data[key]?.id, userService, data[key]);
+          // If the user is not a staff in the unit it will be null
+          return userRole?.user;
+        },
+      },
+      {
+        keys: ['taskDefinition', 'task_definition_id'],
+        toEntityFn: (data: object, key: string, tutorNote: TutorNote) => {
+          const taskDefinition = tutorNote.unitRole.unit.taskDefinitions.find(
+            (td) => td.id === data['task_definition_id'],
+          );
+          return taskDefinition;
+        },
+      },
+      {
+        keys: ['project', 'project_id'],
+        toEntityFn: (data: object, key: string, tutorNote: TutorNote) => {
+          const project = tutorNote.unitRole.unit.students.find((p) => p.id === data['project_id']);
+          return project;
+        },
       },
-    });
+    );
 
     this.mapping.addJsonKey('note', 'createdAt', 'updatedAt');
   }
@@ -36,7 +59,12 @@ export class TutorNoteService extends CachedEntityService<TutorNote> {
     return new TutorNote(other);
   }
 
-  public addNote(unitRole: UnitRole, text: string, originalNote: TutorNote): Observable<TutorNote> {
+  public addNote(
+    unitRole: UnitRole,
+    text: string,
+    task?: Task,
+    originalNote?: TutorNote,
+  ): Observable<TutorNote> {
     const pathId = {
       unitRoleId: unitRole.id,
     };
@@ -46,6 +74,10 @@ export class TutorNoteService extends CachedEntityService<TutorNote> {
       body.append('reply_to_id', originalNote?.id.toString());
     }
 
+    if (task) {
+      body.append('task_id', task?.id.toString());
+    }
+
     body.append('note', text);
 
     const opts: RequestOptions<TutorNote> = {endpointFormat: this.endpointFormat};
diff --git a/src/app/doubtfire-angular.module.ts b/src/app/doubtfire-angular.module.ts
index d456484b19..26841bbd77 100644
--- a/src/app/doubtfire-angular.module.ts
+++ b/src/app/doubtfire-angular.module.ts
@@ -36,7 +36,7 @@ import {MatSnackBarModule} from '@angular/material/snack-bar';
 import {MatPaginatorModule} from '@angular/material/paginator';
 import {MatTooltipModule} from '@angular/material/tooltip';
 import {MatSlideToggleModule} from '@angular/material/slide-toggle';
-import {MatChipsModule} from '@angular/material/chips';
+import {MatChipListbox, MatChipsModule} from '@angular/material/chips';
 import {MatGridListModule} from '@angular/material/grid-list';
 import {PdfViewerModule} from 'ng2-pdf-viewer';
 import {UIRouterUpgradeModule} from '@uirouter/angular-hybrid';
@@ -668,6 +668,7 @@ const GANTT_CHART_CONFIG = {
     NgxGanttModule,
     MatSidenavModule,
     MonacoEditorModule.forRoot(),
+    MatChipListbox,
   ],
 })
 
diff --git a/src/app/projects/states/dashboard/directives/task-dashboard/directives/tutor-notes-view/tutor-notes-view.component.html b/src/app/projects/states/dashboard/directives/task-dashboard/directives/tutor-notes-view/tutor-notes-view.component.html
index c9f9bf872f..fb240c9b45 100644
--- a/src/app/projects/states/dashboard/directives/task-dashboard/directives/tutor-notes-view/tutor-notes-view.component.html
+++ b/src/app/projects/states/dashboard/directives/task-dashboard/directives/tutor-notes-view/tutor-notes-view.component.html
@@ -1,8 +1,15 @@
-<div class="p-4 h-[calc(100%-50px)]">
+<div class="p-4 h-[calc(100%-154px)]">
   <div class="flex items-center">
     <mat-icon class="mr-2">comment</mat-icon>
     <h1 class="mat-headline m-0">Tutor Notes for {{ unitRole?.user?.name }}</h1>
   </div>
+  @if (task) {
+    <p>
+      {{ task.definition.abbreviation }} {{ task.definition.name }} ({{
+        task.project.student.name
+      }})
+    </p>
+  }
   <p>
     Notes left here are for staff-only discussion about the feedback given to the student. Use this
     space to question, clarify, or discuss the tutor’s feedback. These notes are visible to
diff --git a/src/app/projects/states/tutor-notes/tutor-notes.component.html b/src/app/projects/states/tutor-notes/tutor-notes.component.html
index f3d87a9981..529581a4fe 100644
--- a/src/app/projects/states/tutor-notes/tutor-notes.component.html
+++ b/src/app/projects/states/tutor-notes/tutor-notes.component.html
@@ -6,7 +6,7 @@
   } -->
   <div #staffNotesContainer class="flex-1 overflow-auto p-4">
     @if (!loadingStaffNotes) {
-      @for (note of unitRole?.tutorNotesCache?.currentValues; track note) {
+      @for (note of filteredNotes; track note) {
         @if (note.replyToId) {
           <div
             class="text-md px-2 py-2 select-none rounded-t-md border border-gray-300 flex items-center justify-between"
@@ -44,6 +44,7 @@
             <mat-icon type="button" (click)="replyToNote(note)">reply</mat-icon>
             <mat-icon type="button" (click)="deleteNote(note)">delete</mat-icon>
           </div>
+
           <div class="flex justify-between">
             <mat-card-title class="flex items-center">
               <user-icon size="25" [user]="note.user" style="margin-right: 8px"></user-icon>
@@ -59,6 +60,23 @@
             </div>
           </div>
 
+          <div class="flex gap-5 py-2">
+            @if (note.taskDefinition) {
+              <mat-chip class="w-fit"
+                >{{ note.taskDefinition?.abbreviation }} {{ note.taskDefinition.name }}</mat-chip
+              >
+            }
+            @if (note.project) {
+              <!-- TODO: you should be able to click this chip to view the project and task definition -->
+              <mat-chip
+                chipListSelectable
+                class="w-fit cursor-pointer"
+                (click)="openProject($event, note)"
+                >{{ note.project?.student.name }}</mat-chip
+              >
+            }
+          </div>
+
           <mat-card-content class="py-4">
             @if (editingNote && editingNote.id === note.id) {
               <mat-form-field appearance="outline" class="flex items-center w-full">
@@ -93,6 +111,19 @@
   </div>
 
   <div class="h-[150px] border-t border-gray-300 p-4">
+    <mat-chip-listbox selectable multiple class="mb-4">
+      <mat-chip-option value="all" (click)="toggleSelection('all')">All Tasks</mat-chip-option>
+      @for (option of taskDefinitionFilters; track option) {
+        <mat-chip-option
+          [value]="option"
+          [selected]="selectedTaskDefinitions.get(option)"
+          (click)="toggleSelection(option)"
+        >
+          {{ option }}
+        </mat-chip-option>
+      }
+    </mat-chip-listbox>
+
     @if (replyingToNote) {
       <div
         class="text-md px-2 py-2 select-none rounded-t-md border border-b-0 border-gray-300 flex items-center justify-between"
@@ -113,7 +144,7 @@
     }
 
     <mat-form-field appearance="outline" class="flex items-center w-full">
-      <mat-label>Staff Note</mat-label>
+      <mat-label>Tutor note</mat-label>
       <textarea
         matInput
         [(ngModel)]="noteText"
diff --git a/src/app/projects/states/tutor-notes/tutor-notes.component.ts b/src/app/projects/states/tutor-notes/tutor-notes.component.ts
index 9b2b2c4eff..2c98ee217c 100644
--- a/src/app/projects/states/tutor-notes/tutor-notes.component.ts
+++ b/src/app/projects/states/tutor-notes/tutor-notes.component.ts
@@ -56,6 +56,9 @@ export class TutorNotesComponent implements OnInit {
       this.tutorNoteService.updateTutorNoteReplies(this.unitRole?.tutorNotesCache.currentValues);
       this.scrollDown();
     });
+    if (this.task) {
+      this.selectedTaskDefinitions.set(this.task.definition.abbreviation, true);
+    }
   }
 
   scrollToComment(commentID: number) {
@@ -77,23 +80,27 @@ export class TutorNotesComponent implements OnInit {
 
     this.noteText = '';
 
-    this.tutorNoteService.addNote(this.unitRole, noteText, this.replyingToNote).subscribe({
-      next: (note) => {
-        this.alertService.success('Succesfully submitted note', 4000);
-        this.scrollDown();
-        // TODO: maybe we do export a tutorNoteCount? but then we dont know for which tasks they are for
-        // TODO: itll only be helpful in the header notifications icon...
-        // TODO: otherwise, we could just load all the notes for a mentors mentee when loading the audt page?
-
-        // this.project.staffNoteCount++;
-        this.replyingToNote = null;
-        this.tutorNoteService.updateTutorNoteReplies(this.unitRole?.tutorNotesCache.currentValues);
-      },
-      error: (error) => {
-        this.alertService.error(`Failed to create note: ${error}`, 4000);
-        this.noteText = noteText;
-      },
-    });
+    this.tutorNoteService
+      .addNote(this.unitRole, noteText, this.task, this.replyingToNote)
+      .subscribe({
+        next: (note) => {
+          this.alertService.success('Succesfully submitted note', 4000);
+          this.scrollDown();
+          // TODO: maybe we do export a tutorNoteCount? but then we dont know for which tasks they are for
+          // TODO: itll only be helpful in the header notifications icon...
+          // TODO: otherwise, we could just load all the notes for a mentors mentee when loading the audt page?
+
+          // this.project.staffNoteCount++;
+          this.replyingToNote = null;
+          this.tutorNoteService.updateTutorNoteReplies(
+            this.unitRole?.tutorNotesCache.currentValues,
+          );
+        },
+        error: (error) => {
+          this.alertService.error(`Failed to create note: ${error}`, 4000);
+          this.noteText = noteText;
+        },
+      });
   }
 
   public updateNote() {
@@ -164,4 +171,44 @@ export class TutorNotesComponent implements OnInit {
       setTimeout(() => el.classList.remove('flash-highlight'), 1000);
     }
   }
+
+  public selectedTaskDefinitions: Map<string, boolean> = new Map<string, boolean>();
+
+  public get filteredNotes() {
+    const selected = this.selectedTaskDefinitions;
+    const allSelected = selected.size === 0 || selected.get('all');
+
+    return (
+      this.unitRole?.tutorNotesCache?.currentValues?.filter((note) => {
+        const abbr = note.taskDefinition?.abbreviation;
+        // if (!abbr) return false; // skip notes without taskDefinition
+        if (allSelected) return true;
+        return selected.get(abbr);
+      }) ?? []
+    );
+  }
+
+  toggleSelection(option: string) {
+    if (this.selectedTaskDefinitions.get(option)) {
+      this.selectedTaskDefinitions.set(option, false);
+    } else {
+      this.selectedTaskDefinitions.set(option, true);
+    }
+  }
+
+  public get taskDefinitionFilters() {
+    return (
+      this.unitRole.tutorNotesCache.currentValues
+        .map((note) => note.taskDefinition?.abbreviation)
+        .filter(Boolean) ?? []
+    );
+  }
+
+  openProject(event: Event, note: TutorNote) {
+    event.stopPropagation();
+    const link = document.createElement('a');
+    link.href = `/projects/${note.project.id}/dashboard/${note.taskDefinition.abbreviation}?tutor=true`;
+    link.target = '_blank';
+    link.click();
+  }
 }

From 1e92d8a675bfb3352d48de838b0a6e882a06a7ac Mon Sep 17 00:00:00 2001
From: b0ink <40929320+b0ink@users.noreply.github.com>
Date: Tue, 13 Jan 2026 15:01:52 +1100
Subject: [PATCH 07/38] refactor: mark tutor notes as read

---
 src/app/api/models/tutor-note.ts              |  7 +++++
 src/app/api/models/unit-role.ts               |  1 +
 src/app/api/services/tutor-note.service.ts    | 26 +++++++++++++++++++
 src/app/api/services/unit-role.service.ts     |  1 +
 src/app/common/header/header.component.html   |  8 ++++++
 .../tutor-notes/tutor-notes.component.html    | 12 ++++++++-
 .../tutor-notes/tutor-notes.component.ts      | 23 +++++++++++++---
 7 files changed, 74 insertions(+), 4 deletions(-)

diff --git a/src/app/api/models/tutor-note.ts b/src/app/api/models/tutor-note.ts
index 3eb16acae3..7ba6e16867 100644
--- a/src/app/api/models/tutor-note.ts
+++ b/src/app/api/models/tutor-note.ts
@@ -17,6 +17,8 @@ export class TutorNote extends Entity {
   note: string;
   replyTo?: TutorNote;
   replyToId: number;
+  readByUnitRole: boolean;
+
   createdAt: Date;
   updatedAt: Date;
 
@@ -29,6 +31,11 @@ export class TutorNote extends Entity {
     }
   }
 
+  public get noteIsForMe(): boolean {
+    const userService: UserService = AppInjector.get(UserService);
+    return this.unitRole.user.id === userService.currentUser.id;
+  }
+
   public get authorIsMe(): boolean {
     const userService: UserService = AppInjector.get(UserService);
     return this.user.id === userService.currentUser.id;
diff --git a/src/app/api/models/unit-role.ts b/src/app/api/models/unit-role.ts
index 982f4fd600..3fb4df72b5 100644
--- a/src/app/api/models/unit-role.ts
+++ b/src/app/api/models/unit-role.ts
@@ -14,6 +14,7 @@ export class UnitRole extends Entity {
   observerOnly: boolean;
   // mentor: UnitRole;
   mentorId: number;
+  tutorNoteCount: number;
 
   public readonly tutorNotesCache: EntityCache<TutorNote> = new EntityCache<TutorNote>();
 
diff --git a/src/app/api/services/tutor-note.service.ts b/src/app/api/services/tutor-note.service.ts
index a78db5dd01..b3bc8140c3 100644
--- a/src/app/api/services/tutor-note.service.ts
+++ b/src/app/api/services/tutor-note.service.ts
@@ -11,6 +11,8 @@ export class TutorNoteService extends CachedEntityService<TutorNote> {
   // public readonly staffNoteAdded$: EventEmitter<StaffNote> = new EventEmitter();
 
   protected readonly endpointFormat = 'unit_roles/:unitRoleId:/tutor_notes/:id:';
+  protected readonly markAsReadEndpointFormat =
+    'unit_roles/:unitRoleId:/tutor_notes/:id:/mark_as_read';
 
   constructor(
     httpClient: HttpClient,
@@ -50,6 +52,7 @@ export class TutorNoteService extends CachedEntityService<TutorNote> {
           return project;
         },
       },
+      'readByUnitRole',
     );
 
     this.mapping.addJsonKey('note', 'createdAt', 'updatedAt');
@@ -127,6 +130,29 @@ export class TutorNoteService extends CachedEntityService<TutorNote> {
     );
   }
 
+  public markAsRead(unitRole: UnitRole, note: TutorNote): Observable<boolean> {
+    const pathId = {
+      unitRoleId: unitRole.id,
+      id: note.id,
+    };
+
+    const opts: RequestOptions<TutorNote> = {endpointFormat: this.markAsReadEndpointFormat};
+    opts.cache = unitRole.tutorNotesCache;
+    opts.constructorParams = unitRole;
+
+    return this.put<boolean>(pathId, opts).pipe(
+      tap((response: boolean) => {
+        if (response) {
+          note.readByUnitRole = true;
+          unitRole.tutorNoteCount--;
+          // unitRole.tutorNoteCount = unitRole.tutorNotesCache.currentValues.filter(
+          //   (note) => !note.readByUnitRole,
+          // ).length;
+        }
+      }),
+    );
+  }
+
   public loadTutorNotes(unitRole: UnitRole, useFetch: boolean = false): Observable<TutorNote[]> {
     const options: RequestOptions<TutorNote> = {
       endpointFormat: this.endpointFormat,
diff --git a/src/app/api/services/unit-role.service.ts b/src/app/api/services/unit-role.service.ts
index 00102dd37a..e4aca28602 100644
--- a/src/app/api/services/unit-role.service.ts
+++ b/src/app/api/services/unit-role.service.ts
@@ -64,6 +64,7 @@ export class UnitRoleService extends CachedEntityService<UnitRole> {
       },
       'observerOnly',
       'mentorId',
+      'tutorNoteCount',
     );
 
     this.mapping.addJsonKey('roleId', 'userId', 'unitId', 'role', 'observerOnly', 'mentorId');
diff --git a/src/app/common/header/header.component.html b/src/app/common/header/header.component.html
index 96706776c9..ab98204bd1 100644
--- a/src/app/common/header/header.component.html
+++ b/src/app/common/header/header.component.html
@@ -45,6 +45,14 @@
         <mat-icon>cloud_sync_outline</mat-icon>
       </button>
     }
+    @if (currentUnit && currentUnitRole && currentUnitRole.tutorNoteCount) {
+      <!-- TODO: Replace this with a more standardised notification system -->
+      <button mat-icon-button (click)="(console.log)" matTooltip="Tutor Notes">
+        <mat-icon [matBadge]="currentUnitRole.tutorNoteCount" matBadgeSize="small"
+          >feedback</mat-icon
+        >
+      </button>
+    }
     @if (currentUnit) {
       <button mat-icon-button (click)="showMyQr()">
         <mat-icon>qr_code_scanner</mat-icon>
diff --git a/src/app/projects/states/tutor-notes/tutor-notes.component.html b/src/app/projects/states/tutor-notes/tutor-notes.component.html
index 529581a4fe..71dc1eaa25 100644
--- a/src/app/projects/states/tutor-notes/tutor-notes.component.html
+++ b/src/app/projects/states/tutor-notes/tutor-notes.component.html
@@ -37,7 +37,17 @@
           (mouseover)="hoveredNoteId = note.id"
           (mouseout)="hoveredNoteId = null"
         >
-          <div [hidden]="hoveredNoteId !== note.id" class="flex absolute right-5 bottom-5">
+          <div
+            [hidden]="hoveredNoteId !== note.id"
+            class="flex items-center absolute right-5 bottom-5"
+          >
+            @if (!note.readByUnitRole) {
+              @if (note.noteIsForMe) {
+                <button mat-button color="primary" (click)="markAsRead(note)">Mark as Read</button>
+              }
+            } @else {
+              <div class="text-gray-400 italic px-5">Read by tutor</div>
+            }
             @if (note.authorIsMe) {
               <mat-icon type="button" (click)="editNote(note)">edit</mat-icon>
             }
diff --git a/src/app/projects/states/tutor-notes/tutor-notes.component.ts b/src/app/projects/states/tutor-notes/tutor-notes.component.ts
index 2c98ee217c..44edae3534 100644
--- a/src/app/projects/states/tutor-notes/tutor-notes.component.ts
+++ b/src/app/projects/states/tutor-notes/tutor-notes.component.ts
@@ -121,6 +121,21 @@ export class TutorNotesComponent implements OnInit {
     });
   }
 
+  public markAsRead(note: TutorNote) {
+    this.tutorNoteService.markAsRead(this.unitRole, note).subscribe({
+      next: (response) => {
+        if (response) {
+          this.alertService.success(`Marked note as read`, 3000);
+        } else {
+          this.alertService.error(`Failed to mark as read`, 6000);
+        }
+      },
+      error: (error) => {
+        this.alertService.error(`Failed to mark as read: ${error}`, 6000);
+      },
+    });
+  }
+
   public deleteNote(note: TutorNote) {
     this.confirmationModalService.show(
       'Delete note',
@@ -197,11 +212,13 @@ export class TutorNotesComponent implements OnInit {
   }
 
   public get taskDefinitionFilters() {
-    return (
+    const abbrs =
       this.unitRole.tutorNotesCache.currentValues
         .map((note) => note.taskDefinition?.abbreviation)
-        .filter(Boolean) ?? []
-    );
+        .filter(Boolean) ?? [];
+
+    // Remove duplicates
+    return Array.from(new Set(abbrs));
   }
 
   openProject(event: Event, note: TutorNote) {

From 1d60f44a10387a5a55c227420a4b57ca13f679ef Mon Sep 17 00:00:00 2001
From: b0ink <40929320+b0ink@users.noreply.github.com>
Date: Tue, 13 Jan 2026 16:01:59 +1100
Subject: [PATCH 08/38] refactor: view tutor notes via modal

---
 src/app/common/header/header.component.html   |  2 +-
 src/app/common/header/header.component.ts     |  6 +++
 .../tutor-notes-modal.component.html          |  1 +
 .../tutor-notes-modal.component.scss          |  0
 .../tutor-notes-modal.component.ts            | 22 +++++++++++
 .../tutor-notes-modal.service.ts              | 33 ++++++++++++++++
 src/app/doubtfire-angular.module.ts           |  2 +
 .../tutor-notes-view.component.html           |  2 +-
 .../tutor-notes-view.component.ts             | 29 ++++++++------
 .../task-dashboard.component.html             |  2 +-
 .../tutor-notes/tutor-notes.component.ts      | 38 +++++++++++++------
 11 files changed, 111 insertions(+), 26 deletions(-)
 create mode 100644 src/app/common/modals/tutor-notes-modal/tutor-notes-modal.component.html
 create mode 100644 src/app/common/modals/tutor-notes-modal/tutor-notes-modal.component.scss
 create mode 100644 src/app/common/modals/tutor-notes-modal/tutor-notes-modal.component.ts
 create mode 100644 src/app/common/modals/tutor-notes-modal/tutor-notes-modal.service.ts

diff --git a/src/app/common/header/header.component.html b/src/app/common/header/header.component.html
index ab98204bd1..7b494c2198 100644
--- a/src/app/common/header/header.component.html
+++ b/src/app/common/header/header.component.html
@@ -47,7 +47,7 @@
     }
     @if (currentUnit && currentUnitRole && currentUnitRole.tutorNoteCount) {
       <!-- TODO: Replace this with a more standardised notification system -->
-      <button mat-icon-button (click)="(console.log)" matTooltip="Tutor Notes">
+      <button mat-icon-button (click)="openTutorNotes()" matTooltip="Tutor Notes">
         <mat-icon [matBadge]="currentUnitRole.tutorNoteCount" matBadgeSize="small"
           >feedback</mat-icon
         >
diff --git a/src/app/common/header/header.component.ts b/src/app/common/header/header.component.ts
index 4601057b15..c2041fb83a 100644
--- a/src/app/common/header/header.component.ts
+++ b/src/app/common/header/header.component.ts
@@ -13,6 +13,7 @@ import {SidekiqJobEntry, SidekiqJobService} from 'src/app/api/services/sidekiq-j
 import {SidekiqJobsModalService} from '../modals/sidekiq-jobs-modal/sidekiq-jobs-modal.service';
 import {QrModalService} from '../modals/qr-modal/qr-modal.service';
 import {StateService} from '@uirouter/core';
+import {TutorNotesModalService} from '../modals/tutor-notes-modal/tutor-notes-modal.service';
 
 @Component({
   selector: 'app-header',
@@ -58,6 +59,7 @@ export class HeaderComponent implements OnInit, OnDestroy {
     private sidekiqJobsModalService: SidekiqJobsModalService,
     private qrModalService: QrModalService,
     private stateService: StateService,
+    private tutorNotesModal: TutorNotesModalService,
   ) {}
 
   ngOnInit(): void {
@@ -210,4 +212,8 @@ export class HeaderComponent implements OnInit, OnDestroy {
   get currentUser(): User {
     return this.userService.currentUser;
   }
+
+  openTutorNotes() {
+    this.tutorNotesModal.show(null, this.currentUnitRole);
+  }
 }
diff --git a/src/app/common/modals/tutor-notes-modal/tutor-notes-modal.component.html b/src/app/common/modals/tutor-notes-modal/tutor-notes-modal.component.html
new file mode 100644
index 0000000000..ac97e33fe5
--- /dev/null
+++ b/src/app/common/modals/tutor-notes-modal/tutor-notes-modal.component.html
@@ -0,0 +1 @@
+<f-tutor-notes-view [task]="task" [unitRole]="unitRole"></f-tutor-notes-view>
diff --git a/src/app/common/modals/tutor-notes-modal/tutor-notes-modal.component.scss b/src/app/common/modals/tutor-notes-modal/tutor-notes-modal.component.scss
new file mode 100644
index 0000000000..e69de29bb2
diff --git a/src/app/common/modals/tutor-notes-modal/tutor-notes-modal.component.ts b/src/app/common/modals/tutor-notes-modal/tutor-notes-modal.component.ts
new file mode 100644
index 0000000000..70cf2c78af
--- /dev/null
+++ b/src/app/common/modals/tutor-notes-modal/tutor-notes-modal.component.ts
@@ -0,0 +1,22 @@
+import {Component, Inject, OnInit} from '@angular/core';
+import {MAT_DIALOG_DATA} from '@angular/material/dialog';
+import {Task} from 'src/app/api/models/task';
+import {UnitRole} from 'src/app/api/models/unit-role';
+import {TutorNotesModalData} from './tutor-notes-modal.service';
+
+@Component({
+  selector: 'f-tutor-notes-modal',
+  templateUrl: './tutor-notes-modal.component.html',
+  styleUrl: './tutor-notes-modal.component.scss',
+})
+export class TutorNotesModalComponent implements OnInit {
+  constructor(@Inject(MAT_DIALOG_DATA) public data: TutorNotesModalData) {}
+
+  task?: Task;
+  unitRole?: UnitRole;
+
+  ngOnInit() {
+    this.task = this.data.task;
+    this.unitRole = this.data.unitRole;
+  }
+}
diff --git a/src/app/common/modals/tutor-notes-modal/tutor-notes-modal.service.ts b/src/app/common/modals/tutor-notes-modal/tutor-notes-modal.service.ts
new file mode 100644
index 0000000000..152821d97c
--- /dev/null
+++ b/src/app/common/modals/tutor-notes-modal/tutor-notes-modal.service.ts
@@ -0,0 +1,33 @@
+import {Injectable} from '@angular/core';
+import {MatDialog} from '@angular/material/dialog';
+import {Task} from 'src/app/api/models/task';
+import {UnitRole} from 'src/app/api/models/unit-role';
+import {TutorNotesModalComponent} from './tutor-notes-modal.component';
+
+export interface TutorNotesModalData {
+  task?: Task;
+  unitRole?: UnitRole;
+}
+
+@Injectable({
+  providedIn: 'root',
+})
+export class TutorNotesModalService {
+  constructor(public dialog: MatDialog) {}
+
+  public show(task?: Task, unitRole?: UnitRole) {
+    const _dialogRef = this.dialog.open<TutorNotesModalComponent, TutorNotesModalData>(
+      TutorNotesModalComponent,
+      {
+        data: {
+          task,
+          unitRole,
+        },
+        width: '100%',
+        height: '90vh',
+        maxWidth: '900px',
+        panelClass: 'overflow-y-auto',
+      },
+    );
+  }
+}
diff --git a/src/app/doubtfire-angular.module.ts b/src/app/doubtfire-angular.module.ts
index 26841bbd77..7367495f7c 100644
--- a/src/app/doubtfire-angular.module.ts
+++ b/src/app/doubtfire-angular.module.ts
@@ -316,6 +316,7 @@ import {TutorNotesComponent} from './projects/states/tutor-notes/tutor-notes.com
 import {TutorNotesViewComponent} from './projects/states/dashboard/directives/task-dashboard/directives/tutor-notes-view/tutor-notes-view.component';
 import {TutorNoteService} from './api/services/tutor-note.service';
 import {ModerationComponent} from './units/states/tasks/inbox/directives/moderation/moderation.component';
+import {TutorNotesModalComponent} from './common/modals/tutor-notes-modal/tutor-notes-modal.component';
 
 // See https://stackoverflow.com/questions/55721254/how-to-change-mat-datepicker-date-format-to-dd-mm-yyyy-in-simplest-way/58189036#58189036
 const MY_DATE_FORMAT = {
@@ -507,6 +508,7 @@ const GANTT_CHART_CONFIG = {
     TutorNotesComponent,
     TutorNotesViewComponent,
     ModerationComponent,
+    TutorNotesModalComponent,
   ],
   providers: [
     // Services we provide
diff --git a/src/app/projects/states/dashboard/directives/task-dashboard/directives/tutor-notes-view/tutor-notes-view.component.html b/src/app/projects/states/dashboard/directives/task-dashboard/directives/tutor-notes-view/tutor-notes-view.component.html
index fb240c9b45..71ea152187 100644
--- a/src/app/projects/states/dashboard/directives/task-dashboard/directives/tutor-notes-view/tutor-notes-view.component.html
+++ b/src/app/projects/states/dashboard/directives/task-dashboard/directives/tutor-notes-view/tutor-notes-view.component.html
@@ -16,5 +16,5 @@ <h1 class="mat-headline m-0">Tutor Notes for {{ unitRole?.user?.name }}</h1>
     {{ unitRole?.user?.name }} and other convenors.
   </p>
 
-  <f-tutor-notes [task]="task"></f-tutor-notes>
+  <f-tutor-notes [task]="task" [unitRole]="unitRole"></f-tutor-notes>
 </div>
diff --git a/src/app/projects/states/dashboard/directives/task-dashboard/directives/tutor-notes-view/tutor-notes-view.component.ts b/src/app/projects/states/dashboard/directives/task-dashboard/directives/tutor-notes-view/tutor-notes-view.component.ts
index 99c0ea1297..25c88f643d 100644
--- a/src/app/projects/states/dashboard/directives/task-dashboard/directives/tutor-notes-view/tutor-notes-view.component.ts
+++ b/src/app/projects/states/dashboard/directives/task-dashboard/directives/tutor-notes-view/tutor-notes-view.component.ts
@@ -1,21 +1,28 @@
-import {Component, Input} from '@angular/core';
+import {Component, Inject, Input, OnInit} from '@angular/core';
+import {MAT_DIALOG_DATA} from '@angular/material/dialog';
+import {UnitRole} from 'src/app/api/models/unit-role';
+import {QrModalData} from 'src/app/common/modals/qr-modal/qr-modal.service';
 
 @Component({
   selector: 'f-tutor-notes-view',
   templateUrl: './tutor-notes-view.component.html',
   styleUrls: ['./tutor-notes-view.component.scss'],
 })
-export class TutorNotesViewComponent {
+export class TutorNotesViewComponent implements OnInit {
   @Input() task?;
-  // @Input() project;
-  public get unitRole() {
-    const enrolments = this.task.project.tutorialEnrolmentsCache.currentValues.filter(
-      (t) => t.tutorialStream.name === this.task.definition.tutorialStream.name,
-    );
-    // TODO: is checking for just the one tutorial enrolment correct? should be..
-    if (enrolments.length === 1) {
-      const user = enrolments[0].tutor;
-      return this.task.unit.staff.find((ur) => ur.user.id === user.id);
+
+  @Input() unitRole: UnitRole;
+
+  ngOnInit(): void {
+    if (this.task && !this.unitRole) {
+      const enrolments = this.task.project.tutorialEnrolmentsCache.currentValues.filter(
+        (t) => t.tutorialStream.name === this.task.definition.tutorialStream.name,
+      );
+      // TODO: is checking for just the one tutorial enrolment correct? should be..
+      if (enrolments.length === 1) {
+        const user = enrolments[0].tutor;
+        this.unitRole = this.task.unit.staff.find((ur) => ur.user.id === user.id);
+      }
     }
   }
 }
diff --git a/src/app/projects/states/dashboard/directives/task-dashboard/task-dashboard.component.html b/src/app/projects/states/dashboard/directives/task-dashboard/task-dashboard.component.html
index 7d9aa48482..931c569b6e 100644
--- a/src/app/projects/states/dashboard/directives/task-dashboard/task-dashboard.component.html
+++ b/src/app/projects/states/dashboard/directives/task-dashboard/task-dashboard.component.html
@@ -25,7 +25,7 @@
     </ng-template>
 
     <ng-template [ngSwitchCase]="DashboardViews.tutor_notes">
-      <f-tutor-notes-view [task]="task"></f-tutor-notes-view>
+      <f-tutor-notes-view [task]="task" [unitRole]="unitRole"></f-tutor-notes-view>
     </ng-template>
 
     <ng-template [ngSwitchCase]="DashboardViews.discussion_prompts">
diff --git a/src/app/projects/states/tutor-notes/tutor-notes.component.ts b/src/app/projects/states/tutor-notes/tutor-notes.component.ts
index 44edae3534..fa7b038a2d 100644
--- a/src/app/projects/states/tutor-notes/tutor-notes.component.ts
+++ b/src/app/projects/states/tutor-notes/tutor-notes.component.ts
@@ -1,5 +1,5 @@
 import {Component, ElementRef, Input, OnInit, ViewChild} from '@angular/core';
-import {Task, UserService} from 'src/app/api/models/doubtfire-model';
+import {Task, UnitRole, UserService} from 'src/app/api/models/doubtfire-model';
 import {TutorNote} from 'src/app/api/models/tutor-note';
 import {TutorNoteService} from 'src/app/api/services/tutor-note.service';
 import {ConfirmationModalService} from 'src/app/common/modals/confirmation-modal/confirmation-modal.service';
@@ -14,23 +14,25 @@ export class TutorNotesComponent implements OnInit {
   @ViewChild('staffNotesContainer') staffNotesContainer!: ElementRef;
   @ViewChild('staffNoteEditor', {static: false}) staffNoteEditor!: ElementRef<HTMLTextAreaElement>;
 
-  // @Input() unitRole: UnitRole;
+  @Input() unitRole: UnitRole;
   @Input() task: Task;
+
+
   // @Input() project: Project;
 
   // TODO: allow unitRole Input(), but if its nil, we need to set it during OnInit
   // TODO: otherwise, if task is null and unitRole exists (When current users access their own tutor notes, here we can load tutor notes with a unitRole and no task)
 
-  public get unitRole() {
-    const enrolments = this.task.project.tutorialEnrolmentsCache.currentValues.filter(
-      (t) => t.tutorialStream.name === this.task.definition.tutorialStream.name,
-    );
-    // TODO: is checking for just the one tutorial enrolment correct? should be..
-    if (enrolments.length === 1) {
-      const user = enrolments[0].tutor;
-      return this.task.unit.staff.find((ur) => ur.user.id === user.id);
-    }
-  }
+  // public get unitRole() {
+  //   const enrolments = this.task.project.tutorialEnrolmentsCache.currentValues.filter(
+  //     (t) => t.tutorialStream.name === this.task.definition.tutorialStream.name,
+  //   );
+  //   // TODO: is checking for just the one tutorial enrolment correct? should be..
+  //   if (enrolments.length === 1) {
+  //     const user = enrolments[0].tutor;
+  //     return this.task.unit.staff.find((ur) => ur.user.id === user.id);
+  //   }
+  // }
 
   loadingStaffNotes: boolean = true;
 
@@ -50,6 +52,18 @@ export class TutorNotesComponent implements OnInit {
     private confirmationModalService: ConfirmationModalService,
   ) {}
   ngOnInit(): void {
+    console.log(this.task, this.unitRole);
+    if (this.task && !this.unitRole) {
+      const enrolments = this.task.project.tutorialEnrolmentsCache.currentValues.filter(
+        (t) => t.tutorialStream.name === this.task.definition.tutorialStream.name,
+      );
+      // TODO: is checking for just the one tutorial enrolment correct? should be..
+      if (enrolments.length === 1) {
+        const user = enrolments[0].tutor;
+        this.unitRole = this.task.unit.staff.find((ur) => ur.user.id === user.id);
+      }
+    }
+
     this.loadingStaffNotes = true;
     this.tutorNoteService.loadTutorNotes(this.unitRole).subscribe((notes) => {
       this.loadingStaffNotes = false;

From 4b0239ebfa5131fd23eeb9565dfb741b232e4430 Mon Sep 17 00:00:00 2001
From: b0ink <40929320+b0ink@users.noreply.github.com>
Date: Mon, 2 Feb 2026 16:50:06 +1100
Subject: [PATCH 09/38] fix: remove task definition filters for moderation

---
 src/app/units/states/tasks/moderation/moderation.coffee | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/src/app/units/states/tasks/moderation/moderation.coffee b/src/app/units/states/tasks/moderation/moderation.coffee
index cc61515008..4a483fd314 100644
--- a/src/app/units/states/tasks/moderation/moderation.coffee
+++ b/src/app/units/states/tasks/moderation/moderation.coffee
@@ -26,5 +26,8 @@ angular.module('doubtfire.units.states.tasks.moderation', [
   $scope.showSearchOptions = false
   $scope.filters = {
     tutorialIdSelected: 'all'
+    taskDefinition: null
+    taskDefinitionIdSelected: null
   }
+  $scope.taskData.taskDefMode = false
 )

From 3ab7cdcad5eb8d2f83f86d42f4ac7bd5607bde63 Mon Sep 17 00:00:00 2001
From: b0ink <40929320+b0ink@users.noreply.github.com>
Date: Mon, 2 Feb 2026 17:25:06 +1100
Subject: [PATCH 10/38] chore: add dividers for moderation buttons

---
 src/app/common/footer/footer.component.html | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/app/common/footer/footer.component.html b/src/app/common/footer/footer.component.html
index 71b483c52f..cea9eb8d70 100644
--- a/src/app/common/footer/footer.component.html
+++ b/src/app/common/footer/footer.component.html
@@ -107,7 +107,7 @@
 
         <mat-divider [vertical]="true"></mat-divider>
 
-        <div class="w-[120px] flex justify-start">
+        <div class="flex justify-start">
           <button
             [disabled]="selectedTask === undefined"
             mat-icon-button
@@ -123,7 +123,9 @@
       </span>
 
       @if (viewType === 'moderation' && selectedTask) {
+        <mat-divider class="ml-10" [vertical]="true"></mat-divider>
         <f-moderation [task]="selectedTask"></f-moderation>
+        <mat-divider class="mr-10" [vertical]="true"></mat-divider>
       }
     </div>
 

From b167de973d32363b41ce3b87079bb62679a286d2 Mon Sep 17 00:00:00 2001
From: b0ink <40929320+b0ink@users.noreply.github.com>
Date: Tue, 3 Feb 2026 11:26:21 +1100
Subject: [PATCH 11/38] refactor: add tooltip

---
 src/app/projects/states/tutor-notes/tutor-notes.component.html | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/app/projects/states/tutor-notes/tutor-notes.component.html b/src/app/projects/states/tutor-notes/tutor-notes.component.html
index 71dc1eaa25..8e22a43a6b 100644
--- a/src/app/projects/states/tutor-notes/tutor-notes.component.html
+++ b/src/app/projects/states/tutor-notes/tutor-notes.component.html
@@ -77,10 +77,11 @@
               >
             }
             @if (note.project) {
-              <!-- TODO: you should be able to click this chip to view the project and task definition -->
               <mat-chip
                 chipListSelectable
                 class="w-fit cursor-pointer"
+                matTooltip="Click to view student"
+                matTooltipPosition="above"
                 (click)="openProject($event, note)"
                 >{{ note.project?.student.name }}</mat-chip
               >

From 2da6c290adef7edec61a71795241e897e6a2ae88 Mon Sep 17 00:00:00 2001
From: b0ink <40929320+b0ink@users.noreply.github.com>
Date: Tue, 3 Feb 2026 12:10:49 +1100
Subject: [PATCH 12/38] refactor: clean up tasks tutor getter

---
 src/app/common/footer/footer.component.ts     | 19 +------------
 .../tutor-notes-view.component.ts             | 14 ++--------
 .../tutor-notes/tutor-notes.component.ts      | 27 +------------------
 3 files changed, 4 insertions(+), 56 deletions(-)

diff --git a/src/app/common/footer/footer.component.ts b/src/app/common/footer/footer.component.ts
index 7c562b3d33..d0d6c9d9ee 100644
--- a/src/app/common/footer/footer.component.ts
+++ b/src/app/common/footer/footer.component.ts
@@ -53,25 +53,9 @@ export class FooterComponent implements OnInit {
       (this.warningText?.nativeElement.getBoundingClientRect().width + totalPaddingOffset) / 2;
   }
 
-  public markingTutor(task: Task): UnitRole {
-    const enrolments = task.project.tutorialEnrolmentsCache.currentValues.filter(
-      (t) => t.tutorialStream.abbreviation === task.definition.tutorialStream.abbreviation,
-    );
-    // TODO: is checking for just the one tutorial enrolment correct? should be..
-    if (enrolments.length === 1) {
-      const user = enrolments[0].tutor;
-      return task.unit.staff.find((ur) => ur.user.id === user.id);
-    }
-    return null;
-  }
-
   public canAccessTutorNotes(task: Task): boolean {
-    // TODO: if the access is truthy because current user == marking tutor
-    // TODO: we should only reveal the icon if a note has been left by their mentor
-
-    const tutor = this.markingTutor(task);
+    const tutor = task.tutor;
     if (!tutor) {
-      console.log('tutor invalid!');
       return false;
     }
 
@@ -95,7 +79,6 @@ export class FooterComponent implements OnInit {
   }
 
   public viewTutorNotes() {
-    console.log(`viewing tutor notes for `, this.markingTutor(this.selectedTask));
     this.selectedTaskService.showTutorNotes();
   }
 
diff --git a/src/app/projects/states/dashboard/directives/task-dashboard/directives/tutor-notes-view/tutor-notes-view.component.ts b/src/app/projects/states/dashboard/directives/task-dashboard/directives/tutor-notes-view/tutor-notes-view.component.ts
index 25c88f643d..33ba1dd5a4 100644
--- a/src/app/projects/states/dashboard/directives/task-dashboard/directives/tutor-notes-view/tutor-notes-view.component.ts
+++ b/src/app/projects/states/dashboard/directives/task-dashboard/directives/tutor-notes-view/tutor-notes-view.component.ts
@@ -1,7 +1,5 @@
-import {Component, Inject, Input, OnInit} from '@angular/core';
-import {MAT_DIALOG_DATA} from '@angular/material/dialog';
+import {Component, Input, OnInit} from '@angular/core';
 import {UnitRole} from 'src/app/api/models/unit-role';
-import {QrModalData} from 'src/app/common/modals/qr-modal/qr-modal.service';
 
 @Component({
   selector: 'f-tutor-notes-view',
@@ -10,19 +8,11 @@ import {QrModalData} from 'src/app/common/modals/qr-modal/qr-modal.service';
 })
 export class TutorNotesViewComponent implements OnInit {
   @Input() task?;
-
   @Input() unitRole: UnitRole;
 
   ngOnInit(): void {
     if (this.task && !this.unitRole) {
-      const enrolments = this.task.project.tutorialEnrolmentsCache.currentValues.filter(
-        (t) => t.tutorialStream.name === this.task.definition.tutorialStream.name,
-      );
-      // TODO: is checking for just the one tutorial enrolment correct? should be..
-      if (enrolments.length === 1) {
-        const user = enrolments[0].tutor;
-        this.unitRole = this.task.unit.staff.find((ur) => ur.user.id === user.id);
-      }
+      this.unitRole = this.task.tutor;
     }
   }
 }
diff --git a/src/app/projects/states/tutor-notes/tutor-notes.component.ts b/src/app/projects/states/tutor-notes/tutor-notes.component.ts
index fa7b038a2d..cb09cddee8 100644
--- a/src/app/projects/states/tutor-notes/tutor-notes.component.ts
+++ b/src/app/projects/states/tutor-notes/tutor-notes.component.ts
@@ -17,23 +17,6 @@ export class TutorNotesComponent implements OnInit {
   @Input() unitRole: UnitRole;
   @Input() task: Task;
 
-
-  // @Input() project: Project;
-
-  // TODO: allow unitRole Input(), but if its nil, we need to set it during OnInit
-  // TODO: otherwise, if task is null and unitRole exists (When current users access their own tutor notes, here we can load tutor notes with a unitRole and no task)
-
-  // public get unitRole() {
-  //   const enrolments = this.task.project.tutorialEnrolmentsCache.currentValues.filter(
-  //     (t) => t.tutorialStream.name === this.task.definition.tutorialStream.name,
-  //   );
-  //   // TODO: is checking for just the one tutorial enrolment correct? should be..
-  //   if (enrolments.length === 1) {
-  //     const user = enrolments[0].tutor;
-  //     return this.task.unit.staff.find((ur) => ur.user.id === user.id);
-  //   }
-  // }
-
   loadingStaffNotes: boolean = true;
 
   noteText: string = '';
@@ -52,16 +35,8 @@ export class TutorNotesComponent implements OnInit {
     private confirmationModalService: ConfirmationModalService,
   ) {}
   ngOnInit(): void {
-    console.log(this.task, this.unitRole);
     if (this.task && !this.unitRole) {
-      const enrolments = this.task.project.tutorialEnrolmentsCache.currentValues.filter(
-        (t) => t.tutorialStream.name === this.task.definition.tutorialStream.name,
-      );
-      // TODO: is checking for just the one tutorial enrolment correct? should be..
-      if (enrolments.length === 1) {
-        const user = enrolments[0].tutor;
-        this.unitRole = this.task.unit.staff.find((ur) => ur.user.id === user.id);
-      }
+      this.unitRole = this.task.tutor;
     }
 
     this.loadingStaffNotes = true;

From 335378ae352405083295c69a4551233e4ebf955d Mon Sep 17 00:00:00 2001
From: b0ink <40929320+b0ink@users.noreply.github.com>
Date: Tue, 3 Feb 2026 12:27:26 +1100
Subject: [PATCH 13/38] refactor: rename vars to tutor notes

---
 .../tutor-notes/tutor-notes.component.html    |  8 ++---
 .../tutor-notes/tutor-notes.component.ts      | 29 ++++++++-----------
 2 files changed, 16 insertions(+), 21 deletions(-)

diff --git a/src/app/projects/states/tutor-notes/tutor-notes.component.html b/src/app/projects/states/tutor-notes/tutor-notes.component.html
index 8e22a43a6b..b587307e5c 100644
--- a/src/app/projects/states/tutor-notes/tutor-notes.component.html
+++ b/src/app/projects/states/tutor-notes/tutor-notes.component.html
@@ -4,8 +4,8 @@
       No staff notes for {{ project.student.preferredName }} {{ project.student.lastName }}
     </div>
   } -->
-  <div #staffNotesContainer class="flex-1 overflow-auto p-4">
-    @if (!loadingStaffNotes) {
+  <div #tutorNotesContainer class="flex-1 overflow-auto p-4">
+    @if (!loadingTutorNotes) {
       @for (note of filteredNotes; track note) {
         @if (note.replyToId) {
           <div
@@ -93,8 +93,8 @@
               <mat-form-field appearance="outline" class="flex items-center w-full">
                 <mat-label>Update Note</mat-label>
                 <textarea
-                  (input)="autoResizeStaffNoteEditor()"
-                  #staffNoteEditor
+                  (input)="autoResizeTutorNoteEditor()"
+                  #tutorNoteEditor
                   matInput
                   [(ngModel)]="editingNoteText"
                   class="resize-none w-full px-2 py-1 placeholder:px-2 placeholder:py-1 overflow-hidden"
diff --git a/src/app/projects/states/tutor-notes/tutor-notes.component.ts b/src/app/projects/states/tutor-notes/tutor-notes.component.ts
index cb09cddee8..6f83b31b3d 100644
--- a/src/app/projects/states/tutor-notes/tutor-notes.component.ts
+++ b/src/app/projects/states/tutor-notes/tutor-notes.component.ts
@@ -11,13 +11,13 @@ import {AlertService} from 'src/app/common/services/alert.service';
   styleUrl: './tutor-notes.component.scss',
 })
 export class TutorNotesComponent implements OnInit {
-  @ViewChild('staffNotesContainer') staffNotesContainer!: ElementRef;
-  @ViewChild('staffNoteEditor', {static: false}) staffNoteEditor!: ElementRef<HTMLTextAreaElement>;
+  @ViewChild('tutorNotesContainer') tutorNotesContainer!: ElementRef;
+  @ViewChild('tutorNoteEditor', {static: false}) tutorNoteEditor!: ElementRef<HTMLTextAreaElement>;
 
   @Input() unitRole: UnitRole;
   @Input() task: Task;
 
-  loadingStaffNotes: boolean = true;
+  loadingTutorNotes: boolean = true;
 
   noteText: string = '';
 
@@ -39,9 +39,9 @@ export class TutorNotesComponent implements OnInit {
       this.unitRole = this.task.tutor;
     }
 
-    this.loadingStaffNotes = true;
+    this.loadingTutorNotes = true;
     this.tutorNoteService.loadTutorNotes(this.unitRole).subscribe((notes) => {
-      this.loadingStaffNotes = false;
+      this.loadingTutorNotes = false;
       this.tutorNoteService.updateTutorNoteReplies(this.unitRole?.tutorNotesCache.currentValues);
       this.scrollDown();
     });
@@ -56,7 +56,7 @@ export class TutorNotesComponent implements OnInit {
 
   scrollDown() {
     setTimeout(() => {
-      const el = this.staffNotesContainer.nativeElement;
+      const el = this.tutorNotesContainer.nativeElement;
       el.scrollTop = el.scrollHeight;
     }, 50);
   }
@@ -72,14 +72,9 @@ export class TutorNotesComponent implements OnInit {
     this.tutorNoteService
       .addNote(this.unitRole, noteText, this.task, this.replyingToNote)
       .subscribe({
-        next: (note) => {
+        next: (_note) => {
           this.alertService.success('Succesfully submitted note', 4000);
           this.scrollDown();
-          // TODO: maybe we do export a tutorNoteCount? but then we dont know for which tasks they are for
-          // TODO: itll only be helpful in the header notifications icon...
-          // TODO: otherwise, we could just load all the notes for a mentors mentee when loading the audt page?
-
-          // this.project.staffNoteCount++;
           this.replyingToNote = null;
           this.tutorNoteService.updateTutorNoteReplies(
             this.unitRole?.tutorNotesCache.currentValues,
@@ -99,7 +94,7 @@ export class TutorNotesComponent implements OnInit {
     }
 
     this.tutorNoteService.updateNote(this.unitRole, this.editingNote, noteText).subscribe({
-      next: (note) => {
+      next: (_note) => {
         this.alertService.success('Succesfully updated note', 4000);
         this.editingNote = null;
         this.editingNoteText = '';
@@ -150,8 +145,8 @@ export class TutorNotesComponent implements OnInit {
     this.editingNote = note;
     this.editingNoteText = note.note;
     setTimeout(() => {
-      this.autoResizeStaffNoteEditor();
-      this.staffNoteEditor?.nativeElement.focus();
+      this.autoResizeTutorNoteEditor();
+      this.tutorNoteEditor?.nativeElement.focus();
     });
   }
 
@@ -160,8 +155,8 @@ export class TutorNotesComponent implements OnInit {
     this.editingNoteText = '';
   }
 
-  public autoResizeStaffNoteEditor() {
-    const el = this.staffNoteEditor.nativeElement;
+  public autoResizeTutorNoteEditor() {
+    const el = this.tutorNoteEditor.nativeElement;
     el.style.height = 'auto';
     el.offsetHeight;
     el.style.height = el.scrollHeight + 'px';

From bbe85379245f680ced516296a145ec86fb95f620 Mon Sep 17 00:00:00 2001
From: b0ink <40929320+b0ink@users.noreply.github.com>
Date: Wed, 4 Feb 2026 10:27:54 +1100
Subject: [PATCH 14/38] refactor: add typing

---
 .../directives/staff-task-list/staff-task-list.component.ts | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/src/app/units/states/tasks/inbox/directives/staff-task-list/staff-task-list.component.ts b/src/app/units/states/tasks/inbox/directives/staff-task-list/staff-task-list.component.ts
index 8be2d3a77e..7d853e2ce0 100644
--- a/src/app/units/states/tasks/inbox/directives/staff-task-list/staff-task-list.component.ts
+++ b/src/app/units/states/tasks/inbox/directives/staff-task-list/staff-task-list.component.ts
@@ -71,8 +71,10 @@ export class StaffTaskListComponent implements OnInit, OnChanges, OnDestroy {
 
   @Input() isNarrow: boolean;
 
+  @Input() viewType: 'inbox' | 'explorer' | 'moderation';
+
   userHasTutorials: boolean;
-  filteredTasks: any[] = null;
+  filteredTasks: Task[] = null;
 
   studentFilter: {
     id: number | string;
@@ -82,7 +84,7 @@ export class StaffTaskListComponent implements OnInit, OnChanges, OnDestroy {
     tutorial?: Tutorial;
   }[] = null;
 
-  tasks: any[] = null;
+  tasks: Task[] = null;
 
   // hasJplagReport: boolean = false;
 

From c34a12c4cc85fa05a4b658d803dccd37c2c0668c Mon Sep 17 00:00:00 2001
From: b0ink <40929320+b0ink@users.noreply.github.com>
Date: Wed, 4 Feb 2026 10:37:53 +1100
Subject: [PATCH 15/38] refactor: clean up unused code

---
 src/app/api/models/tutor-note.ts           | 10 +++-------
 src/app/api/models/unit-role.ts            |  6 +++---
 src/app/api/services/tutor-note.service.ts | 15 ++++-----------
 3 files changed, 10 insertions(+), 21 deletions(-)

diff --git a/src/app/api/models/tutor-note.ts b/src/app/api/models/tutor-note.ts
index 7ba6e16867..148d5fa9eb 100644
--- a/src/app/api/models/tutor-note.ts
+++ b/src/app/api/models/tutor-note.ts
@@ -1,14 +1,12 @@
 import {Entity} from 'ngx-entity-service';
-import {Project, Unit, User, UserService, Task, UnitRole, TaskDefinition} from './doubtfire-model';
 import {AppInjector} from 'src/app/app-injector';
 import {AlertService} from 'src/app/common/services/alert.service';
-import {StaffNoteService} from '../services/staff-note.service';
 import {TutorNoteService} from '../services/tutor-note.service';
+import {Project, Task, TaskDefinition, UnitRole, User, UserService} from './doubtfire-model';
 
 export class TutorNote extends Entity {
   id: number;
 
-  // project: Project;
   unitRole: UnitRole;
   task?: Task;
   taskDefinition?: TaskDefinition;
@@ -27,7 +25,7 @@ export class TutorNote extends Entity {
     if (data) {
       this.unitRole = data;
     } else {
-      console.error('Failed to get project');
+      console.error('Failed to get unit role');
     }
   }
 
@@ -46,10 +44,8 @@ export class TutorNote extends Entity {
     tutorNoteService
       .delete({unitRoleId: this.unitRole.id, id: this.id}, {cache: this.unitRole.tutorNotesCache})
       .subscribe({
-        next: (response: object) => {
+        next: () => {
           AppInjector.get(AlertService).error('Successfully deleted tutor note', 4000);
-          // this.project.staffNoteCount--;
-          // staffNoteService.updateStaffNoteReplies(this.project.staffNoteCache.currentValues);
         },
         error: (error) => {
           AppInjector.get(AlertService).error(error?.message || error || 'Unknown error', 2000);
diff --git a/src/app/api/models/unit-role.ts b/src/app/api/models/unit-role.ts
index 3fb4df72b5..837d1d8eb6 100644
--- a/src/app/api/models/unit-role.ts
+++ b/src/app/api/models/unit-role.ts
@@ -12,15 +12,15 @@ export class UnitRole extends Entity {
   user: User;
   unit: Unit;
   observerOnly: boolean;
-  // mentor: UnitRole;
   mentorId: number;
   tutorNoteCount: number;
 
-  public readonly tutorNotesCache: EntityCache<TutorNote> = new EntityCache<TutorNote>();
-
   public get mentor(): UnitRole {
     return this.unit?.staff.find((ur) => ur.id === this.mentorId);
   }
+
+  public readonly tutorNotesCache: EntityCache<TutorNote> = new EntityCache<TutorNote>();
+
   /**
    * The id for updated roles - but we need to move away from this to the role string...
    * @deprecated
diff --git a/src/app/api/services/tutor-note.service.ts b/src/app/api/services/tutor-note.service.ts
index b3bc8140c3..a1c85cbb30 100644
--- a/src/app/api/services/tutor-note.service.ts
+++ b/src/app/api/services/tutor-note.service.ts
@@ -8,8 +8,6 @@ import {TutorNote} from '../models/tutor-note';
 
 @Injectable()
 export class TutorNoteService extends CachedEntityService<TutorNote> {
-  // public readonly staffNoteAdded$: EventEmitter<StaffNote> = new EventEmitter();
-
   protected readonly endpointFormat = 'unit_roles/:unitRoleId:/tutor_notes/:id:';
   protected readonly markAsReadEndpointFormat =
     'unit_roles/:unitRoleId:/tutor_notes/:id:/mark_as_read';
@@ -29,16 +27,15 @@ export class TutorNoteService extends CachedEntityService<TutorNote> {
       'replyToId',
       {
         keys: ['user', 'user_id'],
-        toEntityFn: (data: object, key: string, tutorNote: TutorNote) => {
+        toEntityFn: (data: object, _key: string, tutorNote: TutorNote) => {
           const userRole = tutorNote.unitRole.unit.staff.find((s) => s.user.id === data['user_id']);
-          // const user = this.userService.cache.getOrCreate(data[key]?.id, userService, data[key]);
           // If the user is not a staff in the unit it will be null
           return userRole?.user;
         },
       },
       {
         keys: ['taskDefinition', 'task_definition_id'],
-        toEntityFn: (data: object, key: string, tutorNote: TutorNote) => {
+        toEntityFn: (data: object, _key: string, tutorNote: TutorNote) => {
           const taskDefinition = tutorNote.unitRole.unit.taskDefinitions.find(
             (td) => td.id === data['task_definition_id'],
           );
@@ -58,7 +55,7 @@ export class TutorNoteService extends CachedEntityService<TutorNote> {
     this.mapping.addJsonKey('note', 'createdAt', 'updatedAt');
   }
 
-  public createInstanceFrom(json: object, other?: UnitRole): TutorNote {
+  public createInstanceFrom(_json: object, other?: UnitRole): TutorNote {
     return new TutorNote(other);
   }
 
@@ -88,11 +85,7 @@ export class TutorNoteService extends CachedEntityService<TutorNote> {
     opts.body = body;
     opts.constructorParams = unitRole;
 
-    return this.create(pathId, opts).pipe(
-      tap((note: TutorNote) => {
-        // this.staffNoteAdded$.emit(note);
-      }),
-    );
+    return this.create(pathId, opts);
   }
 
   public updateTutorNoteReplies(tutorNotes: readonly TutorNote[]) {

From bbdd745afbcb624e438d5f00886dcd0909d3d3a1 Mon Sep 17 00:00:00 2001
From: b0ink <40929320+b0ink@users.noreply.github.com>
Date: Wed, 4 Feb 2026 10:41:18 +1100
Subject: [PATCH 16/38] fix: check for valid tutor

---
 src/app/common/footer/footer.component.html | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/app/common/footer/footer.component.html b/src/app/common/footer/footer.component.html
index cea9eb8d70..efd547190a 100644
--- a/src/app/common/footer/footer.component.html
+++ b/src/app/common/footer/footer.component.html
@@ -169,7 +169,7 @@
               mat-icon-button
               matTooltipPosition="above"
               class="button small-button"
-              [disabled]="!markingTutor(selectedTask)"
+              [disabled]="!selectedTask.tutor"
               (click)="viewTutorNotes()"
             >
               <mat-icon class="small-icon">local_library</mat-icon>

From eebfb368c73d64d38f00556fa597d409d8254c13 Mon Sep 17 00:00:00 2001
From: b0ink <40929320+b0ink@users.noreply.github.com>
Date: Mon, 9 Feb 2026 13:23:20 +1100
Subject: [PATCH 17/38] feat: init escalation request

---
 src/app/api/models/project.ts                       |  3 ++-
 src/app/api/models/task.ts                          |  9 +++++++++
 src/app/api/services/project.service.ts             |  1 +
 .../task-submission-card.component.html             |  5 ++++-
 .../task-submission-card.component.ts               | 13 +++++++++++++
 5 files changed, 29 insertions(+), 2 deletions(-)

diff --git a/src/app/api/models/project.ts b/src/app/api/models/project.ts
index 2afef9ffa2..6eed890270 100644
--- a/src/app/api/models/project.ts
+++ b/src/app/api/models/project.ts
@@ -43,7 +43,8 @@ export class Project extends Entity {
 
   public hasPortfolio: boolean;
   public portfolioStatus: number;
-  public portfolioFiles: { kind: string; name: string; idx: number }[];
+  public portfolioFiles: {kind: string; name: string; idx: number}[];
+  public escalationAttemptsRemaining: number;
 
   public taskStats: {
     key: TaskStatusEnum;
diff --git a/src/app/api/models/task.ts b/src/app/api/models/task.ts
index e8f71eed5c..c9d84836cb 100644
--- a/src/app/api/models/task.ts
+++ b/src/app/api/models/task.ts
@@ -1016,4 +1016,13 @@ export class Task extends Entity {
       },
     );
   }
+
+  public requestFeedbackReview(): Observable<boolean> {
+    const httpClient: HttpClient = AppInjector.get(HttpClient);
+    const url = `${AppInjector.get(DoubtfireConstants).API_URL}/projects/${
+      this.project.id
+    }/task_def_id/${this.definition.id}/feedback_review`;
+
+    return httpClient.post<boolean>(url, {});
+  }
 }
diff --git a/src/app/api/services/project.service.ts b/src/app/api/services/project.service.ts
index a897e9c1dd..f5573dafcb 100644
--- a/src/app/api/services/project.service.ts
+++ b/src/app/api/services/project.service.ts
@@ -207,6 +207,7 @@ export class ProjectService extends CachedEntityService<Project> {
           });
         },
       },
+      'escalationAttemptsRemaining',
     );
 
     this.mapping.addJsonKey(
diff --git a/src/app/projects/states/dashboard/directives/task-dashboard/directives/task-submission-card/task-submission-card.component.html b/src/app/projects/states/dashboard/directives/task-dashboard/directives/task-submission-card/task-submission-card.component.html
index ca9f850437..2569a4b7f5 100644
--- a/src/app/projects/states/dashboard/directives/task-dashboard/directives/task-submission-card/task-submission-card.component.html
+++ b/src/app/projects/states/dashboard/directives/task-dashboard/directives/task-submission-card/task-submission-card.component.html
@@ -51,7 +51,10 @@
       <button [hidden]="!canReuploadEvidence" mat-menu-item (click)="uploadAlternateFiles()">
         Upload alternate files
       </button>
-      <button [hidden]="!canRegeneratePdf" mat-menu-item (click)="regeneratePdf()">Regenerate PDFs</button>
+      <button [hidden]="!canRegeneratePdf" mat-menu-item (click)="regeneratePdf()">
+        Regenerate PDFs
+      </button>
+      <button mat-menu-item (click)="escalateFeedback()">Request feedback review</button>
     </mat-menu>
   </mat-card-actions>
 </mat-card>
diff --git a/src/app/projects/states/dashboard/directives/task-dashboard/directives/task-submission-card/task-submission-card.component.ts b/src/app/projects/states/dashboard/directives/task-dashboard/directives/task-submission-card/task-submission-card.component.ts
index 771b1ce53e..881c8bbe49 100644
--- a/src/app/projects/states/dashboard/directives/task-dashboard/directives/task-submission-card/task-submission-card.component.ts
+++ b/src/app/projects/states/dashboard/directives/task-dashboard/directives/task-submission-card/task-submission-card.component.ts
@@ -74,6 +74,19 @@ export class TaskSubmissionCardComponent implements OnChanges, OnInit {
     });
   }
 
+  escalateFeedback(): void {
+    // TODO: Open a modal
+    this.task.requestFeedbackReview().subscribe({
+      next: (response) => {
+        console.log(response);
+        this.alerts.success(`Requested feedback review for this task!`, 3000);
+      },
+      error: (error) => {
+        this.alerts.error(`An error occurred: ${error}`, 3000);
+      },
+    });
+  }
+
   downloadSubmission(): void {
     this.fileDownloader.downloadFile(this.urls.pdf, `${this.task.definition.abbreviation}.pdf`);
   }

From cd2614de0054480ae89bf14159cb631be8eb7caa Mon Sep 17 00:00:00 2001
From: b0ink <40929320+b0ink@users.noreply.github.com>
Date: Mon, 9 Feb 2026 13:23:54 +1100
Subject: [PATCH 18/38] chore: add moderation type field

---
 src/app/api/models/task.ts           | 2 ++
 src/app/api/services/task.service.ts | 1 +
 2 files changed, 3 insertions(+)

diff --git a/src/app/api/models/task.ts b/src/app/api/models/task.ts
index c9d84836cb..374fa13eae 100644
--- a/src/app/api/models/task.ts
+++ b/src/app/api/models/task.ts
@@ -46,6 +46,8 @@ export class Task extends Entity {
   numNewComments: number = 0;
   hasExtensions: boolean;
 
+  moderationType: 'random_sample' | 'escalation' | 'first_feedback';
+
   project: Project;
   definition: TaskDefinition;
 
diff --git a/src/app/api/services/task.service.ts b/src/app/api/services/task.service.ts
index 54d92d49c1..3fd9dc0532 100644
--- a/src/app/api/services/task.service.ts
+++ b/src/app/api/services/task.service.ts
@@ -98,6 +98,7 @@ export class TaskService extends CachedEntityService<Task> {
           });
         },
       },
+      'moderationType',
     );
 
     this.mapping.addJsonKey('qualityPts', 'grade', 'includeInPortfolio', 'trigger');

From 6623ee1227b56808ec9879e6edf913542e4d11a6 Mon Sep 17 00:00:00 2001
From: b0ink <40929320+b0ink@users.noreply.github.com>
Date: Mon, 9 Feb 2026 13:24:44 +1100
Subject: [PATCH 19/38] refactor: add alternate ui for escalated tasks

---
 .../moderation/moderation.component.html      | 92 ++++++++++++-------
 1 file changed, 59 insertions(+), 33 deletions(-)

diff --git a/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.html b/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.html
index 2260d2346a..37de64a432 100644
--- a/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.html
+++ b/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.html
@@ -1,36 +1,62 @@
 <!-- <div class="flex justify-center py-2">Tutor: {{ task.tutor?.user.name }}</div> -->
 <span class="center-buttons flex justify-around gap-2">
-  <button
-    [disabled]="task === undefined || isModerated(task)"
-    mat-icon-button
-    matTooltip="Show more from this tutor"
-    matTooltipPosition="above"
-    aria-label=""
-    class="button"
-    (click)="showMore()"
-  >
-    <mat-icon>flag</mat-icon>
-  </button>
-  <button
-    [disabled]="task === undefined || isModerated(task)"
-    mat-icon-button
-    matTooltip="Dismiss moderation task"
-    matTooltipPosition="above"
-    aria-label=""
-    class="button large-button"
-    (click)="dismiss()"
-  >
-    <mat-icon>archive</mat-icon>
-  </button>
-  <button
-    [disabled]="task === undefined || isModerated(task)"
-    mat-icon-button
-    matTooltip="Show less from this tutor"
-    matTooltipPosition="above"
-    aria-label=""
-    class="button"
-    (click)="showLess()"
-  >
-    <mat-icon>thumb_up</mat-icon>
-  </button>
+  @if (task.moderationType === 'random_sample' || task.moderationType === 'first_feedback') {
+    <button
+      [disabled]="task === undefined || isModerated(task)"
+      mat-icon-button
+      matTooltip="Show more from this tutor"
+      matTooltipPosition="above"
+      aria-label=""
+      class="button"
+      (click)="showMore()"
+    >
+      <mat-icon>flag</mat-icon>
+    </button>
+    <button
+      [disabled]="task === undefined || isModerated(task)"
+      mat-icon-button
+      matTooltip="Dismiss moderation task"
+      matTooltipPosition="above"
+      aria-label=""
+      class="button large-button"
+      (click)="dismiss()"
+    >
+      <mat-icon>archive</mat-icon>
+    </button>
+    <button
+      [disabled]="task === undefined || isModerated(task)"
+      mat-icon-button
+      matTooltip="Show less from this tutor"
+      matTooltipPosition="above"
+      aria-label=""
+      class="button"
+      (click)="showLess()"
+    >
+      <mat-icon>thumb_up</mat-icon>
+    </button>
+  } @else if (task.moderationType === 'escalation') {
+    <button
+      [disabled]="task === undefined || isModerated(task)"
+      mat-icon-button
+      matTooltip="Overruled"
+      matTooltipPosition="above"
+      aria-label=""
+      class="button"
+      (click)="showMore()"
+    >
+      <mat-icon>flag</mat-icon>
+    </button>
+
+    <button
+      [disabled]="task === undefined || isModerated(task)"
+      mat-icon-button
+      matTooltip="Upheld"
+      matTooltipPosition="above"
+      aria-label=""
+      class="button"
+      (click)="showLess()"
+    >
+      <mat-icon>thumb_up</mat-icon>
+    </button>
+  }
 </span>

From 8f7ae524177d31d46a9082fb92bd2e85683a1f77 Mon Sep 17 00:00:00 2001
From: b0ink <40929320+b0ink@users.noreply.github.com>
Date: Tue, 17 Feb 2026 11:09:43 +1100
Subject: [PATCH 20/38] chore: add moderation action type

---
 src/app/api/models/task.ts                    | 16 +++++++--
 src/app/doubtfire-angular.module.ts           |  2 ++
 .../moderation/moderation.component.html      |  6 ++--
 .../moderation/moderation.component.ts        | 33 +++++++++++++++----
 4 files changed, 45 insertions(+), 12 deletions(-)

diff --git a/src/app/api/models/task.ts b/src/app/api/models/task.ts
index 374fa13eae..6db7b17f5c 100644
--- a/src/app/api/models/task.ts
+++ b/src/app/api/models/task.ts
@@ -29,6 +29,18 @@ import {gradeTaskModal, uploadSubmissionModal} from 'src/app/ajs-upgraded-provid
 import {AlertService} from 'src/app/common/services/alert.service';
 import {MappingFunctions} from '../services/mapping-fn';
 
+export const FeedbackModerationAction = {
+  ShowMore: 'show_more',
+  ShowLess: 'show_less',
+  DismissOk: 'dismiss_ok',
+  DismissGood: 'dismiss_good',
+  Upheld: 'upheld',
+  Overturn: 'overturn',
+} as const;
+
+export type FeedbackModerationActionType =
+  (typeof FeedbackModerationAction)[keyof typeof FeedbackModerationAction];
+
 export class Task extends Entity {
   id: number;
 
@@ -997,7 +1009,7 @@ export class Task extends Entity {
     return false;
   }
 
-  public moderateFeedback(score: -1 | 0 | 1): Observable<boolean> {
+  public moderateFeedback(action: FeedbackModerationActionType): Observable<boolean> {
     const unitRoleService: UnitRoleService = AppInjector.get(UnitRoleService);
 
     const tutor = this.tutor;
@@ -1013,7 +1025,7 @@ export class Task extends Entity {
       {
         endpointFormat: '/unit_roles/:id:/moderation/:taskId:',
         body: {
-          score: score,
+          action,
         },
       },
     );
diff --git a/src/app/doubtfire-angular.module.ts b/src/app/doubtfire-angular.module.ts
index 5419d9206d..23c13a0e46 100644
--- a/src/app/doubtfire-angular.module.ts
+++ b/src/app/doubtfire-angular.module.ts
@@ -318,6 +318,7 @@ import {TutorNotesViewComponent} from './projects/states/dashboard/directives/ta
 import {TutorNoteService} from './api/services/tutor-note.service';
 import {ModerationComponent} from './units/states/tasks/inbox/directives/moderation/moderation.component';
 import {TutorNotesModalComponent} from './common/modals/tutor-notes-modal/tutor-notes-modal.component';
+import {FeedbackAppealModalComponent} from './tasks/modals/feedback-appeal-modal/feedback-appeal-modal.component';
 
 // See https://stackoverflow.com/questions/55721254/how-to-change-mat-datepicker-date-format-to-dd-mm-yyyy-in-simplest-way/58189036#58189036
 const MY_DATE_FORMAT = {
@@ -511,6 +512,7 @@ const GANTT_CHART_CONFIG = {
     TutorNotesViewComponent,
     ModerationComponent,
     TutorNotesModalComponent,
+    FeedbackAppealModalComponent,
   ],
   providers: [
     // Services we provide
diff --git a/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.html b/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.html
index 37de64a432..ed7e9be397 100644
--- a/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.html
+++ b/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.html
@@ -38,11 +38,11 @@
     <button
       [disabled]="task === undefined || isModerated(task)"
       mat-icon-button
-      matTooltip="Overruled"
+      matTooltip="Overturn"
       matTooltipPosition="above"
       aria-label=""
       class="button"
-      (click)="showMore()"
+      (click)="overturn()"
     >
       <mat-icon>flag</mat-icon>
     </button>
@@ -54,7 +54,7 @@
       matTooltipPosition="above"
       aria-label=""
       class="button"
-      (click)="showLess()"
+      (click)="upheld()"
     >
       <mat-icon>thumb_up</mat-icon>
     </button>
diff --git a/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.ts b/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.ts
index 7ae8b2745c..a61937cae6 100644
--- a/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.ts
+++ b/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.ts
@@ -1,5 +1,5 @@
 import {Component, Input} from '@angular/core';
-import {Task} from 'src/app/api/models/task';
+import {FeedbackModerationActionType, Task} from 'src/app/api/models/task';
 import {ConfirmationModalService} from 'src/app/common/modals/confirmation-modal/confirmation-modal.service';
 import {AlertService} from 'src/app/common/services/alert.service';
 
@@ -18,13 +18,32 @@ export class ModerationComponent {
     private confirmationModal: ConfirmationModalService,
   ) {}
 
+  overturn() {
+    this.confirmationModal.show(
+      'Overturn',
+      `There were concerns with feedback given to the student and you have changed the status of the task.`,
+      () => {
+        this.moderateTask('overturn');
+      },
+    );
+  }
+
+  upheld() {
+    this.confirmationModal.show(
+      'Upheld',
+      `The feedback given to the student was justified, and no changes are required.`,
+      () => {
+        this.moderateTask('upheld');
+      },
+    );
+  }
+
   showMore() {
-    console.log('bad!');
     this.confirmationModal.show(
       'Show more from this tutor',
       `There were concerns with feedback and you would like to see more tasks from this tutor. You will review this task again if further feedback is provided.`,
       () => {
-        this.moderateTask(-1);
+        this.moderateTask('show_more');
       },
     );
   }
@@ -34,7 +53,7 @@ export class ModerationComponent {
       'Show less from this tutor',
       `The feedback provided by the tutor is satisfactory and you would like to see less of their feedback for moderation. You won't see this task again.`,
       () => {
-        this.moderateTask(1);
+        this.moderateTask('show_less');
       },
     );
   }
@@ -44,13 +63,13 @@ export class ModerationComponent {
       'Dismiss from moderation',
       'This task will be removed from moderation and will not get a follow up. This will not affect how many tasks from this tutor are selected for moderation.',
       () => {
-        this.moderateTask(0);
+        this.moderateTask('dismiss_ok');
       },
     );
   }
 
-  private moderateTask(score: -1 | 0 | 1) {
-    this.task.moderateFeedback(score).subscribe({
+  private moderateTask(action: FeedbackModerationActionType) {
+    this.task.moderateFeedback(action).subscribe({
       next: (response) => {
         console.log(response);
         this.alertService.success(`Task moderated`, 2000);

From bffcfe3668f17668a685882aa55b10b4a0c34353 Mon Sep 17 00:00:00 2001
From: b0ink <40929320+b0ink@users.noreply.github.com>
Date: Tue, 17 Feb 2026 11:11:51 +1100
Subject: [PATCH 21/38] feat: feedback appeal modal

---
 .../task-submission-card.component.html       |  2 +-
 .../task-submission-card.component.ts         | 29 +++++---------
 .../feedback-appeal-modal.component.html      | 15 +++++++
 .../feedback-appeal-modal.component.scss      |  0
 .../feedback-appeal-modal.component.ts        | 40 +++++++++++++++++++
 .../feedback-appeal-modal.service.ts          | 29 ++++++++++++++
 6 files changed, 96 insertions(+), 19 deletions(-)
 create mode 100644 src/app/tasks/modals/feedback-appeal-modal/feedback-appeal-modal.component.html
 create mode 100644 src/app/tasks/modals/feedback-appeal-modal/feedback-appeal-modal.component.scss
 create mode 100644 src/app/tasks/modals/feedback-appeal-modal/feedback-appeal-modal.component.ts
 create mode 100644 src/app/tasks/modals/feedback-appeal-modal/feedback-appeal-modal.service.ts

diff --git a/src/app/projects/states/dashboard/directives/task-dashboard/directives/task-submission-card/task-submission-card.component.html b/src/app/projects/states/dashboard/directives/task-dashboard/directives/task-submission-card/task-submission-card.component.html
index 2569a4b7f5..528489783c 100644
--- a/src/app/projects/states/dashboard/directives/task-dashboard/directives/task-submission-card/task-submission-card.component.html
+++ b/src/app/projects/states/dashboard/directives/task-dashboard/directives/task-submission-card/task-submission-card.component.html
@@ -54,7 +54,7 @@
       <button [hidden]="!canRegeneratePdf" mat-menu-item (click)="regeneratePdf()">
         Regenerate PDFs
       </button>
-      <button mat-menu-item (click)="escalateFeedback()">Request feedback review</button>
+      <button mat-menu-item (click)="openFeedbackAppealModal()">Request feedback review</button>
     </mat-menu>
   </mat-card-actions>
 </mat-card>
diff --git a/src/app/projects/states/dashboard/directives/task-dashboard/directives/task-submission-card/task-submission-card.component.ts b/src/app/projects/states/dashboard/directives/task-dashboard/directives/task-submission-card/task-submission-card.component.ts
index 881c8bbe49..0c68db5717 100644
--- a/src/app/projects/states/dashboard/directives/task-dashboard/directives/task-submission-card/task-submission-card.component.ts
+++ b/src/app/projects/states/dashboard/directives/task-dashboard/directives/task-submission-card/task-submission-card.component.ts
@@ -1,9 +1,10 @@
-import { Component, Inject, Input, OnChanges, OnInit, SimpleChanges } from '@angular/core';
-import { uploadSubmissionModal } from 'src/app/ajs-upgraded-providers';
-import { Task } from 'src/app/api/models/task';
-import { TaskService } from 'src/app/api/services/task.service';
-import { FileDownloaderService } from 'src/app/common/file-downloader/file-downloader.service';
-import { AlertService } from 'src/app/common/services/alert.service';
+import {Component, Inject, Input, OnChanges, OnInit, SimpleChanges} from '@angular/core';
+import {uploadSubmissionModal} from 'src/app/ajs-upgraded-providers';
+import {Task} from 'src/app/api/models/task';
+import {TaskService} from 'src/app/api/services/task.service';
+import {FileDownloaderService} from 'src/app/common/file-downloader/file-downloader.service';
+import {AlertService} from 'src/app/common/services/alert.service';
+import {FeedbackAppealModalService} from 'src/app/tasks/modals/feedback-appeal-modal/feedback-appeal-modal.service';
 
 @Component({
   selector: 'f-task-submission-card',
@@ -21,7 +22,8 @@ export class TaskSubmissionCardComponent implements OnChanges, OnInit {
     private taskService: TaskService,
     @Inject(uploadSubmissionModal) private UploadSubmissionModal,
     private alerts: AlertService,
-    private fileDownloader: FileDownloaderService
+    private fileDownloader: FileDownloaderService,
+    private feedbackAppealService: FeedbackAppealModalService,
   ) {}
 
   ngOnInit(): void {
@@ -74,17 +76,8 @@ export class TaskSubmissionCardComponent implements OnChanges, OnInit {
     });
   }
 
-  escalateFeedback(): void {
-    // TODO: Open a modal
-    this.task.requestFeedbackReview().subscribe({
-      next: (response) => {
-        console.log(response);
-        this.alerts.success(`Requested feedback review for this task!`, 3000);
-      },
-      error: (error) => {
-        this.alerts.error(`An error occurred: ${error}`, 3000);
-      },
-    });
+  openFeedbackAppealModal(): void {
+    this.feedbackAppealService.show(this.task);
   }
 
   downloadSubmission(): void {
diff --git a/src/app/tasks/modals/feedback-appeal-modal/feedback-appeal-modal.component.html b/src/app/tasks/modals/feedback-appeal-modal/feedback-appeal-modal.component.html
new file mode 100644
index 0000000000..7b8dd0ec9b
--- /dev/null
+++ b/src/app/tasks/modals/feedback-appeal-modal/feedback-appeal-modal.component.html
@@ -0,0 +1,15 @@
+<h2 mat-dialog-title>Appeal Feedback</h2>
+<mat-dialog-content>
+  <p>
+    If you are not happy with the feedback or decision of the marking for your task, you may appeal
+    it.
+  </p>
+  <p>
+    You have <b>{{ task.project.escalationAttemptsRemaining }} requests</b> left to appeal. If the
+    feedback you have received is overturned, you'll receive another appeal request
+  </p>
+  <mat-dialog-actions class="flex justify-between">
+    <button mat-raised-button (click)="dismissModal()">Close</button>
+    <button mat-raised-button color="primary" (click)="submit()">Submit</button>
+  </mat-dialog-actions>
+</mat-dialog-content>
diff --git a/src/app/tasks/modals/feedback-appeal-modal/feedback-appeal-modal.component.scss b/src/app/tasks/modals/feedback-appeal-modal/feedback-appeal-modal.component.scss
new file mode 100644
index 0000000000..e69de29bb2
diff --git a/src/app/tasks/modals/feedback-appeal-modal/feedback-appeal-modal.component.ts b/src/app/tasks/modals/feedback-appeal-modal/feedback-appeal-modal.component.ts
new file mode 100644
index 0000000000..6669c3d28d
--- /dev/null
+++ b/src/app/tasks/modals/feedback-appeal-modal/feedback-appeal-modal.component.ts
@@ -0,0 +1,40 @@
+import {Component, Inject, OnInit} from '@angular/core';
+import {MAT_DIALOG_DATA, MatDialogRef} from '@angular/material/dialog';
+import {Task} from 'src/app/api/models/task';
+import {AlertService} from 'src/app/common/services/alert.service';
+import {FeedbackAppealModalData} from './feedback-appeal-modal.service';
+
+@Component({
+  selector: 'f-feedback-appeal-modal',
+  templateUrl: './feedback-appeal-modal.component.html',
+  styleUrl: './feedback-appeal-modal.component.scss',
+})
+export class FeedbackAppealModalComponent implements OnInit {
+  task: Task;
+
+  constructor(
+    public dialogRef: MatDialogRef<FeedbackAppealModalComponent>,
+    @Inject(MAT_DIALOG_DATA) public data: FeedbackAppealModalData,
+    private alerts: AlertService,
+  ) {}
+
+  ngOnInit() {
+    this.task = this.data.task;
+  }
+
+  submit(): void {
+    this.task.requestFeedbackReview().subscribe({
+      next: (_response) => {
+        this.alerts.success(`Requested feedback review for this task!`, 3000);
+      },
+      error: (error) => {
+        this.alerts.error(`An error occurred: ${error}`, 3000);
+      },
+    });
+    this.dismissModal();
+  }
+
+  public dismissModal() {
+    this.dialogRef.close();
+  }
+}
diff --git a/src/app/tasks/modals/feedback-appeal-modal/feedback-appeal-modal.service.ts b/src/app/tasks/modals/feedback-appeal-modal/feedback-appeal-modal.service.ts
new file mode 100644
index 0000000000..d6726ac1b4
--- /dev/null
+++ b/src/app/tasks/modals/feedback-appeal-modal/feedback-appeal-modal.service.ts
@@ -0,0 +1,29 @@
+import {Injectable} from '@angular/core';
+import {MatDialog} from '@angular/material/dialog';
+import {Task} from 'src/app/api/models/task';
+import {FeedbackAppealModalComponent} from './feedback-appeal-modal.component';
+
+export interface FeedbackAppealModalData {
+  task: Task;
+}
+
+@Injectable({
+  providedIn: 'root',
+})
+export class FeedbackAppealModalService {
+  constructor(public dialog: MatDialog) {}
+
+  public show(task: Task) {
+    const _dialogRef = this.dialog.open<FeedbackAppealModalComponent, FeedbackAppealModalData>(
+      FeedbackAppealModalComponent,
+      {
+        data: {
+          task: task,
+        },
+        position: {top: '2.5%'},
+        width: '100%',
+        maxWidth: '650px',
+      },
+    );
+  }
+}

From 3f819d1607450bb271df3b1045799811eee442e8 Mon Sep 17 00:00:00 2001
From: b0ink <40929320+b0ink@users.noreply.github.com>
Date: Tue, 17 Feb 2026 11:50:04 +1100
Subject: [PATCH 22/38] feat: add feedback review requested comment

---
 .../task-status-card.component.html           | 47 +++++++++++--------
 .../task-status-card.component.ts             |  6 +++
 .../task-submission-card.component.html       |  1 -
 .../task-submission-card.component.ts         |  5 --
 .../feedback-appeal-modal.component.html      | 29 +++++++++---
 .../feedback-appeal-modal.component.ts        | 11 ++++-
 .../task-comments-viewer.component.html       | 21 +++++++++
 7 files changed, 88 insertions(+), 32 deletions(-)

diff --git a/src/app/projects/states/dashboard/directives/task-dashboard/directives/task-status-card/task-status-card.component.html b/src/app/projects/states/dashboard/directives/task-dashboard/directives/task-status-card/task-status-card.component.html
index 7463d4fb13..7b49a65daa 100644
--- a/src/app/projects/states/dashboard/directives/task-dashboard/directives/task-status-card/task-status-card.component.html
+++ b/src/app/projects/states/dashboard/directives/task-dashboard/directives/task-status-card/task-status-card.component.html
@@ -41,25 +41,34 @@ <h5>{{ task?.statusLabel() }}</h5>
     }
   </mat-card-content>
 
-  <mat-card-actions class="space-x-2">
-    <button
-      mat-stroked-button
-      (click)="uploadSubmission()"
-      [disabled]="
-        isReadyForFeedback() || task?.blockedByPrerequisiteTasks() || isSubmittedForPortfolio()
-      "
-    >
-      Upload Submission
-    </button>
+  <mat-card-actions class="justify-between w-full">
+    <div class="space-x-2">
+      <button
+        mat-stroked-button
+        (click)="uploadSubmission()"
+        [disabled]="
+          isReadyForFeedback() || task?.blockedByPrerequisiteTasks() || isSubmittedForPortfolio()
+        "
+      >
+        Upload Submission
+      </button>
 
-    @if (task?.canApplyForExtension()) {
-      <button mat-stroked-button (click)="applyForExtension()">Request extension</button>
-    }
-    @if (task?.inSubmittedState() && task?.requiresFileUpload()) {
-      <button mat-stroked-button (click)="updateFilesInSubmission()">Upload new files</button>
-    }
-    <button mat-raised-button color="primary" (click)="openDiscussionQrCode()">
-      Show Discussion QR
-    </button>
+      @if (task?.canApplyForExtension()) {
+        <button mat-stroked-button (click)="applyForExtension()">Request extension</button>
+      }
+      @if (task?.inSubmittedState() && task?.requiresFileUpload()) {
+        <button mat-stroked-button (click)="updateFilesInSubmission()">Upload new files</button>
+      }
+    </div>
+
+    <div>
+      <button mat-icon-button aria-label="More task options" [matMenuTriggerFor]="menu">
+        <mat-icon>more_vert</mat-icon>
+      </button>
+      <div fxFlex></div>
+      <mat-menu #menu="matMenu">
+        <button mat-menu-item (click)="openFeedbackAppealModal()">Request Feedback Review</button>
+      </mat-menu>
+    </div>
   </mat-card-actions>
 </mat-card>
diff --git a/src/app/projects/states/dashboard/directives/task-dashboard/directives/task-status-card/task-status-card.component.ts b/src/app/projects/states/dashboard/directives/task-dashboard/directives/task-status-card/task-status-card.component.ts
index 70934d7d51..d08c1b1ffd 100644
--- a/src/app/projects/states/dashboard/directives/task-dashboard/directives/task-status-card/task-status-card.component.ts
+++ b/src/app/projects/states/dashboard/directives/task-dashboard/directives/task-status-card/task-status-card.component.ts
@@ -11,6 +11,7 @@ import {SubmissionTypeModalService} from 'src/app/tasks/modals/submission-type-m
 
 import {Project} from 'src/app/api/models/project';
 import {UserService} from 'src/app/api/services/user.service';
+import {FeedbackAppealModalService} from 'src/app/tasks/modals/feedback-appeal-modal/feedback-appeal-modal.service';
 @Component({
   selector: 'f-task-status-card',
   templateUrl: './task-status-card.component.html',
@@ -27,6 +28,7 @@ export class TaskStatusCardComponent implements OnChanges, AfterViewInit {
     private doubtfireConstants: DoubtfireConstants,
     private submissionTypeModalService: SubmissionTypeModalService,
     private userService: UserService,
+    private feedbackAppealService: FeedbackAppealModalService,
   ) {}
 
   @Input() task: Task;
@@ -111,4 +113,8 @@ export class TaskStatusCardComponent implements OnChanges, AfterViewInit {
       this.task.refresh();
     });
   }
+
+  openFeedbackAppealModal(): void {
+    this.feedbackAppealService.show(this.task);
+  }
 }
diff --git a/src/app/projects/states/dashboard/directives/task-dashboard/directives/task-submission-card/task-submission-card.component.html b/src/app/projects/states/dashboard/directives/task-dashboard/directives/task-submission-card/task-submission-card.component.html
index 528489783c..1ffa220a01 100644
--- a/src/app/projects/states/dashboard/directives/task-dashboard/directives/task-submission-card/task-submission-card.component.html
+++ b/src/app/projects/states/dashboard/directives/task-dashboard/directives/task-submission-card/task-submission-card.component.html
@@ -54,7 +54,6 @@
       <button [hidden]="!canRegeneratePdf" mat-menu-item (click)="regeneratePdf()">
         Regenerate PDFs
       </button>
-      <button mat-menu-item (click)="openFeedbackAppealModal()">Request feedback review</button>
     </mat-menu>
   </mat-card-actions>
 </mat-card>
diff --git a/src/app/projects/states/dashboard/directives/task-dashboard/directives/task-submission-card/task-submission-card.component.ts b/src/app/projects/states/dashboard/directives/task-dashboard/directives/task-submission-card/task-submission-card.component.ts
index 0c68db5717..0952260f5a 100644
--- a/src/app/projects/states/dashboard/directives/task-dashboard/directives/task-submission-card/task-submission-card.component.ts
+++ b/src/app/projects/states/dashboard/directives/task-dashboard/directives/task-submission-card/task-submission-card.component.ts
@@ -23,7 +23,6 @@ export class TaskSubmissionCardComponent implements OnChanges, OnInit {
     @Inject(uploadSubmissionModal) private UploadSubmissionModal,
     private alerts: AlertService,
     private fileDownloader: FileDownloaderService,
-    private feedbackAppealService: FeedbackAppealModalService,
   ) {}
 
   ngOnInit(): void {
@@ -76,10 +75,6 @@ export class TaskSubmissionCardComponent implements OnChanges, OnInit {
     });
   }
 
-  openFeedbackAppealModal(): void {
-    this.feedbackAppealService.show(this.task);
-  }
-
   downloadSubmission(): void {
     this.fileDownloader.downloadFile(this.urls.pdf, `${this.task.definition.abbreviation}.pdf`);
   }
diff --git a/src/app/tasks/modals/feedback-appeal-modal/feedback-appeal-modal.component.html b/src/app/tasks/modals/feedback-appeal-modal/feedback-appeal-modal.component.html
index 7b8dd0ec9b..d56373c0ee 100644
--- a/src/app/tasks/modals/feedback-appeal-modal/feedback-appeal-modal.component.html
+++ b/src/app/tasks/modals/feedback-appeal-modal/feedback-appeal-modal.component.html
@@ -1,15 +1,32 @@
-<h2 mat-dialog-title>Appeal Feedback</h2>
+<h2 mat-dialog-title>Request Feedback Review</h2>
 <mat-dialog-content>
   <p>
-    If you are not happy with the feedback or decision of the marking for your task, you may appeal
-    it.
+    If you believe the feedback or marking on this task does not accurately reflect your work, you
+    may request a review. This allows another staff member to reassess the feedback and determine
+    whether it should be confirmed or revised.
   </p>
+
   <p>
-    You have <b>{{ task.project.escalationAttemptsRemaining }} requests</b> left to appeal. If the
-    feedback you have received is overturned, you'll receive another appeal request
+    If you would like to provide context, you can add a comment to the task explaining why you
+    believe the feedback should be reviewed. Referencing specific criteria may help clarify your
+    concerns.
   </p>
+
+  <p>
+    You have
+    <b>
+      {{ task.project.escalationAttemptsRemaining }} review request{{
+        task.project.escalationAttemptsRemaining !== 1 ? 's' : ''
+      }}
+    </b>
+    remaining for this unit. If the original feedback is revised as a result of this review, the
+    request will not be counted against your remaining total.
+  </p>
+
+  <p class="opacity-70">Review decisions are final once resolved.</p>
+
   <mat-dialog-actions class="flex justify-between">
-    <button mat-raised-button (click)="dismissModal()">Close</button>
+    <button mat-raised-button (click)="dismissModal()">Cancel</button>
     <button mat-raised-button color="primary" (click)="submit()">Submit</button>
   </mat-dialog-actions>
 </mat-dialog-content>
diff --git a/src/app/tasks/modals/feedback-appeal-modal/feedback-appeal-modal.component.ts b/src/app/tasks/modals/feedback-appeal-modal/feedback-appeal-modal.component.ts
index 6669c3d28d..3004560f7a 100644
--- a/src/app/tasks/modals/feedback-appeal-modal/feedback-appeal-modal.component.ts
+++ b/src/app/tasks/modals/feedback-appeal-modal/feedback-appeal-modal.component.ts
@@ -3,6 +3,7 @@ import {MAT_DIALOG_DATA, MatDialogRef} from '@angular/material/dialog';
 import {Task} from 'src/app/api/models/task';
 import {AlertService} from 'src/app/common/services/alert.service';
 import {FeedbackAppealModalData} from './feedback-appeal-modal.service';
+import {TaskService} from 'src/app/api/services/task.service';
 
 @Component({
   selector: 'f-feedback-appeal-modal',
@@ -16,6 +17,7 @@ export class FeedbackAppealModalComponent implements OnInit {
     public dialogRef: MatDialogRef<FeedbackAppealModalComponent>,
     @Inject(MAT_DIALOG_DATA) public data: FeedbackAppealModalData,
     private alerts: AlertService,
+    private taskService: TaskService,
   ) {}
 
   ngOnInit() {
@@ -25,7 +27,14 @@ export class FeedbackAppealModalComponent implements OnInit {
   submit(): void {
     this.task.requestFeedbackReview().subscribe({
       next: (_response) => {
-        this.alerts.success(`Requested feedback review for this task!`, 3000);
+        this.alerts.success(
+          `Requested feedback review for ${this.task.definition.abbreviation} ${this.task.definition.name}`,
+          3000,
+        );
+        setTimeout(() => {
+          // Fetch the "Feedback Review Requested" comment
+          this.taskService.notifyStatusChange(this.task);
+        }, 250);
       },
       error: (error) => {
         this.alerts.error(`An error occurred: ${error}`, 3000);
diff --git a/src/app/tasks/task-comments-viewer/task-comments-viewer.component.html b/src/app/tasks/task-comments-viewer/task-comments-viewer.component.html
index 8cbc442d50..924ec54829 100644
--- a/src/app/tasks/task-comments-viewer/task-comments-viewer.component.html
+++ b/src/app/tasks/task-comments-viewer/task-comments-viewer.component.html
@@ -102,6 +102,27 @@
                 </mat-card>
               </div>
 
+              <div
+                *ngSwitchCase="'feedback_review_request'"
+                class="w-full flex flex-col items-center my-2"
+                [matTooltip]="comment.createdAt | humanizedDate"
+                matTooltipPosition="above"
+                matTooltipShowDelay="1000"
+              >
+                <mat-card class="w-full self-end" appearance="outlined">
+                  <mat-card-header>
+                    <mat-icon class="m-auto" style="color: #9696969d">comment</mat-icon>
+                  </mat-card-header>
+
+                  <mat-card-content
+                    style="color: #9696969d"
+                    class="flex flex-col justify-center items-center"
+                  >
+                    {{ comment.text }}
+                  </mat-card-content>
+                </mat-card>
+              </div>
+
               <div
                 *ngSwitchCase="'checked_in'"
                 class="w-full flex flex-col items-center my-2"

From c98be20f4ca9bb4fbbdabd2294b80d6a89e2f626 Mon Sep 17 00:00:00 2001
From: b0ink <40929320+b0ink@users.noreply.github.com>
Date: Tue, 17 Feb 2026 11:51:12 +1100
Subject: [PATCH 23/38] chore: revert task-submission-card

---
 .../task-submission-card.component.html         |  6 ++----
 .../task-submission-card.component.ts           | 17 ++++++++---------
 2 files changed, 10 insertions(+), 13 deletions(-)

diff --git a/src/app/projects/states/dashboard/directives/task-dashboard/directives/task-submission-card/task-submission-card.component.html b/src/app/projects/states/dashboard/directives/task-dashboard/directives/task-submission-card/task-submission-card.component.html
index 1ffa220a01..8d7141d892 100644
--- a/src/app/projects/states/dashboard/directives/task-dashboard/directives/task-submission-card/task-submission-card.component.html
+++ b/src/app/projects/states/dashboard/directives/task-dashboard/directives/task-submission-card/task-submission-card.component.html
@@ -51,9 +51,7 @@
       <button [hidden]="!canReuploadEvidence" mat-menu-item (click)="uploadAlternateFiles()">
         Upload alternate files
       </button>
-      <button [hidden]="!canRegeneratePdf" mat-menu-item (click)="regeneratePdf()">
-        Regenerate PDFs
-      </button>
+      <button [hidden]="!canRegeneratePdf" mat-menu-item (click)="regeneratePdf()">Regenerate PDFs</button>
     </mat-menu>
   </mat-card-actions>
-</mat-card>
+</mat-card>
\ No newline at end of file
diff --git a/src/app/projects/states/dashboard/directives/task-dashboard/directives/task-submission-card/task-submission-card.component.ts b/src/app/projects/states/dashboard/directives/task-dashboard/directives/task-submission-card/task-submission-card.component.ts
index 0952260f5a..8c10c71c40 100644
--- a/src/app/projects/states/dashboard/directives/task-dashboard/directives/task-submission-card/task-submission-card.component.ts
+++ b/src/app/projects/states/dashboard/directives/task-dashboard/directives/task-submission-card/task-submission-card.component.ts
@@ -1,10 +1,9 @@
-import {Component, Inject, Input, OnChanges, OnInit, SimpleChanges} from '@angular/core';
-import {uploadSubmissionModal} from 'src/app/ajs-upgraded-providers';
-import {Task} from 'src/app/api/models/task';
-import {TaskService} from 'src/app/api/services/task.service';
-import {FileDownloaderService} from 'src/app/common/file-downloader/file-downloader.service';
-import {AlertService} from 'src/app/common/services/alert.service';
-import {FeedbackAppealModalService} from 'src/app/tasks/modals/feedback-appeal-modal/feedback-appeal-modal.service';
+import { Component, Inject, Input, OnChanges, OnInit, SimpleChanges } from '@angular/core';
+import { uploadSubmissionModal } from 'src/app/ajs-upgraded-providers';
+import { Task } from 'src/app/api/models/task';
+import { TaskService } from 'src/app/api/services/task.service';
+import { FileDownloaderService } from 'src/app/common/file-downloader/file-downloader.service';
+import { AlertService } from 'src/app/common/services/alert.service';
 
 @Component({
   selector: 'f-task-submission-card',
@@ -22,7 +21,7 @@ export class TaskSubmissionCardComponent implements OnChanges, OnInit {
     private taskService: TaskService,
     @Inject(uploadSubmissionModal) private UploadSubmissionModal,
     private alerts: AlertService,
-    private fileDownloader: FileDownloaderService,
+    private fileDownloader: FileDownloaderService
   ) {}
 
   ngOnInit(): void {
@@ -82,4 +81,4 @@ export class TaskSubmissionCardComponent implements OnChanges, OnInit {
   downloadSubmissionFiles(): void {
     this.fileDownloader.downloadFile(this.urls.files, `${this.task.definition.abbreviation}.zip`);
   }
-}
+}
\ No newline at end of file

From 92215724d3bb9ff6f8db184cf5ec0d887ec78283 Mon Sep 17 00:00:00 2001
From: b0ink <40929320+b0ink@users.noreply.github.com>
Date: Tue, 17 Feb 2026 14:44:38 +1100
Subject: [PATCH 24/38] feat: add moderation confirmation modal and allow
 dismiss all

---
 src/app/api/models/task.ts                    |  6 ++-
 src/app/doubtfire-angular.module.ts           |  2 +
 .../confirm-moderation-modal.component.html   | 28 +++++++++++
 .../confirm-moderation-modal.component.scss   |  0
 .../confirm-moderation-modal.component.ts     | 43 +++++++++++++++++
 .../confirm-moderation-modal.service.ts       | 47 +++++++++++++++++++
 .../moderation/moderation.component.ts        | 43 ++++++++++++-----
 7 files changed, 157 insertions(+), 12 deletions(-)
 create mode 100644 src/app/units/states/tasks/inbox/directives/moderation/confirm-moderation-modal/confirm-moderation-modal.component.html
 create mode 100644 src/app/units/states/tasks/inbox/directives/moderation/confirm-moderation-modal/confirm-moderation-modal.component.scss
 create mode 100644 src/app/units/states/tasks/inbox/directives/moderation/confirm-moderation-modal/confirm-moderation-modal.component.ts
 create mode 100644 src/app/units/states/tasks/inbox/directives/moderation/confirm-moderation-modal/confirm-moderation-modal.service.ts

diff --git a/src/app/api/models/task.ts b/src/app/api/models/task.ts
index 6db7b17f5c..7ef9d5e716 100644
--- a/src/app/api/models/task.ts
+++ b/src/app/api/models/task.ts
@@ -1009,7 +1009,10 @@ export class Task extends Entity {
     return false;
   }
 
-  public moderateFeedback(action: FeedbackModerationActionType): Observable<boolean> {
+  public moderateFeedback(
+    action: FeedbackModerationActionType,
+    applyToAll: boolean = false,
+  ): Observable<boolean> {
     const unitRoleService: UnitRoleService = AppInjector.get(UnitRoleService);
 
     const tutor = this.tutor;
@@ -1026,6 +1029,7 @@ export class Task extends Entity {
         endpointFormat: '/unit_roles/:id:/moderation/:taskId:',
         body: {
           action,
+          apply_to_all: applyToAll,
         },
       },
     );
diff --git a/src/app/doubtfire-angular.module.ts b/src/app/doubtfire-angular.module.ts
index 23c13a0e46..ba5b377a60 100644
--- a/src/app/doubtfire-angular.module.ts
+++ b/src/app/doubtfire-angular.module.ts
@@ -319,6 +319,7 @@ import {TutorNoteService} from './api/services/tutor-note.service';
 import {ModerationComponent} from './units/states/tasks/inbox/directives/moderation/moderation.component';
 import {TutorNotesModalComponent} from './common/modals/tutor-notes-modal/tutor-notes-modal.component';
 import {FeedbackAppealModalComponent} from './tasks/modals/feedback-appeal-modal/feedback-appeal-modal.component';
+import {ConfirmModerationModalComponent} from './units/states/tasks/inbox/directives/moderation/confirm-moderation-modal/confirm-moderation-modal.component';
 
 // See https://stackoverflow.com/questions/55721254/how-to-change-mat-datepicker-date-format-to-dd-mm-yyyy-in-simplest-way/58189036#58189036
 const MY_DATE_FORMAT = {
@@ -513,6 +514,7 @@ const GANTT_CHART_CONFIG = {
     ModerationComponent,
     TutorNotesModalComponent,
     FeedbackAppealModalComponent,
+    ConfirmModerationModalComponent,
   ],
   providers: [
     // Services we provide
diff --git a/src/app/units/states/tasks/inbox/directives/moderation/confirm-moderation-modal/confirm-moderation-modal.component.html b/src/app/units/states/tasks/inbox/directives/moderation/confirm-moderation-modal/confirm-moderation-modal.component.html
new file mode 100644
index 0000000000..0d05c54ed2
--- /dev/null
+++ b/src/app/units/states/tasks/inbox/directives/moderation/confirm-moderation-modal/confirm-moderation-modal.component.html
@@ -0,0 +1,28 @@
+<h3 mat-dialog-title>
+  <div class="flex flex-row items-center">
+    <i class="fa fa-question-circle fa-2x pull-left"></i>
+    <div class="flex flex-col">
+      <div>{{ title }}</div>
+      <small>Please confirm that you want to perform this action.</small>
+    </div>
+  </div>
+</h3>
+<mat-dialog-content>
+  <p>
+    {{ description }}
+  </p>
+  @if (showDismissAll) {
+    <p>
+      <mat-checkbox [(ngModel)]="dismissAllTasks">
+        Apply this action to all {{ task.definition.abbreviation }} {{ task.definition.name }} tasks
+        from {{ task.tutor?.user.name }} currently waiting for moderation
+      </mat-checkbox>
+    </p>
+  }
+</mat-dialog-content>
+<mat-dialog-actions>
+  <button mat-raised-button color="primary" (click)="callback(dismissAllTasks)">
+    <i class="fa fa-check"></i> Confirm
+  </button>
+  <button mat-button color="warn" (click)="dismiss()">Cancel</button>
+</mat-dialog-actions>
diff --git a/src/app/units/states/tasks/inbox/directives/moderation/confirm-moderation-modal/confirm-moderation-modal.component.scss b/src/app/units/states/tasks/inbox/directives/moderation/confirm-moderation-modal/confirm-moderation-modal.component.scss
new file mode 100644
index 0000000000..e69de29bb2
diff --git a/src/app/units/states/tasks/inbox/directives/moderation/confirm-moderation-modal/confirm-moderation-modal.component.ts b/src/app/units/states/tasks/inbox/directives/moderation/confirm-moderation-modal/confirm-moderation-modal.component.ts
new file mode 100644
index 0000000000..738e4481d1
--- /dev/null
+++ b/src/app/units/states/tasks/inbox/directives/moderation/confirm-moderation-modal/confirm-moderation-modal.component.ts
@@ -0,0 +1,43 @@
+import {Component, Inject, OnInit} from '@angular/core';
+import {ConfirmModerationModalData} from './confirm-moderation-modal.service';
+import {MatDialogRef, MAT_DIALOG_DATA} from '@angular/material/dialog';
+import {TaskService} from 'src/app/api/services/task.service';
+import {AlertService} from 'src/app/common/services/alert.service';
+import {FeedbackModerationActionType} from 'src/app/api/models/task';
+import {Task} from 'src/app/api/models/task';
+
+@Component({
+  selector: 'f-confirm-moderation-modal',
+  templateUrl: './confirm-moderation-modal.component.html',
+  styleUrl: './confirm-moderation-modal.component.scss',
+})
+export class ConfirmModerationModalComponent implements OnInit {
+  task: Task;
+  title: string;
+  description: string;
+  action: FeedbackModerationActionType;
+  showDismissAll: boolean;
+  callback: (applyToAll: boolean) => void;
+
+  dismissAllTasks: boolean = false;
+
+  constructor(
+    public dialogRef: MatDialogRef<ConfirmModerationModalComponent>,
+    @Inject(MAT_DIALOG_DATA) public data: ConfirmModerationModalData,
+    private alerts: AlertService,
+    private taskService: TaskService,
+  ) {}
+
+  ngOnInit() {
+    this.task = this.data.task;
+    this.title = this.data.title;
+    this.description = this.data.description;
+    this.action = this.data.action;
+    this.showDismissAll = this.data.showDismissAll;
+    this.callback = this.data.callback;
+  }
+
+  public dismiss() {
+    this.dialogRef.close();
+  }
+}
diff --git a/src/app/units/states/tasks/inbox/directives/moderation/confirm-moderation-modal/confirm-moderation-modal.service.ts b/src/app/units/states/tasks/inbox/directives/moderation/confirm-moderation-modal/confirm-moderation-modal.service.ts
new file mode 100644
index 0000000000..bd271f321c
--- /dev/null
+++ b/src/app/units/states/tasks/inbox/directives/moderation/confirm-moderation-modal/confirm-moderation-modal.service.ts
@@ -0,0 +1,47 @@
+import {Injectable} from '@angular/core';
+import {MatDialog} from '@angular/material/dialog';
+import {FeedbackModerationActionType} from 'src/app/api/models/task';
+import {ConfirmModerationModalComponent} from './confirm-moderation-modal.component';
+import {Task} from 'src/app/api/models/task';
+
+export interface ConfirmModerationModalData {
+  task: Task;
+  title: string;
+  description: string;
+  action: FeedbackModerationActionType;
+  showDismissAll: boolean;
+  callback: (applyToAll: boolean) => void;
+}
+
+@Injectable({
+  providedIn: 'root',
+})
+export class ConfirmModerationModalService {
+  constructor(public dialog: MatDialog) {}
+
+  public show(
+    task: Task,
+    title: string,
+    description: string,
+    action: FeedbackModerationActionType,
+    showDismissAll: boolean,
+    callback: (applyToAll: boolean) => void,
+  ) {
+    const _dialogRef = this.dialog.open<
+      ConfirmModerationModalComponent,
+      ConfirmModerationModalData
+    >(ConfirmModerationModalComponent, {
+      data: {
+        task,
+        title,
+        description,
+        action,
+        showDismissAll,
+        callback,
+      },
+      position: {top: '2.5%'},
+      width: '100%',
+      maxWidth: '650px',
+    });
+  }
+}
diff --git a/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.ts b/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.ts
index a61937cae6..0453b44279 100644
--- a/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.ts
+++ b/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.ts
@@ -2,6 +2,7 @@ import {Component, Input} from '@angular/core';
 import {FeedbackModerationActionType, Task} from 'src/app/api/models/task';
 import {ConfirmationModalService} from 'src/app/common/modals/confirmation-modal/confirmation-modal.service';
 import {AlertService} from 'src/app/common/services/alert.service';
+import {ConfirmModerationModalService} from './confirm-moderation-modal/confirm-moderation-modal.service';
 
 @Component({
   selector: 'f-moderation',
@@ -16,12 +17,16 @@ export class ModerationComponent {
   constructor(
     private alertService: AlertService,
     private confirmationModal: ConfirmationModalService,
+    private confirmModerationModal: ConfirmModerationModalService,
   ) {}
 
   overturn() {
-    this.confirmationModal.show(
+    this.confirmModerationModal.show(
+      this.task,
       'Overturn',
       `There were concerns with feedback given to the student and you have changed the status of the task.`,
+      'overturn',
+      false,
       () => {
         this.moderateTask('overturn');
       },
@@ -29,9 +34,12 @@ export class ModerationComponent {
   }
 
   upheld() {
-    this.confirmationModal.show(
+    this.confirmModerationModal.show(
+      this.task,
       'Upheld',
       `The feedback given to the student was justified, and no changes are required.`,
+      'upheld',
+      false,
       () => {
         this.moderateTask('upheld');
       },
@@ -39,9 +47,12 @@ export class ModerationComponent {
   }
 
   showMore() {
-    this.confirmationModal.show(
+    this.confirmModerationModal.show(
+      this.task,
       'Show more from this tutor',
       `There were concerns with feedback and you would like to see more tasks from this tutor. You will review this task again if further feedback is provided.`,
+      'show_more',
+      false,
       () => {
         this.moderateTask('show_more');
       },
@@ -49,30 +60,40 @@ export class ModerationComponent {
   }
 
   showLess() {
-    this.confirmationModal.show(
+    this.confirmModerationModal.show(
+      this.task,
       'Show less from this tutor',
       `The feedback provided by the tutor is satisfactory and you would like to see less of their feedback for moderation. You won't see this task again.`,
-      () => {
-        this.moderateTask('show_less');
+      'show_less',
+      true,
+      (applyToAll) => {
+        this.moderateTask('show_less', applyToAll);
       },
     );
   }
 
   dismiss() {
-    this.confirmationModal.show(
+    this.confirmModerationModal.show(
+      this.task,
       'Dismiss from moderation',
       'This task will be removed from moderation and will not get a follow up. This will not affect how many tasks from this tutor are selected for moderation.',
-      () => {
-        this.moderateTask('dismiss_ok');
+      'dismiss_ok',
+      true,
+      (applyToAll) => {
+        this.moderateTask('dismiss_ok', applyToAll);
       },
     );
   }
 
-  private moderateTask(action: FeedbackModerationActionType) {
-    this.task.moderateFeedback(action).subscribe({
+  private moderateTask(action: FeedbackModerationActionType, applyToAll: boolean = false) {
+    this.task.moderateFeedback(action, applyToAll).subscribe({
       next: (response) => {
         console.log(response);
         this.alertService.success(`Task moderated`, 2000);
+        if (applyToAll) {
+          // TODO: we should this.setModerated for all tasks in our queue
+          // TODO: should it just refresh the queue?
+        }
         this.setModerated(this.task);
       },
       error: (error) => {

From ec45c3c369d4c05501673264d4394e871a31bc99 Mon Sep 17 00:00:00 2001
From: b0ink <40929320+b0ink@users.noreply.github.com>
Date: Tue, 17 Feb 2026 15:00:06 +1100
Subject: [PATCH 25/38] chore: dismiss modal

---
 .../confirm-moderation-modal.component.html                  | 4 ++--
 .../confirm-moderation-modal.component.ts                    | 5 +++++
 2 files changed, 7 insertions(+), 2 deletions(-)

diff --git a/src/app/units/states/tasks/inbox/directives/moderation/confirm-moderation-modal/confirm-moderation-modal.component.html b/src/app/units/states/tasks/inbox/directives/moderation/confirm-moderation-modal/confirm-moderation-modal.component.html
index 0d05c54ed2..1cc2804d28 100644
--- a/src/app/units/states/tasks/inbox/directives/moderation/confirm-moderation-modal/confirm-moderation-modal.component.html
+++ b/src/app/units/states/tasks/inbox/directives/moderation/confirm-moderation-modal/confirm-moderation-modal.component.html
@@ -15,13 +15,13 @@ <h3 mat-dialog-title>
     <p>
       <mat-checkbox [(ngModel)]="dismissAllTasks">
         Apply this action to all {{ task.definition.abbreviation }} {{ task.definition.name }} tasks
-        from {{ task.tutor?.user.name }} currently waiting for moderation
+        from {{ task.tutor?.user.name ?? 'N/A' }}, currently waiting for moderation
       </mat-checkbox>
     </p>
   }
 </mat-dialog-content>
 <mat-dialog-actions>
-  <button mat-raised-button color="primary" (click)="callback(dismissAllTasks)">
+  <button mat-raised-button color="primary" (click)="runCallback()">
     <i class="fa fa-check"></i> Confirm
   </button>
   <button mat-button color="warn" (click)="dismiss()">Cancel</button>
diff --git a/src/app/units/states/tasks/inbox/directives/moderation/confirm-moderation-modal/confirm-moderation-modal.component.ts b/src/app/units/states/tasks/inbox/directives/moderation/confirm-moderation-modal/confirm-moderation-modal.component.ts
index 738e4481d1..9b61056ad6 100644
--- a/src/app/units/states/tasks/inbox/directives/moderation/confirm-moderation-modal/confirm-moderation-modal.component.ts
+++ b/src/app/units/states/tasks/inbox/directives/moderation/confirm-moderation-modal/confirm-moderation-modal.component.ts
@@ -37,6 +37,11 @@ export class ConfirmModerationModalComponent implements OnInit {
     this.callback = this.data.callback;
   }
 
+  public runCallback() {
+    this.callback(this.dismissAllTasks);
+    this.dismiss();
+  }
+
   public dismiss() {
     this.dialogRef.close();
   }

From f1f6aed7bfa2839cc0adb7f62d7180b403a68fab Mon Sep 17 00:00:00 2001
From: b0ink <40929320+b0ink@users.noreply.github.com>
Date: Tue, 17 Feb 2026 15:34:46 +1100
Subject: [PATCH 26/38] feat: filter tasks by tutor

---
 src/app/common/filters/tasks-by-tutor.pipe.ts | 15 +++++++
 .../staff-task-list.component.html            | 20 ++++++++++
 .../staff-task-list.component.ts              | 39 +++++++++++++++++++
 3 files changed, 74 insertions(+)
 create mode 100644 src/app/common/filters/tasks-by-tutor.pipe.ts

diff --git a/src/app/common/filters/tasks-by-tutor.pipe.ts b/src/app/common/filters/tasks-by-tutor.pipe.ts
new file mode 100644
index 0000000000..c95634d6a3
--- /dev/null
+++ b/src/app/common/filters/tasks-by-tutor.pipe.ts
@@ -0,0 +1,15 @@
+import {Pipe, PipeTransform} from '@angular/core';
+import {Task} from '../../api/models/doubtfire-model';
+
+@Pipe({
+  name: 'tasksByTutor',
+})
+export class TasksByTutorPipe implements PipeTransform {
+  transform(tasks: Task[], unitRoleId?: number | string): Task[] {
+    if (!tasks) return tasks;
+
+    if (!unitRoleId || unitRoleId === 'all') return tasks;
+
+    return tasks.filter((task) => task.tutor?.id === unitRoleId);
+  }
+}
diff --git a/src/app/units/states/tasks/inbox/directives/staff-task-list/staff-task-list.component.html b/src/app/units/states/tasks/inbox/directives/staff-task-list/staff-task-list.component.html
index 2a36a9704f..cbe2bdbff4 100644
--- a/src/app/units/states/tasks/inbox/directives/staff-task-list/staff-task-list.component.html
+++ b/src/app/units/states/tasks/inbox/directives/staff-task-list/staff-task-list.component.html
@@ -153,6 +153,26 @@
               <mat-icon>{{ states[tutorialSort].icon }}</mat-icon>
             </button>
           </div>
+          <div class="flex flex-row justify-between items-center">
+            <mat-form-field class="flex-1">
+              <mat-label>Tutors</mat-label>
+              <mat-select
+                [disabled]="filters.tutorialIdSelected !== 'all'"
+                [(ngModel)]="filters.unitRoleIdSelected"
+                name="tutor"
+                (selectionChange)="unitRoleIdChanged()"
+              >
+                @for (t of tutorFilter; track t) {
+                  <mat-option [value]="t.id">
+                    <span class="truncate">{{ t.inboxDescription }}</span>
+                  </mat-option>
+                }
+              </mat-select>
+            </mat-form-field>
+            <button disabled mat-icon-button aria-label="Sort by tutor icon">
+              <mat-icon>{{ states[tutorialSort].icon }}</mat-icon>
+            </button>
+          </div>
           <!--/students-->
         </form>
       </mat-expansion-panel>
diff --git a/src/app/units/states/tasks/inbox/directives/staff-task-list/staff-task-list.component.ts b/src/app/units/states/tasks/inbox/directives/staff-task-list/staff-task-list.component.ts
index 7d853e2ce0..fdac3dfc10 100644
--- a/src/app/units/states/tasks/inbox/directives/staff-task-list/staff-task-list.component.ts
+++ b/src/app/units/states/tasks/inbox/directives/staff-task-list/staff-task-list.component.ts
@@ -34,6 +34,7 @@ import {HotkeysService} from '@ngneat/hotkeys';
 import {Router} from '@angular/router';
 import {TaskDefinitionService} from 'src/app/api/services/task-definition.service';
 import {SidekiqProgressModalService} from 'src/app/common/modals/sidekiq-progress-modal/sidekiq-progress-modal.service';
+import {TasksByTutorPipe} from 'src/app/common/filters/tasks-by-tutor.pipe';
 
 @Component({
   selector: 'df-staff-task-list',
@@ -65,6 +66,7 @@ export class StaffTaskListComponent implements OnInit, OnChanges, OnDestroy {
     forceStream: boolean;
     studentName: string;
     tutorialIdSelected: any;
+    unitRoleIdSelected: number | string;
     taskDefinitionIdSelected: number | TaskDefinition;
   };
   @Input() showSearchOptions = true;
@@ -84,6 +86,11 @@ export class StaffTaskListComponent implements OnInit, OnChanges, OnDestroy {
     tutorial?: Tutorial;
   }[] = null;
 
+  tutorFilter: {
+    id: number | string;
+    inboxDescription: string;
+  }[] = null;
+
   tasks: Task[] = null;
 
   // hasJplagReport: boolean = false;
@@ -96,6 +103,7 @@ export class StaffTaskListComponent implements OnInit, OnChanges, OnDestroy {
   definedTasksPipe = new TasksOfTaskDefinitionPipe();
   tasksInTutorialsPipe = new TasksInTutorialsPipe();
   taskWithStudentNamePipe = new TasksForInboxSearchPipe();
+  tasksByTutorPipe = new TasksByTutorPipe();
   // Let's call having a source of tasksForDefinition plus having a task definition
   // auto-selected with the search options open task def mode -- i.e., the mode
   // for selecting tasks by task definitions
@@ -183,6 +191,7 @@ export class StaffTaskListComponent implements OnInit, OnChanges, OnDestroy {
         tutorialIdSelected:
           (this.unitRole.role === 'Tutor' || 'Convenor') && this.userHasTutorials ? 'mine' : 'all',
         tutorials: [],
+        unitRoleIdSelected: 'all',
         taskDefinitionIdSelected: null,
         taskDefinition: null,
         forceStream: true,
@@ -211,6 +220,17 @@ export class StaffTaskListComponent implements OnInit, OnChanges, OnDestroy {
       }),
     ];
 
+    this.tutorFilter = [
+      {
+        id: 'all',
+        inboxDescription: 'All Tutors',
+      },
+      ...this.unit.staff.map((ur: UnitRole) => ({
+        id: ur.id,
+        inboxDescription: ur.user?.name,
+      })),
+    ];
+
     this.tutorialIdChanged(false);
 
     this.setTaskDefFromTaskKey(this.taskData.taskKey);
@@ -301,6 +321,14 @@ export class StaffTaskListComponent implements OnInit, OnChanges, OnDestroy {
         this.filters.forceStream,
       );
     }
+
+    if (this.filters.unitRoleIdSelected) {
+      filteredTasks = this.tasksByTutorPipe.transform(
+        filteredTasks,
+        this.filters.unitRoleIdSelected,
+      );
+    }
+
     filteredTasks = this.taskWithStudentNamePipe.transform(filteredTasks, this.filters.studentName);
     this.filteredTasks = filteredTasks;
 
@@ -326,6 +354,15 @@ export class StaffTaskListComponent implements OnInit, OnChanges, OnDestroy {
     selectEl.focus();
   }
 
+  unitRoleIdChanged(attemptRefreshData: boolean = true): void {
+    this.applyFilters();
+
+    const isExplorerView = this.isTaskDefMode;
+    if (attemptRefreshData && !this.fetchedAllTasks && !isExplorerView) {
+      this.refreshData();
+    }
+  }
+
   tutorialIdChanged(attemptRefreshData: boolean = true): void {
     const tutorialId = this.filters.tutorialIdSelected;
 
@@ -335,11 +372,13 @@ export class StaffTaskListComponent implements OnInit, OnChanges, OnDestroy {
 
     if (tutorialId === 'mine') {
       this.filters.tutorials = this.unit.tutorialsForUserName(this.userService.currentUser.name);
+      this.filters.unitRoleIdSelected = 'all';
     } else if (tutorialId === 'all') {
       // Ignore tutorials filter
       this.filters.tutorials = null;
     } else {
       this.filters.tutorials = [filterOption.tutorial];
+      this.filters.unitRoleIdSelected = 'all';
     }
 
     this.applyFilters();

From e646f831bfd7bc48fa6009bedf6007be49c4054a Mon Sep 17 00:00:00 2001
From: b0ink <40929320+b0ink@users.noreply.github.com>
Date: Tue, 17 Feb 2026 15:56:20 +1100
Subject: [PATCH 27/38] refactor: style action buttons

---
 .../confirm-moderation-modal.component.html                   | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/app/units/states/tasks/inbox/directives/moderation/confirm-moderation-modal/confirm-moderation-modal.component.html b/src/app/units/states/tasks/inbox/directives/moderation/confirm-moderation-modal/confirm-moderation-modal.component.html
index 1cc2804d28..f20aa00ea3 100644
--- a/src/app/units/states/tasks/inbox/directives/moderation/confirm-moderation-modal/confirm-moderation-modal.component.html
+++ b/src/app/units/states/tasks/inbox/directives/moderation/confirm-moderation-modal/confirm-moderation-modal.component.html
@@ -20,9 +20,9 @@ <h3 mat-dialog-title>
     </p>
   }
 </mat-dialog-content>
-<mat-dialog-actions>
+<mat-dialog-actions class="flex justify-between">
+  <button mat-button color="warn" (click)="dismiss()">Cancel</button>
   <button mat-raised-button color="primary" (click)="runCallback()">
     <i class="fa fa-check"></i> Confirm
   </button>
-  <button mat-button color="warn" (click)="dismiss()">Cancel</button>
 </mat-dialog-actions>

From edc55ae0946e88a638f7490daacde44aef69ef20 Mon Sep 17 00:00:00 2001
From: b0ink <40929320+b0ink@users.noreply.github.com>
Date: Tue, 17 Feb 2026 17:07:40 +1100
Subject: [PATCH 28/38] chore: rename staff to tutor

---
 .../feedback-appeal-modal.component.html                      | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/app/tasks/modals/feedback-appeal-modal/feedback-appeal-modal.component.html b/src/app/tasks/modals/feedback-appeal-modal/feedback-appeal-modal.component.html
index d56373c0ee..fdf1eeca0f 100644
--- a/src/app/tasks/modals/feedback-appeal-modal/feedback-appeal-modal.component.html
+++ b/src/app/tasks/modals/feedback-appeal-modal/feedback-appeal-modal.component.html
@@ -2,8 +2,8 @@ <h2 mat-dialog-title>Request Feedback Review</h2>
 <mat-dialog-content>
   <p>
     If you believe the feedback or marking on this task does not accurately reflect your work, you
-    may request a review. This allows another staff member to reassess the feedback and determine
-    whether it should be confirmed or revised.
+    may request a review. This allows another tutor to reassess the feedback and determine whether
+    it should be confirmed or revised.
   </p>
 
   <p>

From 5bc3e920416b16b9ffda33ad67ac0b6904087ffb Mon Sep 17 00:00:00 2001
From: b0ink <40929320+b0ink@users.noreply.github.com>
Date: Tue, 17 Feb 2026 17:20:14 +1100
Subject: [PATCH 29/38] refactor: improve wording

---
 .../inbox/directives/moderation/moderation.component.ts  | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.ts b/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.ts
index 0453b44279..9d2cf9dd5d 100644
--- a/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.ts
+++ b/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.ts
@@ -1,6 +1,5 @@
 import {Component, Input} from '@angular/core';
 import {FeedbackModerationActionType, Task} from 'src/app/api/models/task';
-import {ConfirmationModalService} from 'src/app/common/modals/confirmation-modal/confirmation-modal.service';
 import {AlertService} from 'src/app/common/services/alert.service';
 import {ConfirmModerationModalService} from './confirm-moderation-modal/confirm-moderation-modal.service';
 
@@ -16,7 +15,6 @@ export class ModerationComponent {
 
   constructor(
     private alertService: AlertService,
-    private confirmationModal: ConfirmationModalService,
     private confirmModerationModal: ConfirmModerationModalService,
   ) {}
 
@@ -24,7 +22,8 @@ export class ModerationComponent {
     this.confirmModerationModal.show(
       this.task,
       'Overturn',
-      `There were concerns with feedback given to the student and you have changed the status of the task.`,
+      `The original feedback will be revised and the task outcome updated.
+      Provide clear, constructive feedback explaining the changes made, and record any guidance for the tutor in the tutor notes section.`,
       'overturn',
       false,
       () => {
@@ -37,7 +36,9 @@ export class ModerationComponent {
     this.confirmModerationModal.show(
       this.task,
       'Upheld',
-      `The feedback given to the student was justified, and no changes are required.`,
+      `After reviewing the task and feedback, you are satisfied that the original marking and comments are appropriate and align with the assessment criteria.
+      No changes will be made.
+      Ensure any concerns raised by the student have been considered and addressed where necessary.`,
       'upheld',
       false,
       () => {

From a0fe6ca2d7d6b9505a86f8eb3291a93f6113f9f0 Mon Sep 17 00:00:00 2001
From: b0ink <40929320+b0ink@users.noreply.github.com>
Date: Tue, 17 Feb 2026 17:41:41 +1100
Subject: [PATCH 30/38] refactor: improve styling

---
 .../inbox/directives/moderation/moderation.component.html | 8 +++++---
 .../inbox/directives/moderation/moderation.component.scss | 2 +-
 2 files changed, 6 insertions(+), 4 deletions(-)

diff --git a/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.html b/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.html
index ed7e9be397..efb5f80449 100644
--- a/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.html
+++ b/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.html
@@ -1,5 +1,5 @@
 <!-- <div class="flex justify-center py-2">Tutor: {{ task.tutor?.user.name }}</div> -->
-<span class="center-buttons flex justify-around gap-2">
+<span class="center-buttons flex justify-around gap-5 mx-5">
   @if (task.moderationType === 'random_sample' || task.moderationType === 'first_feedback') {
     <button
       [disabled]="task === undefined || isModerated(task)"
@@ -44,7 +44,8 @@
       class="button"
       (click)="overturn()"
     >
-      <mat-icon>flag</mat-icon>
+      <!-- <mat-icon>flag</mat-icon> -->
+      <mat-icon>gavel</mat-icon>
     </button>
 
     <button
@@ -56,7 +57,8 @@
       class="button"
       (click)="upheld()"
     >
-      <mat-icon>thumb_up</mat-icon>
+      <!-- <mat-icon>thumb_up</mat-icon> -->
+      <mat-icon>verified</mat-icon>
     </button>
   }
 </span>
diff --git a/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.scss b/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.scss
index 9c9f0165e1..168ca35fb9 100644
--- a/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.scss
+++ b/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.scss
@@ -2,7 +2,7 @@
   text-align: center;
 
   .button {
-    margin: 0 6px;
+    // margin: 0 6px;
     transform: scale(0.8);
     border-radius: 100%;
   }

From e5c5a8d1d3e17b65d8165d9e1c493c5f5fb1eece Mon Sep 17 00:00:00 2001
From: b0ink <40929320+b0ink@users.noreply.github.com>
Date: Tue, 17 Feb 2026 17:41:53 +1100
Subject: [PATCH 31/38] chore: add relevant icon

---
 .../confirm-moderation-modal.component.html      | 16 +++++++++++++++-
 1 file changed, 15 insertions(+), 1 deletion(-)

diff --git a/src/app/units/states/tasks/inbox/directives/moderation/confirm-moderation-modal/confirm-moderation-modal.component.html b/src/app/units/states/tasks/inbox/directives/moderation/confirm-moderation-modal/confirm-moderation-modal.component.html
index f20aa00ea3..3ea997a64d 100644
--- a/src/app/units/states/tasks/inbox/directives/moderation/confirm-moderation-modal/confirm-moderation-modal.component.html
+++ b/src/app/units/states/tasks/inbox/directives/moderation/confirm-moderation-modal/confirm-moderation-modal.component.html
@@ -1,6 +1,20 @@
 <h3 mat-dialog-title>
   <div class="flex flex-row items-center">
-    <i class="fa fa-question-circle fa-2x pull-left"></i>
+    <div class="mr-5 flex items-center">
+      @if (action === 'show_less') {
+        <mat-icon>thumb_up</mat-icon>
+      } @else if (action === 'show_more') {
+        <mat-icon>flag</mat-icon>
+      } @else if (action === 'dismiss_ok') {
+        <mat-icon>check_circle</mat-icon>
+      } @else if (action === 'dismiss_good') {
+        <mat-icon>task_alt</mat-icon>
+      } @else if (action === 'overturn') {
+        <mat-icon>gavel</mat-icon>
+      } @else if (action === 'upheld') {
+        <mat-icon>verified</mat-icon>
+      }
+    </div>
     <div class="flex flex-col">
       <div>{{ title }}</div>
       <small>Please confirm that you want to perform this action.</small>

From 4948cc08542fc729bd14a276a30f9dd876516e20 Mon Sep 17 00:00:00 2001
From: b0ink <40929320+b0ink@users.noreply.github.com>
Date: Tue, 17 Feb 2026 17:53:14 +1100
Subject: [PATCH 32/38] refactor: update success alert

---
 .../directives/moderation/moderation.component.ts | 15 ++++++++++-----
 1 file changed, 10 insertions(+), 5 deletions(-)

diff --git a/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.ts b/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.ts
index 9d2cf9dd5d..3e20484d52 100644
--- a/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.ts
+++ b/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.ts
@@ -2,6 +2,7 @@ import {Component, Input} from '@angular/core';
 import {FeedbackModerationActionType, Task} from 'src/app/api/models/task';
 import {AlertService} from 'src/app/common/services/alert.service';
 import {ConfirmModerationModalService} from './confirm-moderation-modal/confirm-moderation-modal.service';
+import {Router} from '@angular/router';
 
 @Component({
   selector: 'f-moderation',
@@ -16,6 +17,7 @@ export class ModerationComponent {
   constructor(
     private alertService: AlertService,
     private confirmModerationModal: ConfirmModerationModalService,
+    private router: Router,
   ) {}
 
   overturn() {
@@ -88,13 +90,16 @@ export class ModerationComponent {
 
   private moderateTask(action: FeedbackModerationActionType, applyToAll: boolean = false) {
     this.task.moderateFeedback(action, applyToAll).subscribe({
-      next: (response) => {
-        console.log(response);
-        this.alertService.success(`Task moderated`, 2000);
+      next: (_response) => {
         if (applyToAll) {
-          // TODO: we should this.setModerated for all tasks in our queue
-          // TODO: should it just refresh the queue?
+          this.alertService.success(
+            `Tasks moderated successfully. Refresh the list to view the updated queue.`,
+            2000,
+          );
+        } else {
+          this.alertService.success(`Task moderated successfully`, 2000);
         }
+
         this.setModerated(this.task);
       },
       error: (error) => {

From 704da11fee6ffd33114eae3236f53bb9ed93bb8d Mon Sep 17 00:00:00 2001
From: b0ink <40929320+b0ink@users.noreply.github.com>
Date: Tue, 17 Feb 2026 17:54:43 +1100
Subject: [PATCH 33/38] chore: increase alert duration

---
 .../tasks/inbox/directives/moderation/moderation.component.ts   | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.ts b/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.ts
index 3e20484d52..991cf71574 100644
--- a/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.ts
+++ b/src/app/units/states/tasks/inbox/directives/moderation/moderation.component.ts
@@ -94,7 +94,7 @@ export class ModerationComponent {
         if (applyToAll) {
           this.alertService.success(
             `Tasks moderated successfully. Refresh the list to view the updated queue.`,
-            2000,
+            6000,
           );
         } else {
           this.alertService.success(`Task moderated successfully`, 2000);

From f2a5eb06cdcd0b23a2dcd4bd05e9ba2a0a659767 Mon Sep 17 00:00:00 2001
From: b0ink <40929320+b0ink@users.noreply.github.com>
Date: Wed, 18 Feb 2026 09:30:33 +1100
Subject: [PATCH 34/38] chore: display tasks tutor

---
 .../confirm-moderation-modal.component.html                    | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/app/units/states/tasks/inbox/directives/moderation/confirm-moderation-modal/confirm-moderation-modal.component.html b/src/app/units/states/tasks/inbox/directives/moderation/confirm-moderation-modal/confirm-moderation-modal.component.html
index 3ea997a64d..907c9793df 100644
--- a/src/app/units/states/tasks/inbox/directives/moderation/confirm-moderation-modal/confirm-moderation-modal.component.html
+++ b/src/app/units/states/tasks/inbox/directives/moderation/confirm-moderation-modal/confirm-moderation-modal.component.html
@@ -17,7 +17,8 @@ <h3 mat-dialog-title>
     </div>
     <div class="flex flex-col">
       <div>{{ title }}</div>
-      <small>Please confirm that you want to perform this action.</small>
+      <!-- <small>Please confirm that you want to perform this action.</small> -->
+      <small>Tutor: {{ task.tutor.user.name }}</small>
     </div>
   </div>
 </h3>

From a0b89f4cc373e3cd49acdf34bba2f0c2b0ba7084 Mon Sep 17 00:00:00 2001
From: b0ink <40929320+b0ink@users.noreply.github.com>
Date: Wed, 18 Feb 2026 09:32:06 +1100
Subject: [PATCH 35/38] refactor: sort tutors alphabetically

---
 .../staff-task-list/staff-task-list.component.ts      | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

diff --git a/src/app/units/states/tasks/inbox/directives/staff-task-list/staff-task-list.component.ts b/src/app/units/states/tasks/inbox/directives/staff-task-list/staff-task-list.component.ts
index fdac3dfc10..2534696fc9 100644
--- a/src/app/units/states/tasks/inbox/directives/staff-task-list/staff-task-list.component.ts
+++ b/src/app/units/states/tasks/inbox/directives/staff-task-list/staff-task-list.component.ts
@@ -225,10 +225,13 @@ export class StaffTaskListComponent implements OnInit, OnChanges, OnDestroy {
         id: 'all',
         inboxDescription: 'All Tutors',
       },
-      ...this.unit.staff.map((ur: UnitRole) => ({
-        id: ur.id,
-        inboxDescription: ur.user?.name,
-      })),
+      ...this.unit.staff
+        .slice()
+        .sort((a, b) => (a.user?.name ?? '').localeCompare(b.user?.name ?? ''))
+        .map((ur: UnitRole) => ({
+          id: ur.id,
+          inboxDescription: ur.user?.name,
+        })),
     ];
 
     this.tutorialIdChanged(false);

From 5c4f7cdbe2d28df88e4e315c1f76b69d2a1dd027 Mon Sep 17 00:00:00 2001
From: b0ink <40929320+b0ink@users.noreply.github.com>
Date: Wed, 18 Feb 2026 11:13:03 +1100
Subject: [PATCH 36/38] chore: remove divider on the right

---
 src/app/common/footer/footer.component.html | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/app/common/footer/footer.component.html b/src/app/common/footer/footer.component.html
index efd547190a..b1caed6d6d 100644
--- a/src/app/common/footer/footer.component.html
+++ b/src/app/common/footer/footer.component.html
@@ -125,7 +125,7 @@
       @if (viewType === 'moderation' && selectedTask) {
         <mat-divider class="ml-10" [vertical]="true"></mat-divider>
         <f-moderation [task]="selectedTask"></f-moderation>
-        <mat-divider class="mr-10" [vertical]="true"></mat-divider>
+        <!-- <mat-divider class="mr-10" [vertical]="true"></mat-divider> -->
       }
     </div>
 

From c7f36032ab6c763038039fca3fa8322d9e5954bc Mon Sep 17 00:00:00 2001
From: b0ink <40929320+b0ink@users.noreply.github.com>
Date: Wed, 18 Feb 2026 13:25:37 +1100
Subject: [PATCH 37/38] refactor: require a comment for review requests

---
 .../feedback-appeal-modal.component.html      | 28 ++++++++++++++-----
 .../feedback-appeal-modal.component.ts        | 10 ++++++-
 2 files changed, 30 insertions(+), 8 deletions(-)

diff --git a/src/app/tasks/modals/feedback-appeal-modal/feedback-appeal-modal.component.html b/src/app/tasks/modals/feedback-appeal-modal/feedback-appeal-modal.component.html
index fdf1eeca0f..131bc86541 100644
--- a/src/app/tasks/modals/feedback-appeal-modal/feedback-appeal-modal.component.html
+++ b/src/app/tasks/modals/feedback-appeal-modal/feedback-appeal-modal.component.html
@@ -6,12 +6,6 @@ <h2 mat-dialog-title>Request Feedback Review</h2>
     it should be confirmed or revised.
   </p>
 
-  <p>
-    If you would like to provide context, you can add a comment to the task explaining why you
-    believe the feedback should be reviewed. Referencing specific criteria may help clarify your
-    concerns.
-  </p>
-
   <p>
     You have
     <b>
@@ -23,10 +17,30 @@ <h2 mat-dialog-title>Request Feedback Review</h2>
     request will not be counted against your remaining total.
   </p>
 
+  <mat-form-field appearance="outline" class="w-full mt-4">
+    <mat-label>Reason for review request</mat-label>
+    <textarea
+      matInput
+      rows="4"
+      maxlength="1000"
+      [(ngModel)]="reviewComment"
+      placeholder="Briefly explain why you believe this feedback should be reviewed. Refer to specific criteria if relevant."
+    ></textarea>
+    <div class="flex justify-end">
+      <mat-hint> {{ reviewComment?.length || 0 }}/1000 </mat-hint>
+    </div>
+  </mat-form-field>
   <p class="opacity-70">Review decisions are final once resolved.</p>
 
   <mat-dialog-actions class="flex justify-between">
     <button mat-raised-button (click)="dismissModal()">Cancel</button>
-    <button mat-raised-button color="primary" (click)="submit()">Submit</button>
+    <button
+      mat-raised-button
+      color="primary"
+      (click)="submit()"
+      [disabled]="!reviewComment || submitting"
+    >
+      Submit
+    </button>
   </mat-dialog-actions>
 </mat-dialog-content>
diff --git a/src/app/tasks/modals/feedback-appeal-modal/feedback-appeal-modal.component.ts b/src/app/tasks/modals/feedback-appeal-modal/feedback-appeal-modal.component.ts
index 3004560f7a..d44c1853ca 100644
--- a/src/app/tasks/modals/feedback-appeal-modal/feedback-appeal-modal.component.ts
+++ b/src/app/tasks/modals/feedback-appeal-modal/feedback-appeal-modal.component.ts
@@ -13,6 +13,9 @@ import {TaskService} from 'src/app/api/services/task.service';
 export class FeedbackAppealModalComponent implements OnInit {
   task: Task;
 
+  reviewComment: string;
+  submitting: boolean;
+
   constructor(
     public dialogRef: MatDialogRef<FeedbackAppealModalComponent>,
     @Inject(MAT_DIALOG_DATA) public data: FeedbackAppealModalData,
@@ -25,6 +28,7 @@ export class FeedbackAppealModalComponent implements OnInit {
   }
 
   submit(): void {
+    this.submitting = true;
     this.task.requestFeedbackReview().subscribe({
       next: (_response) => {
         this.alerts.success(
@@ -34,13 +38,17 @@ export class FeedbackAppealModalComponent implements OnInit {
         setTimeout(() => {
           // Fetch the "Feedback Review Requested" comment
           this.taskService.notifyStatusChange(this.task);
+          setTimeout(() => {
+            this.task.addComment(this.reviewComment);
+          }, 250);
+          this.dismissModal();
         }, 250);
       },
       error: (error) => {
         this.alerts.error(`An error occurred: ${error}`, 3000);
+        this.submitting = false;
       },
     });
-    this.dismissModal();
   }
 
   public dismissModal() {

From 8cdc482e88fda437f8526c7ec30729d00e72b66f Mon Sep 17 00:00:00 2001
From: b0ink <40929320+b0ink@users.noreply.github.com>
Date: Wed, 18 Feb 2026 13:45:29 +1100
Subject: [PATCH 38/38] chore: increase modal max width

---
 .../feedback-appeal-modal/feedback-appeal-modal.service.ts      | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/app/tasks/modals/feedback-appeal-modal/feedback-appeal-modal.service.ts b/src/app/tasks/modals/feedback-appeal-modal/feedback-appeal-modal.service.ts
index d6726ac1b4..622d3bf99f 100644
--- a/src/app/tasks/modals/feedback-appeal-modal/feedback-appeal-modal.service.ts
+++ b/src/app/tasks/modals/feedback-appeal-modal/feedback-appeal-modal.service.ts
@@ -22,7 +22,7 @@ export class FeedbackAppealModalService {
         },
         position: {top: '2.5%'},
         width: '100%',
-        maxWidth: '650px',
+        maxWidth: '700px',
       },
     );
   }