Fix/menu validators demo banner token refresh #1

Closed

soroushdes wants to merge 0 commits from fix/menu-validators-demo-banner-token-refresh into main

pull from: fix/menu-validators-demo-banner-token-refresh

merge into: soroushdes:main

soroushdes:main

Conversation 0 Commits - Files Changed -

soroushdes commented 2026-06-01 15:36:37 +00:00

Owner

Copy Link

Copy Source

No description provided.

soroushdes added 2 commits 2026-06-01 15:36:37 +00:00

fix: menu item/category create, demo banner reach, token refresh, blog publish ... 024a455ab3

Dashboard & API bug fixes for owner-reported breakage:

- MenuController validators (PosValidators): NameEn was required but the
  dashboard sends null when blank, so every manual menu-item create failed
  and category create failed 100% (the form never sends nameEn). Now optional.
- DemoDataBanner: only showed when a cafe was exactly empty, so
  showcase-seeded cafes (2-3 cats / 3-5 items) could never trigger the
  one-click seed. Widened gate to sparse menus (<5 cats && <10 items) and
  added a clear "nothing to add" message when already populated.
- client.ts: added one-time JWT refresh-and-retry on 401 (shared in-flight
  promise) before bouncing to /login. Expired access tokens silently broke
  ticket list, add-table, and other reads.
- Surface API errors as toasts on menu + table mutations (were swallowed
  silently, so failures looked like "nothing happens").
- Admin blog editor: saving an edit dropped IsPublished (defaulted false,
  silently unpublishing the post on every save); now persisted with a
  toggle. Also hoisted the inner Field component to module scope - it was
  remounting every input on each keystroke and dropping focus.
- Admin integrations: replaced raw radio gateway selector with a styled
  RadioDot matching the iOS toggles.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

test: repair test suite broken by feature drift (red -> 81 passing) ... f813cc4854

The test project no longer compiled: recent feature commits changed
interfaces and DTOs without updating the test doubles/call sites, so the
whole suite (and therefore CI) was failing to build.

- NoOpInventoryService: add IInventoryService.GetPurchasesSummaryAsync and
  the new string? userId param on AdjustAsync.
- NoOpLoyaltyService: add ILoyaltyService.RedeemOnOrderAsync.
- NoOpOrderNotificationService: add NotifyCallWaiterAsync.
- New NoOpAbuseProtectionService and NoOpMediaStorageService test doubles.
- QrMenuTests: ReviewService/PublicService gained IAbuseProtectionService +
  IHttpContextAccessor (and ReviewService an IMediaStorageService); wire the
  new no-op doubles + a real HttpContextAccessor.
- PrintingTests: OrderDto gained a DisplayNumber int between CreatedAt and
  Items; pass it.
- DiscoverFilterTests: add missing `using Xunit;` and the new openNow arg on
  DiscoverFilterParams.FromQuery.

Result: dotnet test -> Passed: 81, Failed: 0.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

soroushdes added 1 commit 2026-06-01 18:09:05 +00:00

fix(website): accurate Iran border on homepage map + slow on/off marker blink ... b5a6b1b68d

Replaced the rough 40-point hand-drawn polygon with the real national border (74 vertices, Natural Earth via world.geo.json) and fitted the projection bounding box to Iran's true extent, so the silhouette is recognisable and café markers stay aligned. Reworked the marker animation from a radar-style expanding ring into a slow 3.6s ease-in-out lamp fade (opacity 1->0.2->1) with a halo that glows on and off in sync. Verified via the SVG timeline: opacity 1.0 at 0s, 0.2 at 1.8s, 1.0 at 3.6s.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

soroushdes closed this pull request 2026-06-01 19:17:05 +00:00

soroushdes referenced this issue from a commit 2026-06-25 09:43:29 +00:00

fix(pos): charge the server amount, and don't book unconfirmed card payments

Some checks are pending

Hide all checks

CI/CD / CI · API (dotnet build + test) (pull_request) Successful in 46s

Details

CI/CD / CI · Admin API (dotnet build) (pull_request) Successful in 31s

Details

CI/CD / CI · Dashboard (tsc) (pull_request) Successful in 1m5s

Details

CI/CD / CI · Admin Web (tsc) (pull_request) Successful in 35s

Details

CI/CD / CI · Website (tsc) (pull_request) Successful in 45s

Details

CI/CD / CI · Koja (tsc) (pull_request) Successful in 51s

Details

CI/CD / Deploy · all services (pull_request) Has been skipped

Details

Pull request closed

Please reopen this pull request to perform a merge.

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

No items

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

soroushdes

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: soroushdes/meezi#1