[18.0] [MIG] pos_report_session_summary: Migration to 18.0#1411
[18.0] [MIG] pos_report_session_summary: Migration to 18.0#1411OCA-git-bot merged 22 commits intoOCA:18.0from
Conversation
P-H-Phuc
force-pushed
the
18.0-mig-pos_report_session_summary
branch
from
67770fc to
e31864d
Compare
|
There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. |
github-actions
bot
added
the
stale
P-H-Phuc
force-pushed
the
18.0-mig-pos_report_session_summary
branch
from
e31864d to
436c280
Compare
github-actions
bot
removed
the
stale
|
/ocabot migration pos_report_session_summary Is this ready to be reviewed? |
cristina-hidalgo-tecnativa
left a comment
cristina-hidalgo-tecnativa
left a comment
There was a problem hiding this comment.
Tested locally and on runboat, It installs and works correctly.
It seems the CI failures are not caused by this module. A rebase should fix the main CI error.
I would also suggest rebasing the bot commits from the history.
LGTM.
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
force-pushed
the
18.0-mig-pos_report_session_summary
branch
from
436c280 to
f0b9a3e
Compare
|
/ocabot merge nobump |
|
On my way to merge this fine PR! |
|
Congratulations, your PR was merged at 7357219. Thanks a lot for contributing to OCA. ❤️ |
14 participants
Since Odoo 18.0, the field
cash_register_total_entry_encodingfield inpos.sessionis no longer used, as noted in this PR. Therefore, I have removed theTotal Transactionscolumn from the report template.E.g.