kwila/office-extensions
Watch
1
0
Fork
You've already forked office-extensions
 0
Code Issues Pull requests Projects Releases Packages Activity Actions

feat(dashboard): added annual total sales by employee #59

Merged
addison merged 4 commits from 007-000-001-annual-total-sales-by-employee into main 2026-09-30 04:04:37 -04:00
Conversation 1 Commits 4 Files changed 11 +419 -18
talha commented 2026-09-25 11:04:51 -04:00
Collaborator
Copy link

Spec 7.0.1

Changes

  • Added backend endpoint to give annual sales by employees.
  • Added chart for annual sales by employees.

Testing

  • Make sure you have latest code running.
  • Navigate to Sales dashboard.
  • A full height and half screen bar chart must be displayed for each employee total sales in descending order based on sales.
  • Hover over any employee bar, it must show employee name and total sales till this day in year.
# Spec 7.0.1 ## Changes - Added backend endpoint to give annual sales by employees. - Added chart for annual sales by employees. ## Testing - [x] Make sure you have latest code running. - [x] Navigate to Sales dashboard. - [x] A full height and half screen bar chart must be displayed for each employee total sales in descending order based on sales. - [x] Hover over any employee bar, it must show employee name and total sales till this day in year.
feat(dashboard): added employee sales aggregation and chart components
Some checks failed
CI / frontend (pull_request) Failing after 26s
Details
CI / backend (pull_request) Has been skipped
Details
02a659f642
fix: fixed prettier issue
All checks were successful
CI / frontend (pull_request) Successful in 38s
Details
CI / backend (pull_request) Successful in 34s
Details
Amp Review / review (pull_request) Successful in 1s
Details
d22f8e77bf
talha requested review from addison 2026-09-25 11:12:18 -04:00
addison requested changes 2026-09-29 04:53:20 -04:00
addison left a comment
Copy link

Two small UI tweaks. Looking pretty good!

Two small UI tweaks. Looking pretty good!
frontend/src/components/dashboards/sales/EmployeeSalesChart.tsx Outdated
@ -0,0 +47,4 @@
return (
<div className="flex h-full min-h-0 flex-1 flex-col">
<h2 className="mb-2 flex-shrink-0 text-lg font-semibold text-gray-900">
Annual Total Sales by Employee ({year})
addison commented 2026-09-29 04:52:17 -04:00
Owner
Copy link

No need to include the year in the title.

No need to include the year in the title.
addison marked this conversation as resolved
frontend/src/components/dashboards/sales/SalesDashboardPage.tsx Outdated
@ -25,3 +30,1 @@
report && <DepartmentSalesChart months={report.months} />
)}
</Card>
<div className="grid min-h-0 flex-1 grid-cols-1 gap-4 md:grid-cols-2 md:gap-6">
addison commented 2026-09-29 04:53:02 -04:00
Owner
Copy link

I would prefer to see the charts stacked one above the other rather than side by side.

I would prefer to see the charts stacked one above the other rather than side by side.
addison marked this conversation as resolved
feat(dashboard): update EmployeeSalesChart title and improve SalesDashboardPage layout
Some checks failed
CI / frontend (pull_request) Failing after 24s
Details
CI / backend (pull_request) Has been skipped
Details
a3bb0c1571
refactor(dashboard): remove year prop from EmployeeSalesChart and SalesDashboardPage
All checks were successful
CI / frontend (pull_request) Successful in 38s
Details
CI / backend (pull_request) Successful in 36s
Details
481fd91a93
talha requested review from addison 2026-09-29 07:31:26 -04:00
addison deleted branch 007-000-001-annual-total-sales-by-employee 2026-09-30 04:04:38 -04:00
Sign in to join this conversation.
Reviewers
No reviewers
addison
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
kwila/office-extensions!59
No description provided.