Skip to content

Orphaned progress-report feature (Report model + report.post.ts have no UI caller) #298

Description

@TusharW4ni

Part of #285 (Group D — robustness).

Problem

There is a second, genuinely separate "progress report" feature (Report/ReportData) whose backend exists but has no UI caller — it's orphaned. Meanwhile the UI's "progress report" button actually edits a TherapyNote.

Evidence

  • prisma/schema.prisma:193-208Report + ReportData (dated question/answer sets, Patient.ProgressReports).
  • server/api/profile/report.post.ts (THERAPIST + isAssignedTherapist) writes a Report — but grep across pages/, components/, composables/ for profile/report finds no caller, and there is no read endpoint for Report.
  • The UI's "progress report" modal is components/therapy/ReportModal.vue, which despite its name POSTs to /api/session/notes (a TherapyNote), not to report.post.ts.

Impact

P2 (dead code / confusion). Either a planned feature was never wired, or it's redundant with TherapyNote.

Proposed approach

  • Decide whether Report/progress-reports is a real feature distinct from TherapyNote.
    • If yes: build the UI (create + read) and a GET endpoint.
    • If no: remove report.post.ts, the models, and rename ReportModal.vue to reflect that it edits notes (see D14).

Acceptance criteria

  • Report is either wired end-to-end or removed.
  • No orphaned endpoint/model remains without a decision recorded.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    P2: mediumImprovements, non-blocking bugsarea: patientSearch, profile, medical recordstech-debtCode cleanup, refactoring, dependency maintenance

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions