Skip to content

[IMP] pos_cash_move_reason : add functionnalities on POS UI#1531

Open
quentinDupont wants to merge 2 commits into
OCA:16.0from
grap:16.0-IMP-pos_cash_move_reason-on_UI-QD
Open

[IMP] pos_cash_move_reason : add functionnalities on POS UI#1531
quentinDupont wants to merge 2 commits into
OCA:16.0from
grap:16.0-IMP-pos_cash_move_reason-on_UI-QD

Conversation

@quentinDupont

@quentinDupont quentinDupont commented Apr 21, 2026

Copy link
Copy Markdown

This PR adds the functionnalitiess of pos_cash_move_reason on POS UI : handling Moves for all Payment Methods, see README.

It handles move reason for multiple journal, that's why in Pos Close PopUp you can now see Moves for theses journals.
That's why this module adds button to launch Cash In/Out PopUp from Close PopUp + improve Close PopUp design with CSS.

image image

@OCA-git-bot OCA-git-bot added mod:pos_cash_move_reason Module pos_cash_move_reason series:16.0 labels Apr 21, 2026
@quentinDupont quentinDupont force-pushed the 16.0-IMP-pos_cash_move_reason-on_UI-QD branch from ab19477 to 5c39f93 Compare April 22, 2026 08:21

@flotho flotho left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hi @quentinDupont ,

great thanks for your contribution.
Could you adapt your commit mesage and PR name to follow guidelines https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst#git

@quentinDupont quentinDupont force-pushed the 16.0-IMP-pos_cash_move_reason-on_UI-QD branch from 5c39f93 to 14625ac Compare April 23, 2026 13:59
@quentinDupont quentinDupont changed the title [IMP] Module works in POS UI [IMP] pos_cash_move_reason : add functionnalities on POS UI Apr 23, 2026
@quentinDupont

quentinDupont commented May 21, 2026

Copy link
Copy Markdown
Author

❌ Need to be fix : Odoo only handles Cash movement in UI so move of other journal are counted in Cash journal → i'm on it

Edit : fix ✔️

@quentinDupont quentinDupont marked this pull request as draft May 22, 2026 14:40
@quentinDupont quentinDupont force-pushed the 16.0-IMP-pos_cash_move_reason-on_UI-QD branch from 0128933 to ba1df2e Compare May 25, 2026 10:31
@quentinDupont quentinDupont marked this pull request as ready for review May 25, 2026 10:31
… move in and out for multiple journals

[IMP] filter for having only pos_session payment methods journal [IMP] workflow move reason and close popup works
@quentinDupont quentinDupont force-pushed the 16.0-IMP-pos_cash_move_reason-on_UI-QD branch from ba1df2e to a879a4c Compare May 25, 2026 10:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

mod:pos_cash_move_reason Module pos_cash_move_reason series:16.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants