Skip to content

Minor issues#44

Merged
ManvithaDungi merged 7 commits into
mainfrom
minor-issues
Mar 11, 2026
Merged

Minor issues#44
ManvithaDungi merged 7 commits into
mainfrom
minor-issues

Conversation

@ManvithaDungi

@ManvithaDungi ManvithaDungi commented Mar 10, 2026

Copy link
Copy Markdown
Collaborator

PR Summary — UI Cleanup, API Integration & Role Fixes

Notifications

  • Removed hardcoded notifications from Header.jsx, DashboardLayout.jsx, and NotificationsPanel.jsx
  • Removed static unread badges and broken buttons
  • Replaced with "No new notifications" empty state

Patient Profile

  • Fixed error handling (early return on API failure)
  • Displayed medicalHistory and resolved assignedDoctorId to doctor name
  • Restored rich UI layout using real API data
  • Added inline edit mode (contact number, address, date of birth)
  • Fixed duplicate component build error
  • Removed unused imports

Doctor Dashboard & Appointments

  • Removed mock notifications panel data
  • Removed unused "+ New Appointment" button (patients create appointments)

Lab Orders & Results

Frontend

  • Wired Order Lab Test modal to "Order New Labs" button
  • Removed result-entry fields from doctor UI
  • Removed client-side status / orderedAt overrides
    Backend
  • Fixed ordered_by_id NOT NULL constraint (setOrderedBy)
  • Added default status = "PENDING"
  • Added GET /api/lab-results endpoint
  • Implemented getAllLabTests() service method
    Lab Results UI
  • Removed mock labs and duplicate IDs
  • Added api.labResults.getAll()
  • Fixed field mappings (testName, orderedAt, fileUrl)
  • Improved search/filter handling

Doctor Patient Detail

  • Removed hardcoded mockMedicalHistory and mockLabs
  • States now initialize as []
  • Data loads via:
    • api.labResults.getByPatient(id)
    • api.medicalRecords.getByPatient(id)
  • Updated components to support both API and legacy fields

Nurse Module

Patient Details

  • Removed all mock data
  • Integrated APIs for assigned patients, vitals, and prescriptions
  • Real vitals display and medication schedule
    Vitals Entry
  • Removed mock vitals history
  • Chart and history now load from api.vitalSigns.getByPatient()
  • Replaced hardcoded "Nurse Joy" with real nurse name
    Vitals Page
  • Removed (mock) labels from toast messages

Access Fix

  • Updated PatientAccessValidator to allow NURSE and LAB_TECHNICIAN to access patient vitals

Nurse Tasks (New Feature)

  • Added Create Task functionality
  • Backend: POST /api/nurse/tasks
  • Service method createTask() added
  • Frontend modal form with validation and API integration

UI / Component Fixes

  • Fixed invalid HTML in Table.jsx (<thead> now wraps headers in <tr>)
  • Removed placeholder Add Task button from Nurse Dashboard

@ManvithaDungi ManvithaDungi self-assigned this Mar 10, 2026
@ManvithaDungi ManvithaDungi merged commit 576cb84 into main Mar 11, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants