Skip to content

[18.0][MIG] pos_early_receipt_printing - #1566

Open
Anxo82 wants to merge 6 commits into
OCA:18.0from
sygel-technology:18.0-mig-pos_early_receipt_printing
Open

[18.0][MIG] pos_early_receipt_printing#1566
Anxo82 wants to merge 6 commits into
OCA:18.0from
sygel-technology:18.0-mig-pos_early_receipt_printing

Conversation

@Anxo82

@Anxo82 Anxo82 commented Jun 5, 2026

Copy link
Copy Markdown

@OCA-git-bot OCA-git-bot added series:18.0 mod:pos_early_receipt_printing Module pos_early_receipt_printing labels Jun 5, 2026
@Anxo82
Anxo82 marked this pull request as draft June 5, 2026 12:02
@Anxo82
Anxo82 force-pushed the 18.0-mig-pos_early_receipt_printing branch from 023e187 to 313dcec Compare June 8, 2026 06:27

@Jaimermaccione Jaimermaccione 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.

@Anxo82 Reviewed in a local environment.

Tests performed

  1. POS technical configuration

    • It was verified that the Early Receipt Printing option is available and configurable in the POS settings for Shop type points of sale (not restaurants), under the Bills & Receipts section.
    • It was also verified that it can be configured directly from the point of sale configuration through Point of Sale → Dashboard → (⋮) → Edit, by enabling Early Receipt Printing.
  2. POS interface verification

    • It was confirmed that the Bill button is correctly displayed in the control buttons area, under the Actions menu.
    • It was verified that the button is correctly enabled or disabled depending on whether the current order contains product lines.
    • When clicking the Bill button, the system displays the receipt preview and printing screen before proceeding to payment.

The README correctly describes the configuration steps and usage flow.

LGTM! Thank you!

@ValentinVinagre ValentinVinagre 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.

LGTM 👍🏻

@Anxo82
Anxo82 marked this pull request as ready for review July 22, 2026 15:17
@Anxo82

Anxo82 commented Jul 22, 2026

Copy link
Copy Markdown
Author

@legalsylvain could you please review this PR when you have a chance?
Thanks!

@OCA-git-bot

Copy link
Copy Markdown
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants