-
-
Notifications
You must be signed in to change notification settings - Fork 785
[17.0][OU-ADD] hr_expense: migration to 17.0 #4472
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][OU-ADD] hr_expense: migration to 17.0 #4472
Conversation
|
/ocabot migration hr_expense |
|
|
||
| @openupgrade.migrate() | ||
| def migrate(env, version): | ||
| openupgrade.load_data(env, "hr_expense", "17.0.2.0/noupdate_changes.xml") |
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.
delete template translations
| # NOTHING TO DO: dead code https://github.com/odoo/odoo/pull/110518 | ||
|
|
||
| hr_expense / hr.expense / price_unit (float) : NEW required, isfunction: function, stored | ||
| # DONE pre-migration: rename to unit_price |
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.
why?
| # NOTHING TO DO: deprecated field https://github.com/odoo/odoo/pull/130244/commits/266b482264576903cf736ffe2bddfc71f7a7a4ec | ||
|
|
||
| hr_expense / hr.expense / state (selection) : selection_keys is now '['approved', 'done', 'draft', 'refused', 'reported', 'submitted']' ('['approved', 'done', 'draft', 'refused', 'reported']') | ||
| # DONE pre-migration: if any expense has expense sheet and the sheet is on draft state, then update state of expense to 'reported' |
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.
why? the change only adds new 'submitted' state
|
Superseded by #4826. |
No description provided.