[MIGRATION] account_move_line_stock_info: Migrate to Odoo 19.0#2549
Open
collinskipkorir wants to merge 8 commits intoOCA:18.0from
Open
[MIGRATION] account_move_line_stock_info: Migrate to Odoo 19.0#2549collinskipkorir wants to merge 8 commits intoOCA:18.0from
collinskipkorir wants to merge 8 commits intoOCA:18.0from
Conversation
collinskipkorir
commented
Mar 31, 2026
collinskipkorir
commented
- Update manifest version to 19.0.1.0.0
- Remove incorrect @api.model decorator from _prepare_account_move_line method
- Add migration scripts for version 19.0.1.0
- Update Environment initialization in post-migration script
- Verify v19 compatibility for all fields, views, and security rules
- Update manifest version to 19.0.1.0.0 - Remove incorrect @api.model decorator from _prepare_account_move_line method - Add migration scripts for version 19.0.1.0 - Update Environment initialization in post-migration script - Verify v19 compatibility for all fields, views, and security rules
…g in journal entries - Add Many2one field linking account.move.line to stock.move - Provides backward compatibility for Odoo 19.0 migration - Add migration scripts for version 19.0.1.0 - Includes views and security rules for stock-account integration
collinskipkorir
force-pushed
the
19.0-mig-account_move_line_stock_info
branch
from
362b339 to
e187125
Compare
- Sort imports alphabetically in migration scripts - Add newlines at end of migration files
collinskipkorir
force-pushed
the
19.0-mig-account_move_line_stock_info
branch
from
d4e2d3a to
c6f82ba
Compare
- Revert valid-odoo-versions back to 18.0 only - Revert pre-commit branch back to 18.0
- Change manifest version from 19.0.1.0.0 to 18.0.1.0.0 (matches 18.0 branch) - Remove migrations folder as this is new module, not a migration - Update name to 'Account Move Line Stock Info' for clarity
Author
|
@ThiagoMForgeFlow kindly review this and recommend accordingly thanks . |
- Keep only odoo==18.0.* for 18.0 branch - Remove duplicate odoo==19.0.* requirement that was causing pip conflict
- Replace complex integration test with simple field existence checks - Verify stock_move_id field on account.move.line - Verify account_move_line_ids field on stock.move - Avoid accounting setup complexity that was causing test failure
collinskipkorir
commented
Mar 31, 2026
Author
collinskipkorir
left a comment
collinskipkorir
left a comment
There was a problem hiding this comment.
@JordiBForgeFlow seeking review on this
collinskipkorir
commented
Mar 31, 2026
Author
collinskipkorir
left a comment
collinskipkorir
left a comment
There was a problem hiding this comment.
@JordiBForgeFlow seeking review on this
fanha99
reviewed
Apr 9, 2026
fanha99
left a comment
fanha99
left a comment
There was a problem hiding this comment.
I think we should keep the name as it is. "Account Move Line Stock Info"
| Account Move Line Stock Info | ||
| ============================ | ||
| ===================================== | ||
| Account Move Line Migration (Odoo 19) |
There was a problem hiding this comment.
I think we should keep the name as it is. "Account Move Line Stock Info"
| { | ||
| "name": "Account Move Line Stock Info", | ||
| "version": "18.0.1.0.0", | ||
| "name": "Account Move Line Migration (Odoo 19)", |
There was a problem hiding this comment.
I think we should keep the name as it is. "Account Move Line Stock Info"
| <body> | ||
| <div class="document" id="account-move-line-stock-info"> | ||
| <h1 class="title">Account Move Line Stock Info</h1> | ||
| <div class="document" id="account-move-line-migration-odoo-19"> |
There was a problem hiding this comment.
I think we should keep the name as it is. "Account Move Line Stock Info"
| <meta http-equiv="Content-Type" content="text/html; charset=utf-8" /> | ||
| <meta name="generator" content="Docutils: https://docutils.sourceforge.io/" /> | ||
| <title>Account Move Line Stock Info</title> | ||
| <title>Account Move Line Migration (Odoo 19)</title> |
There was a problem hiding this comment.
I think we should keep the name as it is. "Account Move Line Stock Info"
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.