Skip to content

Conversation

@Amalesh-A
Copy link
Contributor

@Amalesh-A Amalesh-A commented Nov 2, 2025

Description

image

Related PRS (if any):

This backend PR is related to the #4320 frontend PR.
To test this backend PR you need to checkout the #4320 frontend PR.

Main changes explained:

  • Added Weekly Progress section to the Summary Dashboard with title + date range subtitle.
  • Implemented 4 KPI tiles sourced from GET /api/tasks/summary:
    • Total Tasks
    • Tasks Completed This Week
    • Open Tasks
    • Average Completion Time (days)
  • Implemented Weekly Progress line chart (Recharts/Chart.js) sourced from GET /api/tasks/trends:
    • X-axis: week start dates (auto-adjusts to 4/8/12)
    • Y-axis: tasks completed (gridlines enabled)
    • Tooltip on hover (week + completed count)
    • Legend shows a single series: “This Week”
  • Added filters:
    • Start/End date pickers (default: last 8 weeks ending today)
    • Preset buttons: Last 4 Weeks / Last 8 Weeks / Last 12 Weeks
  • Added validations:
    • start <= end
    • date range cannot exceed 12 weeks (84 days) (frontend warning + backend 400)
  • Added Project Status Bar to the dashboard layout.

How to test:

  1. check into current branch: git checkout Amalesh-WeeklyProgress-LineChart
  2. do npm install and ... to run this PR locally
  3. Clear site data/cache
  4. log as an user
  5. navigate to: http://localhost:3000/weeklyprogress
  6. Verify KPI tiles:
  • Total Tasks / Tasks Completed This Week / Open Tasks / Avg Completion Time (days)
  • Confirm responsive behavior (row on desktop, stacked on mobile)
  1. Verify Weekly Progress chart:
  • Title “Weekly Progress” is centered
  • Tooltip shows week + tasks completed on hover
  • Legend shows only “This Week”

Screenshots or videos of changes:

LineChartWeeklyProgress.mp4

@Amalesh-A Amalesh-A added the Do Not Review Do not review or look at code without full context label Nov 2, 2025
@Amalesh-A Amalesh-A added High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible and removed Do Not Review Do not review or look at code without full context labels Jan 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants