Skip to content

[MIG] partner_statement: Migration to 18.0 #1284

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

Open
wants to merge 63 commits into
base: 18.0
Choose a base branch
from

Conversation

kobros-tech
Copy link

@kobros-tech kobros-tech commented Feb 11, 2025

this version is more efficient than #1285

gdgellatly and others added 30 commits February 11, 2025 21:08
Before this change, if a payment was reconciled using the bank statement widget
the move was seperated into many multiples of lines making activity
statements twice as long as necessary.

Now payments are a single line
By default we checked for an empty data dict in _get_report_values
however the mail template sends a file type and editor key by default.

We now check if the required company_id is in the dictionary. We also support
supplying force_company in context for setting the company for auto generated
emails.

[UPD] README.rst

[UPD] Update partner_statement.pot

Update translation files

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

Translation: account-financial-reporting-12.0/account-financial-reporting-12.0-partner_statement
Translate-URL: https://translation.odoo-community.org/projects/account-financial-reporting-12-0/account-financial-reporting-12-0-partner_statement/
In certain scenarios involving mutliple payments for multiple
invoices, where the reconciliation happens together and the debit
move is the smaller item the aging duplicates its value, making
the balance greater than it should be.

This fix works by preventing moves where different lines are both partial
reconciled being grouped together by adding the lines id
to the group by clause of _show_buckets_sql_q1
Translated using Weblate (Portuguese (Brazil))

Currently translated at 17.7% (14 of 79 strings)

Translation: account-financial-reporting-12.0/account-financial-reporting-12.0-partner_statement
Translate-URL: https://translation.odoo-community.org/projects/account-financial-reporting-12-0/account-financial-reporting-12-0-partner_statement/pt_BR/
…ash tests with other modules like as password_security
* Add files via upload

Display only posted entries

* Update activity_statement.py

* Update report_statement_common.py

[UPD] Update partner_statement.pot

Update translation files

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

Translation: account-financial-reporting-12.0/account-financial-reporting-12.0-partner_statement
Translate-URL: https://translation.odoo-community.org/projects/account-financial-reporting-12-0/account-financial-reporting-12-0-partner_statement/
[UPD] Update partner_statement.pot

[UPD] README.rst

Update translation files

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

Translation: account-financial-reporting-13.0/account-financial-reporting-13.0-partner_statement
Translate-URL: https://translation.odoo-community.org/projects/account-financial-reporting-13-0/account-financial-reporting-13-0-partner_statement/
* show partner name at the top.
* simplify view logic for `name` and `ref` printing.
* fallback to journal entry `ref` when the journal item has no
  `ref` set.
* little query formatting changes to enhance readability.

[UPD] Update partner_statement.pot

[UPD] README.rst

partner_statement 13.0.1.1.0

Update translation files

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

Translation: account-financial-reporting-13.0/account-financial-reporting-13.0-partner_statement
Translate-URL: https://translation.odoo-community.org/projects/account-financial-reporting-13-0/account-financial-reporting-13-0-partner_statement/
Detected cases where account_partial_reconcile has null currency_id, thus giving falsy amount_currency, and thus making some reconciled invoices appearing in the outstanding. Used _amount_residual of account move lines as reference to fix it.

partner_statement 13.0.1.2.1
- Added html and xlsx support for activity outgoing statement
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: account-financial-reporting-14.0/account-financial-reporting-14.0-partner_statement
Translate-URL: https://translation.odoo-community.org/projects/account-financial-reporting-14-0/account-financial-reporting-14-0-partner_statement/
weblate and others added 22 commits February 11, 2025 21:08
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: account-financial-reporting-15.0/account-financial-reporting-15.0-partner_statement
Translate-URL: https://translation.odoo-community.org/projects/account-financial-reporting-15-0/account-financial-reporting-15-0-partner_statement/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: account-financial-reporting-15.0/account-financial-reporting-15.0-partner_statement
Translate-URL: https://translation.odoo-community.org/projects/account-financial-reporting-15-0/account-financial-reporting-15-0-partner_statement/
Balance does not change in a blocked line but still needs to be set
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: account-financial-reporting-17.0/account-financial-reporting-17.0-partner_statement
Translate-URL: https://translation.odoo-community.org/projects/account-financial-reporting-17-0/account-financial-reporting-17-0-partner_statement/
Currently translated at 100.0% (107 of 107 strings)

Translation: account-financial-reporting-17.0/account-financial-reporting-17.0-partner_statement
Translate-URL: https://translation.odoo-community.org/projects/account-financial-reporting-17-0/account-financial-reporting-17-0-partner_statement/it/
Currently translated at 100.0% (107 of 107 strings)

Translation: account-financial-reporting-17.0/account-financial-reporting-17.0-partner_statement
Translate-URL: https://translation.odoo-community.org/projects/account-financial-reporting-17-0/account-financial-reporting-17-0-partner_statement/it/
Currently translated at 100.0% (107 of 107 strings)

Translation: account-financial-reporting-17.0/account-financial-reporting-17.0-partner_statement
Translate-URL: https://translation.odoo-community.org/projects/account-financial-reporting-17-0/account-financial-reporting-17-0-partner_statement/it/
Currently translated at 100.0% (107 of 107 strings)

Translation: account-financial-reporting-17.0/account-financial-reporting-17.0-partner_statement
Translate-URL: https://translation.odoo-community.org/projects/account-financial-reporting-17-0/account-financial-reporting-17-0-partner_statement/it/
The date shown on the report should be based on the client's timezone not UTC. (can be off by a day)

(cherry picked from commit 07b5969)
@kobros-tech kobros-tech force-pushed the 18.0-mig-partner_statement branch from c584130 to 1e21a5e Compare February 11, 2025 21:40
Copy link
Contributor

@dannyadair dannyadair left a comment

Choose a reason for hiding this comment

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

Source LGTM, some documentation maybe outdated.
Took it for a spin and appears to be working as expected.

Copy link
Contributor

Choose a reason for hiding this comment

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

In Application / Accounting & Finance, select Billing or Billing Manager

I believe this may be inaccurate / different between Community and Enterprise. Check the boxes for "Use activity statements" and "Use outstanding statements" or pick an accounting role that includes them ("Invoicing").

Copy link
Contributor

@dannyadair dannyadair Mar 19, 2025

Choose a reason for hiding this comment

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

In Application / Accounting & Finance, select Billing or Billing Manager

I believe this may be inaccurate / different between Community and Enterprise. Check the boxes for "Use activity statements" and "Use outstanding statements" or pick an accounting role that includes them ("Invoicing").

@dannyadair
Copy link
Contributor

I'd like to note that v18 Enterprise has account_reports which includes a "Customer Statement" web view with PDF/XLSX export (see screenshots). Not as many features, maybe this should be split with an Enterprise-compatible version adding features? As it stands I think a user can get confused with multiple ways of obtaining a statement.
Screenshot_20250320_125524
Screenshot_20250320_125543
Screenshot_20250320_125610

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.