Skip to content

[16.0][ADD] pos_hide_receipt_line#1404

Closed
adasatorres wants to merge 1 commit into
OCA:16.0from
BinhexTeam:16.0-add-pos-hide_receipt_line
Closed

[16.0][ADD] pos_hide_receipt_line#1404
adasatorres wants to merge 1 commit into
OCA:16.0from
BinhexTeam:16.0-add-pos-hide_receipt_line

Conversation

@adasatorres

Copy link
Copy Markdown

Good morning, this add-on allows you to hide product lines on the point of sale receipt, as long as the line's price is 0.

To do this, go to the product in the backend and enable the Hide Receipt Line field, which is only visible if the product is available in the point of sale. After that, simply go to the point of sale and create a line with the product at a price of 0.

@adasatorres
adasatorres force-pushed the 16.0-add-pos-hide_receipt_line branch 5 times, most recently from d19bf98 to ef1d2f7 Compare July 1, 2025 11:01

@antoniodavid antoniodavid left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@edescalona edescalona left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@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). 🤖

@adasatorres
adasatorres force-pushed the 16.0-add-pos-hide_receipt_line branch from ef1d2f7 to 6507164 Compare July 21, 2025 08:49
@adasatorres
adasatorres force-pushed the 16.0-add-pos-hide_receipt_line branch from 6507164 to d6ae78e Compare October 21, 2025 09:55

@ivantodorovich ivantodorovich left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for your contribution!

I just have a comment regarding the unit tests

)

def test_hide_receipt_line(self):
self.assertTrue(self.product.pos_hide_receipt_line)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This test doesn't add any value. It's just testing the ORM, and we don't have to do that.

What would be useful is to test this module with a js tour that makes sure the receipt line is not actually displayed.

@github-actions

Copy link
Copy Markdown

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.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions Bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Feb 22, 2026
@github-actions github-actions Bot closed this Mar 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved ready to merge stale PR/Issue without recent activity, it'll be soon closed automatically.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants