Skip to content

Conversation

@mihien
Copy link

@mihien mihien commented Oct 30, 2025

Remove the cash total being reset to 0 when clicking on the calculator in the cash opening popup.

@mihien mihien force-pushed the 16.0-add-pos_cash_opening_popup_no_reset branch 2 times, most recently from 052cc9a to 5564a5e Compare November 3, 2025 14:09
Copy link

@harry-ngn harry-ngn left a comment

Choose a reason for hiding this comment

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

Functionnal test successfull

Copy link

@remytms remytms left a comment

Choose a reason for hiding this comment

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

Small change in description phrasing.

@@ -0,0 +1 @@
On the cash opening popup removes the cash total being reset to 0 when clicking on the calculator..
Copy link

Choose a reason for hiding this comment

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

Suggested change
On the cash opening popup removes the cash total being reset to 0 when clicking on the calculator..
Disable reset of the total cash to 0 when clicking on the calculator on the cash opening popup.

Comment on lines 7 to 8
"summary": "On the cash opening popup removes the cash total being reset "
"to 0 when clicking on the calculator",
Copy link

Choose a reason for hiding this comment

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

Suggested change
"summary": "On the cash opening popup removes the cash total being reset "
"to 0 when clicking on the calculator",
"summary": "Disable reset of the total cash to 0 when clicking on the"
"calculator on the cash opening popup.",

@mihien mihien force-pushed the 16.0-add-pos_cash_opening_popup_no_reset branch from 5564a5e to 64a8d50 Compare December 17, 2025 14:49
@mihien mihien requested a review from remytms December 17, 2025 14:49
Copy link

@remytms remytms left a comment

Choose a reason for hiding this comment

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

LGTM.

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.

3 participants