-
-
Notifications
You must be signed in to change notification settings - Fork 195
[14.0][MIG] maintenance_stock: Migration to 14.0 #347
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
base: 14.0
Are you sure you want to change the base?
Conversation
|
Hello @Reyes4711-S73 your PR is based on #337 , isn't it? Please check, because that PR has new adjustments since you've cloned it |
edb1746 to
78728b3
Compare
|
@dalonsod I added your last changes in my migration PR |
78728b3 to
f20543b
Compare
|
Please check errors in tests |
f20543b to
bc0a774
Compare
| </function> | ||
| <function model="stock.inventory" name="action_validate"> | ||
| <function | ||
| eval="[[('state','=','draft'),('id', '=', ref('maintenance_stock.stock_inventory_toner'))]]" |
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.
The inventory for demo product is In Progress when installing the module:

Perhaps it is this line that does not allow validating the inventory. Maybe the state at this moment is confirm (not tested).
| eval="[[('state','=','draft'),('id', '=', ref('maintenance_stock.stock_inventory_toner'))]]" | |
| eval="[[('state','=','confirm'),('id', '=', ref('maintenance_stock.stock_inventory_toner'))]]" |
Can you review please ?
|
@Reyes4711-S73 I've still found some issues in commit history:
|
c9e4180 to
3ee7028
Compare
mamcode
left a comment
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.
Hello @Reyes4711-S73 Thanks for the applied fixes.
Only a comment about the value used for field delivery_steps in stock.warehouse can you review please.
3ee7028 to
117ae55
Compare
|
@mamcode Change done |
mamcode
left a comment
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.
Hello @Reyes4711-S73 thanks, but there are still some smaller details to fix.
74ce106 to
3c453fe
Compare
|
@mamcode done |
mamcode
left a comment
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.
Thanks @Reyes4711-S73
|
@dalonsod Can you check this PR again please? |
dalonsod
left a comment
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.
Tested in runboat 👍 but see at code review required changes
9efae1f to
b573f37
Compare
|
@dalonsod @Reyes4711-S73 Hello, I see errors in the Odoo test job that seem to have nothing to do with the code of this PR, how can we run the jobs again? |
b573f37 to
3ab39ac
Compare
|
Hello @mamcode , it's a known issue, please check odoo/odoo#122569 (comment) |
07b3541 to
22bdf26
Compare
|
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. |
Currently translated at 96.7% (29 of 30 strings) Translation: maintenance-12.0/maintenance-12.0-maintenance_stock Translate-URL: https://translation.odoo-community.org/projects/maintenance-12-0/maintenance-12-0-maintenance_stock/es/
Currently translated at 100.0% (30 of 30 strings) Translation: maintenance-12.0/maintenance-12.0-maintenance_stock Translate-URL: https://translation.odoo-community.org/projects/maintenance-12-0/maintenance-12-0-maintenance_stock/pt_BR/
Currently translated at 100.0% (30 of 30 strings) Translation: maintenance-12.0/maintenance-12.0-maintenance_stock Translate-URL: https://translation.odoo-community.org/projects/maintenance-12-0/maintenance-12-0-maintenance_stock/it/
Currently translated at 100.0% (30 of 30 strings) Translation: maintenance-12.0/maintenance-12.0-maintenance_stock Translate-URL: https://translation.odoo-community.org/projects/maintenance-12-0/maintenance-12-0-maintenance_stock/it/
c5ebbea to
b72ea82
Compare
|
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. |
b72ea82 to
13863ea
Compare
|
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. |
13863ea to
4e91dd0
Compare
Standard migration to 14.0