The bill reads like a bill

[sridharkalaibala]

From a printed bill off Azure's till:

"Table number, order type, covers, total quantity all these information can organize better as per international standards and may be two column or moving total quantity to bottom."

Two different mistakes, and right about both.

Four facts took four lines. Table, order type and covers are read at a glance, not down a list, and stacked one per line they pushed the items further down the paper than the things a guest is actually checking. Now two to a line:

Table               6A  Order type       Dine-in
Covers               4
Total Qty                                      5
Subtotal                                 2290.00

The two-character gutter is load bearing - without it the left value runs into the right label and prints 6AOrder type.

Narrow paper keeps one to a line on purpose. A 58mm roll is 32 characters; halved, that turns "Order type" into "Order ty.". Losing a word to save a line is the wrong trade on the one document the guest keeps.

The count was in the wrong place entirely. Total quantity is not a fact about the table, it is part of the arithmetic - how many things, what they came to. It sat in the header beside the table number, which is where the restaurant looks and not where the guest does. It now prints immediately above the subtotal, and a test pins that nothing is allowed between the two.

Tests

7 new, driven through the real renderer and read back out of the bytes with escpos-preview, so they assert on what the paper says rather than on the source.

The existing switch-liveness test (every switch actually changes the bill when it is turned on) needed teaching rather than loosening: it assumed every switch writes a header row. It now accepts a switch that changes the bill anywhere, and still refuses one that changes nothing.

2809 desktop tests pass.

Not from this branch

tests/unit/middleware/items.validation.test.js and tests/unit/routes/items.routes.test.js fail to load on main already - 2 suites, 0 tests run - and reproduce with this branch stashed. Worth someone looking, since main is the deploy source.

[github-actions]

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.