Skip to content
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

[16.0][MIG] account_invoice_overdue_reminder #269

Merged

Conversation

alexis-via
Copy link
Contributor

Migrated the module with several small improvements/fixes:

alexis-via and others added 30 commits February 28, 2023 12:03
…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>
…e customer invoices

Move "Send overdue reminder" button to the alter banner.
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/
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/
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/
JasminSForgeFlow and others added 8 commits February 28, 2023 12:03
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
Copy link

@Bearnard21 Bearnard21 left a comment

Choose a reason for hiding this comment

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

Functional review

Copy link

@Bearnard21 Bearnard21 left a comment

Choose a reason for hiding this comment

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

Functional review
Tested:
Invoice Reminder Results edited
E-mail template edited all automated strings working properly
Settings adjustment edited and affected
Search filters by Partner, by Overdue counter
Activity as overdue action scheduled
Letter printed
E-mail sent (attachments present)
"Disabled overdue reminder" functioning
All looks good.

@KKamaa
Copy link

KKamaa commented Mar 8, 2023

Looks good!

@alexis-via
Copy link
Contributor Author

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 16.0-ocabot-merge-pr-269-by-alexis-via-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit da87083 into OCA:16.0 Mar 8, 2023
@OCA-git-bot
Copy link
Contributor

@alexis-via The merge process could not be finalized, because command git push origin 16.0-ocabot-merge-pr-269-by-alexis-via-bump-nobump:16.0 failed with output:

To https://github.com/OCA/credit-control
 ! [rejected]        16.0-ocabot-merge-pr-269-by-alexis-via-bump-nobump -> 16.0 (fetch first)
error: failed to push some refs to 'https://***@github.com/OCA/credit-control'
hint: Updates were rejected because the remote contains work that you do
hint: not have locally. This is usually caused by another repository pushing
hint: to the same ref. You may want to first integrate the remote changes
hint: (e.g., 'git pull ...') before pushing again.
hint: See the 'Note about fast-forwards' in 'git push --help' for details.

@OCA-git-bot
Copy link
Contributor

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

@max3903 max3903 added this to the 16.0 milestone Jul 20, 2023
@max3903
Copy link
Sponsor Member

max3903 commented Jul 20, 2023

/ocabot migration account_invoice_overdue_reminder

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.

None yet