Skip to content

Comments

SaiKrisna add labor hours distribution backend#2004

Merged
one-community merged 2 commits intodevelopmentfrom
SaiKrishna_DistributionOfLaborHoursBackend
Feb 21, 2026
Merged

SaiKrisna add labor hours distribution backend#2004
one-community merged 2 commits intodevelopmentfrom
SaiKrishna_DistributionOfLaborHoursBackend

Conversation

@saikrishnaraoyadagiri
Copy link
Contributor

Description

image image

Related PRS (if any):

Main changes explained:

  • Created laborHoursDistributionController.js
  • Created laborHours.js
  • Create laborHoursDistributionRouter.js
  • Updated routes.js

How to test:

  1. check into current branch
  2. do npm install and ... to run this PR locally
  3. Clear site data/cache
  4. Go to postman to test the backend
  5. hit the url to get the data : http://localhost:4500/api/labor-hours/distribution?start_date=2024-01-18&end_date=2024-12-31
  6. it should display the data according to the dates selected from the laborhours collection
  7. validate the data in collection and data showing in postman, it should be same
  8. validate the data with optional field of category

Screenshots or videos of changes:

image image

@saikrishnaraoyadagiri saikrishnaraoyadagiri changed the title feat: add labor hours distribution backend SaiKrisna_add labor hours distribution backend Jan 14, 2026
@one-community one-community added the High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible label Feb 18, 2026
Copy link

@Anusha-Gali Anusha-Gali left a comment

Choose a reason for hiding this comment

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

Hi Sai Krishna,

I have reviewed your PR locally though the requests work as per requirement, i tried using invalid month in start date as you can see below and the API call is accepting and showing the response.

2024-13-18: No 13th month
Image

Working screenshots
Image
Image
Image
Image

@saikrishnaraoyadagiri saikrishnaraoyadagiri changed the title SaiKrisna_add labor hours distribution backend SaiKrisna add labor hours distribution backend Feb 19, 2026
@saikrishnaraoyadagiri
Copy link
Contributor Author

@Anusha-Gali , Invalid date is fixed, please review
Screenshot 2026-02-20 004911

Copy link

@Anusha-Gali Anusha-Gali left a comment

Choose a reason for hiding this comment

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

Hi Sai Krishna,

Thank you for fixing the issue.

Image Image

@one-community
Copy link
Member

Thank you all, merging!

@one-community one-community merged commit 5e699a6 into development Feb 21, 2026
2 checks passed
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.

3 participants