Skip to content

Update hourly salary calculation#655

Merged
niraj2477 merged 6 commits intodevelopfrom
refactor/hourly-rate
Jun 25, 2025
Merged

Update hourly salary calculation#655
niraj2477 merged 6 commits intodevelopfrom
refactor/hourly-rate

Conversation

@niraj2477
Copy link
Copy Markdown
Collaborator

@niraj2477 niraj2477 commented Jun 24, 2025

Description

This PR refactors the hourly salary calculation logic across the codebase to use a common method that accounts for each employee's work schedule. It also removes an unused custom field from the Employee DocType.

Relevant Technical Choices

  • Update common method to work based on the employee schedule.

Testing Instructions

Additional Information:

Screenshot/Screencast

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)

@niraj2477 niraj2477 requested a review from zeel-codder June 24, 2025 12:38
@niraj2477 niraj2477 merged commit b24018e into develop Jun 25, 2025
4 checks passed
@niraj2477 niraj2477 deleted the refactor/hourly-rate branch June 25, 2025 04:56
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.

2 participants