Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[12.0] [MIG] hr_payroll_cancel #533

Merged
merged 11 commits into from
Feb 4, 2019

Conversation

nikul-serpentcs
Copy link
Member

Hr Payroll Cancel

This module allows the user to cancel a payslip whatever the previous state is
without doing a refund. When the user cancel the journal entry is deleted
and the payslip state is set to rejected. Then the user is able to set the
state to draft again and later on he/she is able to confirm again the payslip.

If there’s a refund for a payslip the user should not cancel the entry because
the refund would still be confirm. In that case, the user have either to
confirm again the payslip or cancel the refund.

@nikul-serpentcs nikul-serpentcs mentioned this pull request Dec 22, 2018
32 tasks
@pedrobaeza pedrobaeza added this to the 12.0 milestone Dec 22, 2018
Copy link

@bjeficent bjeficent left a comment

Choose a reason for hiding this comment

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

LGTM. Tested on runbot.

@anandkansagra
Copy link
Member

@nikul-serpentcs ,

Need to manage codecov

@nikul-serpentcs
Copy link
Member Author

@anandkansagra Done!

@nikul-serpentcs
Copy link
Member Author

@aheficent @pedrobaeza Can you please review & move forward

@anandkansagra
Copy link
Member

Still needs to cover more
1

@anandkansagra
Copy link
Member

Still needs to cover more

LGTM @nikul-serpentcs

@nikul-serpentcs
Copy link
Member Author

@aheficent @pedrobaeza Can you please review & move forward

@OCA-git-bot
Copy link
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@pedrobaeza pedrobaeza merged commit 7e04f2f into OCA:12.0 Feb 4, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet