Skip to content

[18.0][MIG] account_move_reconcile_helper#889

Open
BhaveshHeliconia wants to merge 13 commits intoOCA:18.0from
HeliconiaIO:18.0-mig-account_move_reconcile_helper
Open

[18.0][MIG] account_move_reconcile_helper#889
BhaveshHeliconia wants to merge 13 commits intoOCA:18.0from
HeliconiaIO:18.0-mig-account_move_reconcile_helper

Conversation

@BhaveshHeliconia
Copy link
Contributor

@BhaveshHeliconia BhaveshHeliconia commented Jul 22, 2025

No description provided.

@BhaveshHeliconia BhaveshHeliconia force-pushed the 18.0-mig-account_move_reconcile_helper branch from 0bac7af to e2fa218 Compare July 22, 2025 08:39
carlos-lopez-tecnativa
carlos-lopez-tecnativa suggested changes Oct 20, 2025
View reviewed changes
account_move_reconcile_helper/views/account_move_line.xml Outdated
<field name="inherit_id" ref="account.view_move_line_tax_audit_tree" />
<field name="arch" type="xml">
<field name="move_id" position="after">
<field name="matching_number" invisible="1" />
Copy link
Contributor

@carlos-lopez-tecnativa carlos-lopez-tecnativa Oct 20, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is this field necessary?
If it’s used only in a domain or for invisible/readonly/required attributes, I think this field can be removed. Please review.
Or, if it’s necessary, then in the list view you should change invisible to column_invisible

Suggested change
<field name="matching_number" invisible="1" />
<field name="matching_number" column_invisible="1" />

@BhaveshHeliconia BhaveshHeliconia force-pushed the 18.0-mig-account_move_reconcile_helper branch from e2fa218 to ea18b35 Compare October 27, 2025 04:40
@BhaveshHeliconia
Copy link
Contributor Author

BhaveshHeliconia commented Oct 27, 2025

@carlos-lopez-tecnativa Removed that field as it’s not needed. Thanks!

carlos-lopez-tecnativa
carlos-lopez-tecnativa approved these changes Oct 27, 2025
View reviewed changes
Copy link
Contributor

@carlos-lopez-tecnativa carlos-lopez-tecnativa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM
TT58449

@pedrobaeza @sergio-teruel could you please review this?

@pedrobaeza
Copy link
Member

pedrobaeza commented Oct 27, 2025

/ocabot migration account_move_reconcile_helper

@OCA-git-bot OCA-git-bot added this to the 18.0 milestone Oct 27, 2025
@OCA-git-bot OCA-git-bot mentioned this pull request Oct 27, 2025
Open
10 tasks
Reyes4711-S73
Reyes4711-S73 approved these changes Nov 6, 2025
View reviewed changes
Copy link

@Reyes4711-S73 Reyes4711-S73 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Reyes4711-S73
Copy link

Reyes4711-S73 commented Nov 6, 2025

@OCA/accounting-maintainers Please, can you merge this PR?

@OCA-git-bot
Copy link
Contributor

OCA-git-bot commented Nov 6, 2025

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@BhaveshHeliconia
Copy link
Contributor Author

BhaveshHeliconia commented Jan 23, 2026

@OCA/accounting-maintainers Please, can you merge this PR?

@DorianMAG
Copy link

DorianMAG commented Jan 27, 2026

@BhaveshHeliconia
Hi, thx for this work.
Can you rebase plz?
Regards

@BhaveshHeliconia BhaveshHeliconia force-pushed the 18.0-mig-account_move_reconcile_helper branch from ea18b35 to 1ea0873 Compare January 28, 2026 05:07
@BhaveshHeliconia BhaveshHeliconia mentioned this pull request Jan 30, 2026
Merged
benwillig and others added 13 commits February 20, 2026 11:04
@benwillig @BhaveshHeliconia
[ADD] account_move_reconcile_helper
2e2883c
@BhaveshHeliconia
[12.0][MIG] account_move_reconcile_helper
19e9a53
@badbole @BhaveshHeliconia
Translated using Weblate (Croatian)
a6aaa03 
Currently translated at 100.0% (3 of 3 strings)

Translation: account-reconcile-12.0/account-reconcile-12.0-account_move_reconcile_helper
Translate-URL: https://translation.odoo-community.org/projects/account-reconcile-12-0/account-reconcile-12-0-account_move_reconcile_helper/hr/
@kittiu @BhaveshHeliconia
[13.0][IMP] account_move_reconcile_helper: black, isort
f3b23b5
@kittiu @BhaveshHeliconia
[13.0][MIG] account_move_reconcile_helper
91919f3
@hildickethan @BhaveshHeliconia
[14.0][MIG] account_move_reconcile_helper: Migration to 14.0
0984318
@xavier-bouquiaux @BhaveshHeliconia
[14.0][FIX] account_move_reconcile_helper
42b7477 
change action call
@ofonseca-pyming @BhaveshHeliconia
Translated using Weblate (Spanish)
ce391b4 
Currently translated at 100.0% (3 of 3 strings)

Translation: account-reconcile-14.0/account-reconcile-14.0-account_move_reconcile_helper
Translate-URL: https://translation.odoo-community.org/projects/account-reconcile-14-0/account-reconcile-14-0-account_move_reconcile_helper/es/
@ferran-S73 @BhaveshHeliconia
[15.0][MIG] account_move_reconcile_helper
1bff3b9
@mpascuall @BhaveshHeliconia
[MIG] account_move_reconcile_helper: Migration to 16.0
f02e1c3
@mymage @BhaveshHeliconia
Added translation using Weblate (Italian)
b380785
@mymage @BhaveshHeliconia
Translated using Weblate (Italian)
fb9441c 
Currently translated at 100.0% (2 of 2 strings)

Translation: account-reconcile-16.0/account-reconcile-16.0-account_move_reconcile_helper
Translate-URL: https://translation.odoo-community.org/projects/account-reconcile-16-0/account-reconcile-16-0-account_move_reconcile_helper/it/
@BhaveshHeliconia
[MIG] account_move_reconcile_helper: Migration to 18.0
11e8d38
@BhaveshHeliconia BhaveshHeliconia force-pushed the 18.0-mig-account_move_reconcile_helper branch from 1ea0873 to 11e8d38 Compare February 20, 2026 05:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

2 more reviewers

@Reyes4711-S73 Reyes4711-S73 Reyes4711-S73 approved these changes

@carlos-lopez-tecnativa carlos-lopez-tecnativa carlos-lopez-tecnativa approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

approved ready to merge

Projects

None yet

Milestone

18.0

Development

Successfully merging this pull request may close these issues.

Comments