Skip to content

Fix hours mismatch in timesheet list after project change.#651

Merged
zeel-codder merged 51 commits intomainfrom
develop
Jun 23, 2025
Merged

Fix hours mismatch in timesheet list after project change.#651
zeel-codder merged 51 commits intomainfrom
develop

Conversation

@Siddhantsingh1230
Copy link
Copy Markdown
Collaborator

Description

This PR fixes hours mismatch when changing timesheet project.

Relevant Technical Choices

  • Fix key issues in map function

Testing Instructions

  • Go to NPMS.
  • Open Team detail page for an employee.
  • Edit any existing Timesheet.
  • Hours should not mismatch for changed timesheet.

Additional Information:

N/A

Screenshot/Screencast

Fix.Edit.Timesheet.webm

Checklist

  • I have carefully reviewed the code before submitting it for review.
  • This code is adequately covered by unit tests to validate its functionality.
  • I have conducted thorough testing to ensure it functions as intended.
  • A member of the QA team has reviewed and tested this PR (To be checked by QA or code reviewer)

maharrshi and others added 30 commits June 13, 2025 15:30
PR check related changes taken from TC93 branch
Also remove conditionals for get branch run
THe whole section has condition already in place dont have to check again
fix : upated error logging
@Siddhantsingh1230
Copy link
Copy Markdown
Collaborator Author

@niraj2477 Please update description for related changes

Add patch to remove custom fields from project dt
@zeel-codder zeel-codder merged commit 120566b into main Jun 23, 2025
7 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants