Skip to content

[18.0] [MIG] pos_report_session_summary: Migration to 18.0#1

Open
P-H-Phuc wants to merge 22 commits into18.0from
18.0-mig-pos_report_session_summary
Open

[18.0] [MIG] pos_report_session_summary: Migration to 18.0#1
P-H-Phuc wants to merge 22 commits into18.0from
18.0-mig-pos_report_session_summary

Conversation

@P-H-Phuc
Copy link
Copy Markdown
Owner

@P-H-Phuc P-H-Phuc commented Jul 8, 2025

Since Odoo 18.0, the field cash_register_total_entry_encoding field in pos.session is no longer used, as noted in this PR. Therefore, I have removed the Total Transactions column from the report template.
E.g.

  • In version 16.0:

pos_report_session_summary_odoo16

  • In version 18.0:
image

@P-H-Phuc P-H-Phuc self-assigned this Jul 8, 2025
@P-H-Phuc P-H-Phuc force-pushed the 18.0-mig-pos_report_session_summary branch 2 times, most recently from 3dcc4f0 to 67770fc Compare July 8, 2025 08:40
Copy link
Copy Markdown

@thienvh332 thienvh332 left a comment

Choose a reason for hiding this comment

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

A side-by-side result comparison in the PR description might just win the reviewer’s heart (and their approval)! 😄

V16:

image

V18:

image

Comment on lines +1 to +2
- [Trobz](https://www.trobz.com):
- Phan Hong Phuc \<phucph@trobz.com\> No newline at end of file
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Just kidding: I'm seriously concerned about this growing trend of cuckoo-ism! 😆

Add past contributors too. I think they forgot that.

@P-H-Phuc P-H-Phuc force-pushed the 18.0-mig-pos_report_session_summary branch from 67770fc to e31864d Compare July 9, 2025 08:06
@P-H-Phuc P-H-Phuc force-pushed the 18.0-mig-pos_report_session_summary branch from e31864d to 436c280 Compare February 24, 2026 04:02
PierrickBrun and others added 22 commits March 25, 2026 10:17
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: pos-15.0/pos-15.0-pos_report_session_summary
Translate-URL: https://translation.odoo-community.org/projects/pos-15-0/pos-15-0-pos_report_session_summary/
… 'str' object has no attribute 'decimal_places' when printing session summaryossible to print the report. This pr fixes the error: 'str' object has no attribute 'decimal_places' when printing session summary
@P-H-Phuc P-H-Phuc force-pushed the 18.0-mig-pos_report_session_summary branch from 436c280 to f0b9a3e Compare March 25, 2026 03:17
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.