feat: update department handling in invoices and reports to support multiple selections #56

Merged
addison merged 2 commits from 005-002-005-performance-report-department-filter into main 2026-09-21 07:25:41 -04:00
Collaborator

Spec 5.2.5

Changes

  • Added functionality of multiple department select in performance report page both frontend and backend.
  • Updated Sales report page code a bit too because both were using same filters

Testing

  • Make sure you have latest code running.
  • Navigate to Performance report page and try adding multiple departments using filter dropdown.
  • Selected departments data must be filtered. Now try printing performance report page, filters of departments must be reflected there too.
  • Now redirect to Sales report page. It must work like before.
# Spec 5.2.5 ## Changes - Added functionality of multiple department select in performance report page both frontend and backend. - Updated Sales report page code a bit too because both were using same filters ## Testing - [x] Make sure you have latest code running. - [x] Navigate to Performance report page and try adding multiple departments using filter dropdown. - [x] Selected departments data must be filtered. Now try printing performance report page, filters of departments must be reflected there too. - [x] Now redirect to Sales report page. It must work like before.
feat: update department handling in invoices and reports to support multiple selections
All checks were successful
CI / frontend (pull_request) Successful in 43s
CI / backend (pull_request) Successful in 35s
Amp Review / review (pull_request) Successful in 2s
6b108c9351
talha requested review from addison 2026-09-20 07:11:40 -04:00
Owner

On the print page it should list the departments

image

On the print page it should list the departments ![image](/attachments/33738224-58b0-4391-b4d9-9220b3f4b311)
3.1 KiB
addison left a comment

Fix the print page

Fix the print page
feat: update department filter summary to display selected departments as a comma-separated list
All checks were successful
CI / frontend (pull_request) Successful in 39s
CI / backend (pull_request) Successful in 35s
Amp Review / review (pull_request) Successful in 1s
3e8cb835a8
Author
Collaborator

@addison wrote in #56 (comment):

Fix the print page

Updated

@addison wrote in https://git.kwila.cloud/kwila/office-extensions/pulls/56#issuecomment-4969: > Fix the print page Updated
talha requested review from addison 2026-09-21 04:56:41 -04:00
addison deleted branch 005-002-005-performance-report-department-filter 2026-09-21 07:25:41 -04:00
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
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!56
No description provided.