Skip to content

Show split payment amounts on sale details and receipts - #1239

Merged
sridharkalaibala merged 2 commits into
developfrom
codex/split-payment-receipts
Oct 10, 2026
Merged

sridharkalaibala merged 2 commits into
developfrom
codex/split-payment-receipts

Conversation

@sridharkalaibala

@sridharkalaibala sridharkalaibala commented Oct 10, 2026 •

Copy link
Copy Markdown
Contributor

What does this PR do?

Sale details and printed receipts now show the recorded amount for each payment method, such as UPI 300 and Cash 200, with readable aligned rows. The shared reader supports object, JSON and array split-payment records and Captain tender entries. It does not invent missing historical allocations or use the full bill total for an unpaid sale.

Acceptance criteria covered

  • Sale details, designed A4/58mm/80mm receipts and legacy receipts display each recorded tender.
  • Partial payments and mixed parent/tender records avoid inflated collected totals.
  • Payment labels are escaped and existing accounting records remain unchanged.

How was it tested?

  • 126 relevant Node tests passed on the latest develop base.
  • Frontend asset build passed; thermal receipt screenshot inspected locally.
  • Shared helper lint and formatting passed.
  • Attribution and runtime dependency checks passed.
  • Physical printer output was not tested for this display-only change.

Checklist

  • Focused change; human-only attribution and signed-off commit.
  • Works offline without new external calls.
  • No production data or credentials included.

Release-check repair: formatted two existing item pagination files from develop after reproducing the Prettier failure locally. No item behavior changes. Full API formatting check now passes.

Signed-off-by: Sridhar Bala <sridharkalaibala@gmail.com>
Signed-off-by: Sridhar Bala <sridharkalaibala@gmail.com>
@sridharkalaibala
sridharkalaibala merged commit bf14d1e into develop Oct 10, 2026
10 checks passed
@github-actions github-actions Bot added the ready for QA Merged to develop and live on develop.posnic.io - anyone can test it label Oct 10, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Merged to develop. Anyone can test this - you do not need write access.

Try it at https://develop.posnic.io, or run it yourself:

git fetch origin develop && git checkout develop
npm install && npm --prefix api install
npm run dev   # then http://localhost:3000

When you have tested it, say what you did and what happened, and set
QA passed or QA failed. If you cannot set labels, just comment -
a maintainer will.

Reporting that something is broken is as useful as fixing it. It is
better found here than by a shopkeeper.

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

Labels

ready for QA Merged to develop and live on develop.posnic.io - anyone can test it

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant