Skip to content

[18.0][ADD] pos_payment_cash_change#13

Draft
P-H-Phuc wants to merge 5 commits into
18.0from
18.0-mig-pos_payment_change
Draft

[18.0][ADD] pos_payment_cash_change#13
P-H-Phuc wants to merge 5 commits into
18.0from
18.0-mig-pos_payment_change

Conversation

@P-H-Phuc

@P-H-Phuc P-H-Phuc commented Jan 12, 2026

Copy link
Copy Markdown
Owner

This module was initially part of the Foodcoop set of addons. We're now migrating it to 18.0 and taking this occasion to contribute it to OCA.

Changes

  • Remove account_bank_statement.py and accont_bank_statement_line.py: Since v14.0, deprecated button_confirm_bank().
  • Create account.move.line (Put cash in + give change records) when pos.order is paid.

Configuration

image

Result

image

NOTE

  • Journal Entries Put Cash In / Give Change is created when select To Invoince in POS Payment Screen

  • Help me check Account / Credit / Debit amount is correct

@nguyenminhchien nguyenminhchien left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

The module feature did not meet the requirement:
image

class AccountJournal(models.Model):
_inherit = "account.journal"

change_account_id = fields.Many2one(

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

No need this field anymore, change_account_id of pos.payment.method is enough

@P-H-Phuc P-H-Phuc force-pushed the 18.0-mig-pos_payment_change branch 2 times, most recently from 05a211a to 64d22c0 Compare January 26, 2026 10:38

@nguyenminhchien nguyenminhchien left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

  • Sell an order which amount = $ 1,799.00. Customer pays exact amount by cash (no change).
  • Take money out by amount of $ 10,799.84
image

@nguyenminhchien

nguyenminhchien commented Jan 27, 2026

Copy link
Copy Markdown

@P-H-Phuc
Note that, this module will allow us to manage cash in the cashbox by declaring an account for cashbox, for example, 1015012 - Cashbox.

  • Money receives from customer will NOT be in cash account 101501 Cash anymore but in 1015012 - Cashbox. You have to create new journal entry to move the cash from 101501 Cash to 1015012 - Cashbox then.
  • Take money in the cashobx: cash will move to 1015012 - Cashbox instead of 101501 Cash.
  • Take money out the cashbox: cash will move from 1015012 - Cashbox instead of 101501 Cash.

@nguyenminhchien

Copy link
Copy Markdown

I created this PR to replace this one. Please help me review.

@P-H-Phuc P-H-Phuc force-pushed the 18.0-mig-pos_payment_change branch from 64d22c0 to 3eeed1f Compare February 3, 2026 11:34
@P-H-Phuc P-H-Phuc marked this pull request as draft February 5, 2026 10:26
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.

2 participants