[12.0][MIG] account_move_reconcile_helper #246
Conversation
|
I don't know why runbot fail, account_balance_line is migrated to 12.0. Can anyone help me? |
|
You have to add |
|
Oh! Sorry, I miss that! Now it's done and 🍏! cc @benwillig @Jortolsa @fuentes010 @ioans73 @Saphery @Willerr-mo |
|
|
||
| <odoo> | ||
|
|
||
| <record model="ir.ui.view" id="account_move_line_tree_view"> |
There was a problem hiding this comment.
aesthetic change:
id before model
use original id
In the case where an inherited view has a name which does not follow the guidelines set above, prefer naming the inherited view after the original over using a name which follows the guidelines. This eases looking up the original view and other inheritance if they all have the same name.
not approve relevant
| 'data': [ | ||
| 'views/account_move_line.xml', | ||
| ], | ||
| 'demo': [ |
There was a problem hiding this comment.
if you dont use this tag, just delete it
Abranes
force-pushed
the
12.0-mig-account_move_reconcile_helper
branch
from
154cd9f
to
2eb2096
Compare
|
@tbaden Squash and minor changes done! |
|
@pedrobaeza Can you review this? |
|
Merging blindly as it seems there are enough reviewers. /ocabot merge |
|
Hey, thanks for contributing! Proceeding to merge this for you. |
|
Congratulations, your PR was merged at 024232f. Thanks a lot for contributing to OCA. ❤️ PS: Don't worry if GitHub says there are unmerged commits: it is due to a rebase before merge. All commits of this PR have been merged into |
Standard migration