Discussion: scanned table QR ordering page should not show table-bound order history; order history should be tied to the consumer, not the table #351
Closed
firefoxwang
started this conversation in
General
Replies: 3 comments
|
Table token was ment to be unique per scan. Need to review |
0 replies
Table history should be bound to current token.
Yes show them. As always bound to the token.
Yes, add customer account. I thought this was already working.
Back Office should always be able to see any order from any table at any time. Also historical order per table, per customer. |
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Type
Design discussion — not a bug fix; requires human engineers to discuss and decide before development. Without a decision reached, Agents must not start development (Human in the loop).
Origin
This discussion mirrors GitHub issue #350 (#350). GitHub removed the "Convert to discussion" feature, so we re-created it here. Original issue stays as the source of truth for agent task tracking.
Background
The scanned ordering page (
/menu/{table_token}) currently loads and displays the order history of that specific table:GET /menu/{table_token}/order-history(back/app/main.py): no login required, returns the most recent paid/completed orders for that table (default 10, max 50).front/src/app/menu/menu.component.ts/menu.component.html).Problems
Proposed direction (open for discussion)
Core principle: order history on the ordering page must be bound to the consumer (logged-in account), not to the table.
Scenario 1: Logged-in consumer placing an order
Scenario 2: Anonymous (not logged in) table ordering
Questions that need discussion/decision
Order.customer_id? (A starting point already exists inGET /customer/ordersincustomer_routes.py, but the public flow currently does not attach an account.)Constraints
All reactions