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

[MIG] [11.0] hr_payslip_change_state #504

Closed
wants to merge 6 commits into from

Conversation

Beji94
Copy link

@Beji94 Beji94 commented Nov 6, 2018

Change module version and some changes in the view file.

AaronHForgeFlow and others added 5 commits November 2, 2018 11:51
* [ADD] hr_payslip_change_state

* [FIX] Readme was incorrect

* [FIX] Comments at ForgeFlow/eficent-odoo-addons#18

* [FIX] Change texts in wizard as they are in form buttons

* [FIX] style

*  [FIX]Migrate to new api, add test and fix style

* [ADD] travis configuration

*  [FIX]  typo in travis file

*  [FIX] style

*  [FIX] style

*  [FIX] style

*  [FIX] travis file

* Revert " [FIX] travis file"

This reverts commit db79685.

*  [FIX] travis file

* Revert " [FIX] travis file"

This reverts commit f243159.

*  [DEL] travis file

*  [FIX] travis comments for  commit e0138a5

*  [FIX] Comments by @andhit-r on pull request OCA#249

* [ADD] hr_payslip_change_state

* [FIX] Readme was incorrect

* [FIX] Comments at ForgeFlow/eficent-odoo-addons#18

* [FIX] Change texts in wizard as they are in form buttons

* [FIX] style

*  [FIX]Migrate to new api, add test and fix style

*  [FIX] style

*  [FIX] style

*  [FIX] style

*  [FIX] travis comments for  commit e0138a5

*  [FIX] Comments by @andhit-r on pull request OCA#249

* Fix travis. Rebase upstream. Fix unittest

*  [UPT] Add contributor. Edit description in readme.

*  [FIX] class name
@pedrobaeza pedrobaeza added this to the 11.0 milestone Nov 6, 2018
@pedrobaeza pedrobaeza mentioned this pull request Nov 6, 2018
71 tasks
Copy link

@AaronHForgeFlow AaronHForgeFlow left a comment

Choose a reason for hiding this comment

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

👍 Functional test

Copy link
Member

@nikul-serpentcs nikul-serpentcs left a comment

Choose a reason for hiding this comment

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

Minor Change, Code review LGTM

hr_payslip_change_state/__init__.py Outdated Show resolved Hide resolved
@Beji94 Beji94 force-pushed the 11.0-mig-hr_payslip_change_state branch from 8e28bec to 44f0da7 Compare November 6, 2018 11:22
@bjeficent bjeficent force-pushed the 11.0-mig-hr_payslip_change_state branch from 44f0da7 to f20d47b Compare November 8, 2018 09:55
@oca-clabot
Copy link

Hey @Beji94, thank you for your Pull Request.

It looks like some users haven't signed our Contributor License Agreement, yet.
You can read and sign our full Contributor License Agreement here: http://odoo-community.org/page/cla
Here is a list of the users:

Appreciation of efforts,
OCA CLAbot

@Beji94 Beji94 closed this Nov 8, 2018
@LoisRForgeFlow LoisRForgeFlow deleted the 11.0-mig-hr_payslip_change_state branch November 8, 2018 15:56
@oca-clabot
Copy link

Hey @Beji94,
We acknowledge that the following users have signed our Contributor License Agreement:

Appreciation of efforts,
OCA CLAbot

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

10 participants