-
-
Notifications
You must be signed in to change notification settings - Fork 110
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
[17.0][ADD] hr_holidays_remaining_leaves #139
base: 17.0
Are you sure you want to change the base?
[17.0][ADD] hr_holidays_remaining_leaves #139
Conversation
[hr_holidays_remaining_leaves] migrate
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- README.md file shouldn't be touched.
- The commit history contains other modules. Please squash story to reduce it and remove that noise.
- I insist the name should be singular in the extra bits.
hr_holidays_employee_remaining
should serve.
Hi @pedrobaeza
|
It was just a suggestion. What I try is to avoid the plurals, and let me tell you why the guideline about singular forms is there: this comes from the past (6.0, 6.1) where the module names where a total nightmare of mix of plurals, company names, etc, and it was very easy to misspell and commit typos while putting that names in the command line, in automation files, etc. A recurrent one is the |
There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. |
The original development of this module has been done by Mint System.
It can be found in: https://github.com/Mint-System/Odoo-Apps-HR/tree/16.0/hr_holidays_remaining_leaves
This module has been ported to the OCA with their agreement, cc @janikvonrotz
Supersedes: #127