Skip to content

[17.0][MIG] account_invoice_overdue_reminder: backport migration to 17.0#523

Merged
OCA-git-bot merged 51 commits intoOCA:17.0from
NICO-SOLUTIONS:17.0-mig-bp-account_invoice_overdue_reminder
Mar 17, 2026
Merged

[17.0][MIG] account_invoice_overdue_reminder: backport migration to 17.0#523
OCA-git-bot merged 51 commits intoOCA:17.0from
NICO-SOLUTIONS:17.0-mig-bp-account_invoice_overdue_reminder

Conversation

@NICO-SOLUTIONS
Copy link
Copy Markdown
Member

backport migration to 17.0

alexis-via and others added 30 commits November 23, 2025 11:59
…update your mail templates)

Add ability to add contacts as Cc of the reminder email (added to the Cc of the mail template)
Add partner_policy with 3 options to give some choice about which contact should be selected to send reminders
Access reminders from partner via Action menu
Advantages:
1) easy access to the history in the chatter of the partner.
2) no problem to access the email by other users (avoid "The requested
operation cannot be completed due to security restrictions. Please
contact your system administrator.")
Co-authored-by: ypapouin <y.papouin@dec-industrie.com>
Co-authored-by: ypapouin <y.papouin@dec-industrie.com>
[UPD] Update account_invoice_overdue_reminder.pot

[UPD] README.rst

[ADD] icon.png
…e customer invoices

Move "Send overdue reminder" button to the alter banner.

[UPD] Update account_invoice_overdue_reminder.pot

account_invoice_overdue_reminder 14.0.1.1.0

Update translation files

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: credit-control-14.0/credit-control-14.0-account_invoice_overdue_reminder
Translate-URL: https://translation.odoo-community.org/projects/credit-control-14-0/credit-control-14-0-account_invoice_overdue_reminder/
[UPD] Update account_invoice_overdue_reminder.pot

Update translation files

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: credit-control-14.0/credit-control-14.0-account_invoice_overdue_reminder
Translate-URL: https://translation.odoo-community.org/projects/credit-control-14-0/credit-control-14-0-account_invoice_overdue_reminder/
Translated using Weblate (French)

Currently translated at 98.9% (185 of 187 strings)

Translation: credit-control-14.0/credit-control-14.0-account_invoice_overdue_reminder
Translate-URL: https://translation.odoo-community.org/projects/credit-control-14-0/credit-control-14-0-account_invoice_overdue_reminder/fr/
account_invoice_overdue_reminder 14.0.1.1.1
…achment_ids

[UPD] Update account_invoice_overdue_reminder.pot

[UPD] README.rst

Update translation files

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: credit-control-15.0/credit-control-15.0-account_invoice_overdue_reminder
Translate-URL: https://translation.odoo-community.org/projects/credit-control-15-0/credit-control-15-0-account_invoice_overdue_reminder/
result_notes fields converted from fields.Text to fields.Html
In mass view of steps, show buttons in tree view
Fix carriage return in subject of email, which may block the email
Fix crash when validating several steps at the same time in the mass update interface
Forword port PR OCA#236 and OCA#240

[UPD] Update account_invoice_overdue_reminder.pot

[UPD] README.rst

Update translation files

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: credit-control-16.0/credit-control-16.0-account_invoice_overdue_reminder
Translate-URL: https://translation.odoo-community.org/projects/credit-control-16-0/credit-control-16-0-account_invoice_overdue_reminder/
Remove the end step in the wizard (replaced by a notification)
Remove the up_to_date boolean on the start wizard
Code cleanup

[UPD] Update account_invoice_overdue_reminder.pot

account_invoice_overdue_reminder 16.0.1.1.0

Update translation files

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: credit-control-16.0/credit-control-16.0-account_invoice_overdue_reminder
Translate-URL: https://translation.odoo-community.org/projects/credit-control-16-0/credit-control-16-0-account_invoice_overdue_reminder/

Update __manifest__.py

"application": False,
Translated using Weblate (Dutch)

Currently translated at 93.4% (170 of 182 strings)

Translation: credit-control-16.0/credit-control-16.0-account_invoice_overdue_reminder
Translate-URL: https://translation.odoo-community.org/projects/credit-control-16-0/credit-control-16-0-account_invoice_overdue_reminder/nl/

[UPD] README.rst
Translated using Weblate (Spanish)

Currently translated at 78.5% (143 of 182 strings)

Translation: credit-control-16.0/credit-control-16.0-account_invoice_overdue_reminder
Translate-URL: https://translation.odoo-community.org/projects/credit-control-16-0/credit-control-16-0-account_invoice_overdue_reminder/es/

Translated using Weblate (Spanish)

Currently translated at 100.0% (182 of 182 strings)

Translation: credit-control-16.0/credit-control-16.0-account_invoice_overdue_reminder
Translate-URL: https://translation.odoo-community.org/projects/credit-control-16-0/credit-control-16-0-account_invoice_overdue_reminder/es/
Update menu sequence, to have "space" between the previous menu entry
which is account.menu_action_account_payments_receivable with
sequence=15, to be able to insert another menu in-between
Translated using Weblate (Portuguese)

Currently translated at 96.1% (175 of 182 strings)

Translation: credit-control-16.0/credit-control-16.0-account_invoice_overdue_reminder
Translate-URL: https://translation.odoo-community.org/projects/credit-control-16-0/credit-control-16-0-account_invoice_overdue_reminder/pt/

Translated using Weblate (Portuguese)

Currently translated at 100.0% (182 of 182 strings)

Translation: credit-control-16.0/credit-control-16.0-account_invoice_overdue_reminder
Translate-URL: https://translation.odoo-community.org/projects/credit-control-16-0/credit-control-16-0-account_invoice_overdue_reminder/pt/

Update translation files

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: credit-control-16.0/credit-control-16.0-account_invoice_overdue_reminder
Translate-URL: https://translation.odoo-community.org/projects/credit-control-16-0/credit-control-16-0-account_invoice_overdue_reminder/

Translated using Weblate (Dutch)

Currently translated at 93.4% (170 of 182 strings)

Translation: credit-control-16.0/credit-control-16.0-account_invoice_overdue_reminder
Translate-URL: https://translation.odoo-community.org/projects/credit-control-16-0/credit-control-16-0-account_invoice_overdue_reminder/nl/
@thomich
Copy link
Copy Markdown

thomich commented Jan 29, 2026

hello together

is there a time plan, when this is merged into 17.0?

with kind regards
thomi

@rrebollo
Copy link
Copy Markdown

is there a time plan, when this is merged into 17.0?

You could contribute by providing a review on it.

@thomich
Copy link
Copy Markdown

thomich commented Feb 3, 2026

Must fix

  • License mismatch: __manifest__.py = AGPL-3, hooks.py header = LGPL-3.0 → align.
  • Duplicate assignment in wizard/overdue_reminder_wizard.py (action = action = (...)).

Should fix

  • Typo in security/ir.model.access.csv: overdue.reminter.result.

Usage

  • Module installs cleanly without any manual fixes.
  • User flow manually tested (wizard, sending, history/chatter).
  • No obvious runtime errors observed.
  • Review findings clearly identified (2 must-fix, 1 should-fix).

Status

  • LGTM after fixes.

@thomich
Copy link
Copy Markdown

thomich commented Mar 11, 2026

Is there any plan to merge this PR?

Copy link
Copy Markdown

@adasatorres adasatorres left a comment

Choose a reason for hiding this comment

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

Hi @NICO-SOLUTIONS , when you have a moment, could you review the suggestions? Thanks.

…duplicate assignment

- Align license header in hooks.py with module license (AGPL-3)
- Fix typo in security/ir.model.access.csv (overdue.reminder.result)
- Remove duplicate assignment in overdue_reminder_wizard.py
@NICO-SOLUTIONS
Copy link
Copy Markdown
Member Author

@adasatorres
@thomich
I've applied the suggested changes from @thomich.

As mentioned earlier in the discussion, the remaining suggestions from @rebello should be discussed with the original author of the module. We should also make sure that the changes are forward-ported to v18.

@thomich
Copy link
Copy Markdown

thomich commented Mar 12, 2026

@adasatorres @thomich I've applied the suggested changes from @thomich.

As mentioned earlier in the discussion, the remaining suggestions from @rebello should be discussed with the original author of the module. We should also make sure that the changes are forward-ported to v18.

thanks for applying the changes +1.

about discussion remaining suggestions from @rrebollo means, it will take time to merge this PR? just ask, cause our customers are waiting long time for this.

Copy link
Copy Markdown

@adasatorres adasatorres left a comment

Choose a reason for hiding this comment

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

LGTM

@OCA-git-bot
Copy link
Copy Markdown
Contributor

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). 🤖

@NICO-SOLUTIONS
Copy link
Copy Markdown
Member Author

@thomich
From my POV this is ready for getting merged. But this needs to be done by a maintainer. You can download the module from the source PR and implement it on your customer site if you are in urgent need, until this is finished.

I would Like to ask @rrebollo to add his changes in a separate PR when this is merged.

Thanks to all!

@adasatorres
Copy link
Copy Markdown

@OCA/accounting-maintainers, this PR is ready to be merged. Would it be possible for someone to review it and merge it? Thanks.

@adasatorres
Copy link
Copy Markdown

@jelenapoblet, could you please merge this PR?

@jelenapoblet
Copy link
Copy Markdown

/ocabot merge nobump

@OCA-git-bot
Copy link
Copy Markdown
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 17.0-ocabot-merge-pr-523-by-jelenapoblet-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 6f511fb into OCA:17.0 Mar 17, 2026
7 checks passed
@OCA-git-bot
Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at 2cfbeef. Thanks a lot for contributing to OCA. ❤️

@NICO-SOLUTIONS
Copy link
Copy Markdown
Member Author

@jelenapoblet
Thx

@HaraldPanten
Copy link
Copy Markdown

/ocabot migration account_invoice_overdue_reminder

@OCA-git-bot OCA-git-bot added this to the 17.0 milestone Mar 17, 2026
@OCA-git-bot OCA-git-bot mentioned this pull request Mar 17, 2026
11 tasks
@NICO-SOLUTIONS NICO-SOLUTIONS deleted the 17.0-mig-bp-account_invoice_overdue_reminder branch March 18, 2026 09:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.