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

[9.0] port hr_worked_days_hourly_rate #170

Closed
wants to merge 1 commit into from
Closed

Conversation

kossovo
Copy link

@kossovo kossovo commented Oct 19, 2015

This module is compatible with Odoo 9.0 version.
It's installable and all unit tests pass without error.

sorry for my english ^.^

@max3903 max3903 assigned ghost Oct 19, 2015
@max3903 max3903 added this to the 9.0 milestone Oct 19, 2015
@dreispt
Copy link
Sponsor Member

dreispt commented Oct 19, 2015

Let's make it a little more chalenging 😄
Could you please fix the lint messages: https://travis-ci.org/OCA/hr/jobs/86236300#L168

@pedrobaeza pedrobaeza mentioned this pull request Oct 19, 2015
65 tasks
@gurneyalex gurneyalex changed the title Make module installable [9.0] port hr_worked_days_hourly_rate Oct 20, 2015
@gurneyalex
Copy link
Member

@kossovo I updated the title of the PR to something more meaningful when reading the list of open PRs

@charbeljc
Copy link

I second @dreispt's chalenge. @kossovo, as this module is small and simple could you take the time to port it to the new api (ie, getting rid of openerp.osv for model definition and use openerp.models and openerp.fields) ?

@ghost
Copy link

ghost commented Oct 21, 2015

👎 Please, wait to get a v8 api version of this module. If we merge this now, no one will take the time to port it.

@feketemihai
Copy link
Member

👎 For me the same, i think we should wait the merge of #188 and port it after...

@pedrobaeza pedrobaeza closed this Jul 5, 2016
@pedrobaeza pedrobaeza reopened this Jul 5, 2016
@pedrobaeza pedrobaeza closed this Jul 5, 2016
Mraimou pushed a commit to camptocamp/hr that referenced this pull request Nov 25, 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

7 participants