Skip to content

fix: added specialization and remove doctor toggle

Rajuldev Vandana requested to merge fix/doctor-toggle into fix/develop

Overview

This MR removes the Doctor filter from all queue screens and fixes a duplication issue in the Verify Medicines page where replacement medicine details were displayed twice.

What does this MR do and why?

  • The Doctor filter was present in all queues but is not required for the current workflow, making the UI unnecessarily complex. It has been removed to simplify filtering and improve usability.

Changes Made

  • Removed Doctor filter from:
    • Vitals Waiting Queue
    • Vitals Queue
    • Consultation Queue
  • Updated UI layout after removal for consistency

Technical Details

  • Root Cause (Bug):
    Duplicate JSX rendering for replacement details caused the same information to appear twice.

  • Fix:
    Removed redundant rendering block and unified data into a single structured line.

  • No backend changes required.

Type of Change

  • 🐛 Bug fix (non-breaking change that fixes an issue)
  • ✨ New feature (non-breaking change that adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to change)
  • 📝 Documentation update
  • 🎨 UI/UX improvement
  • ♻️ Refactor (no functional changes)
  • ⚡ Performance improvement
  • 🧪 Test update
  • 🔧 Configuration change
  • 🚨 Security fix

Related Issues / References

Screenshots or Screen Recordings

Before After
Doctor filter visible in queues Doctor filter removed

How to Set Up and Validate Locally

  1. Pull this branch:

    git checkout <branch-name>
git pull origin <branch-name>

  2. Install dependencies:

    bun install

  3. Run development server:

    bun dev

  4. Navigate:

    • Vitals Waiting Queue
    • Vitals Queue
    • Consultation Queue
    • Verify Medicines page

  5. Verify:

    • Doctor filter is not visible
    • Specialization and Gender filters work correctly
    • Replacement medicine details appear only once

Testing Done

  • Manual testing completed
  • Unit tests added/updated

Test Cases Covered:

Scenario Expected Result Status
Remove doctor filter Filter not visible ✅
UI layout alignment Proper spacing maintained ✅
Filters working Correct filtering applied ✅

Code Quality Checklist

Code Standards

  • Code follows project conventions
  • No console.log() or debugger statements
  • No unused imports or variables
  • No duplicate code
  • ESLint and Prettier checks pass

React Best Practices

  • Components properly structured
  • Hooks used correctly
  • No unnecessary re-renders

Component Patterns

  • Tailwind used properly
  • UI consistent across screens

API & Data Fetching

  • No API changes required

Error Handling

  • No new errors introduced

Documentation

  • README.md updated
  • .env.example updated
  • CHANGELOG.md updated

Known Limitations / Technical Debt

  • None

Additional Notes

  • This is a frontend-only change
  • No backend dependency

MR Acceptance Checklist

Evaluate this MR against the acceptance checklist.

Edited by Rajuldev Vandana

Merge request reports