Skip to content

Implemented patient logs module for nurse dashboard#416

Merged
KudratAroraa merged 2 commits into
masterfrom
deven/patient-logs-module-for-nurse-dashboard
May 28, 2026
Merged

Implemented patient logs module for nurse dashboard#416
KudratAroraa merged 2 commits into
masterfrom
deven/patient-logs-module-for-nurse-dashboard

Conversation

@iHateErrorsSoMuch
Copy link
Copy Markdown
Collaborator

@iHateErrorsSoMuch iHateErrorsSoMuch commented May 17, 2026

Description

Implemented the Patient Logs module for the Nurse Dashboard.

This feature allows nurses to:

  • Open the Patient Logs module directly from the dashboard
  • Select from assigned patients using a dropdown
  • View patient-specific logs
  • Create new patient logs with validation
  • Delete logs with confirmation dialogs
  • Automatically refresh logs after creating or deleting entries
  • Handle loading, empty, and error states properly

Additional UI improvements were also added, including:

  • Improved toolbar/navigation
  • Better log card styling
  • Formatted date and time display
  • Updated Patient Logs dashboard icon
  • Read-only live date/time display on the add log screen

Backend API integration was completed for:

  • Fetching patient logs
  • Creating patient logs
  • Deleting patient logs

Todos

  • Tested and working locally
  • Code follows the style guidelines of this project
  • I have performed a self-review of my code
  • Code changes documented
  • Requested review from >= 2 devs on the team (one frontend and one backend recommended)

How to test

  1. Login using a nurse account
  2. Open the Nurse Dashboard
  3. Click on the “Patient Logs” module
  4. Select a patient from the dropdown
  5. Verify that logs for the selected patient are displayed
  6. Click “Add Log”
  7. Enter log details and save
  8. Verify the new log appears in the list automatically
  9. Delete a log using the delete button
  10. Verify confirmation dialog appears and log is removed successfully
  11. Test switching between patients and verify logs refresh correctly
  12. Verify empty state message appears for patients without logs

Screenshots and/or Gifs

  • Added screenshots showing:

    • Nurse Dashboard with Patient Logs module
    image
    • Patient Logs screen
image
  • Add Patient Log screen
image
  • Log creation and deletion flow
image image image

Associated MS Planner Tasks

Known Issues

  • Date/time displayed on the add log screen is read-only and backend still controls the final created timestamp.
  • Some backend validation/error responses may vary depending on API availability.

@KudratAroraa KudratAroraa self-assigned this May 28, 2026
Copy link
Copy Markdown
Collaborator

@KudratAroraa KudratAroraa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved.

Patient Logs module looks aligned with the intended nurse dashboard scope. The flow for selecting a patient, viewing logs, creating new logs, deleting logs with confirmation, and refreshing the list is structured clearly, and the add-log implementation is now cleanly focused on patient log creation with validation, loading states. The UI looks good. The nurse dashboard navigation update and patient logs screen additions also fit the feature scope well.

Overall, this is a meaningful module addition and looks good to merge.

@KudratAroraa
Copy link
Copy Markdown
Collaborator

Applied spotless formatting for CI

@KudratAroraa KudratAroraa merged commit 8c758a0 into master May 28, 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

Development

Successfully merging this pull request may close these issues.

2 participants