Skip to content

Added Admin Dashboard Functions and Fixed Error Messages and Bugs#382

Open
ys2426 wants to merge 1 commit into
Gopher-Industries:masterfrom
ys2426:ys2426/admin-dashboard
Open

Added Admin Dashboard Functions and Fixed Error Messages and Bugs#382
ys2426 wants to merge 1 commit into
Gopher-Industries:masterfrom
ys2426:ys2426/admin-dashboard

Conversation

@ys2426
Copy link
Copy Markdown
Collaborator

@ys2426 ys2426 commented Mar 31, 2026

Added Sign out confirmation message; Added New Patient forms fields error message; Fixed Previous button functionality in forms; Implemented Home & Sign out button functions in Sidebar; Removed Triangular Nav buttons with same functionalities as Next/Prev buttons for all pages"

Description

Added Several Additions and few changes to the Admin Dashboard and Pages in Admin module.

List of changes

  • Added a Dialog box asking whether to sign out or stay in when sign out button is clicked (Admin Homepage or Sidebar)
  • Fixed a bug where not all the mandatory fields in the new patient detail forms do not show 'field empty' warnings.
  • Fixed an issue in New Patient detail forms where the "Previous" button does not function when any of the mandatory fields are empty in all the forms
  • Removed a duplicate functionality - Triangular Nav buttons functioning the same as Next/Prev buttons
  • Implemented functionalities for Sidebar buttons - Home and Sign out, for all pages.

Screenshots and/or Gifs

Screenshot 2026-03-31 at 10 17 21 PM Screenshot 2026-03-31 at 10 17 42 PM Screenshot 2026-03-31 at 10 18 34 PM Screenshot 2026-03-31 at 10 19 07 PM

…rror message; Fixed Previous button functionality in forms; Implemented Home & Sign out button functions in Sidebar; Removed Triangular Nav buttons with same functionalities as Next/Prev buttons for all pages
Copy link
Copy Markdown
Collaborator

@iHateErrorsSoMuch iHateErrorsSoMuch left a comment

Choose a reason for hiding this comment

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

_I’ve reviewed the changes and the implementation aligns well with the PR description. The updates to validation, navigation, and sidebar functionality all look consistent and well-structured.

The fixes for mandatory field validation and the Previous button behavior appear to be handled properly, and removing the duplicate triangular navigation helps simplify the UI logic. The addition of the sign-out confirmation dialog and sidebar functionality also looks clean and consistent across pages._

My review is based on code inspection only, as I wasn’t able to run the project locally yet. I didn’t notice any obvious issues in the changes, and overall the code is readable and consistent.

Happy to approve 👍

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.

3 participants