Skip to content

fix(windows): add AppX tile assets to resolve Microsoft Store cert 10.1.1.11 - #4

Merged
itsbkm merged 1 commit into
FreeOpenSourcePOS:mainfrom
khaira777:main
Jun 11, 2026
Merged

fix(windows): add AppX tile assets to resolve Microsoft Store cert 10.1.1.11#4
itsbkm merged 1 commit into
FreeOpenSourcePOS:mainfrom
khaira777:main

Conversation

@khaira777

Copy link
Copy Markdown
Contributor

Problem

The Microsoft Store rejected Flo Cafe 1.6.7 with certification failure 10.1.1.11 On Device Tiles — all tile icons were showing as the default placeholder image.

Root cause: assets/icon.png was a plain blue square (852 bytes). electron-builder uses this as the base to auto-generate AppX tiles when no build/appx/ folder exists, resulting in generic, unbranded tiles in the submitted .appx.

Changes

build/appx/ — 30 branded tile assets (new)

All required AppX tile types × 5 DPI scale variants, sourced from assets/icon-512.png (the actual Flo brand identity):

Tile Sizes
StoreLogo 50px × 5 scales
Square44x44Logo 44px × 5 scales
SmallTile 71px × 5 scales
Square150x150Logo 150px × 5 scales
Wide310x150Logo 310×150px × 5 scales
LargeTile 310px × 5 scales

electron-builder automatically picks up build/appx/ without any package.json changes needed.

assets/icon.png — fixed

Replaced the 852-byte blue square placeholder with the real Flo logo (sourced from assets/icon-256.png). Used for the system tray icon on all platforms.

Platform impact

  • Windows AppX/MSIX: Fixes cert failure ✅
  • Windows NSIS installer: No change
  • macOS (DMG/MAS): Unaffected — uses icon.icns explicitly
  • Linux: Tray icon slightly improved (real logo vs blue square)

- Added build/appx/ with all 30 required tile assets (6 types × 5 scales)
  sourced from assets/icon-512.png (actual Flo brand identity)
- Fixed assets/icon.png which was a plain blue square placeholder;
  replaced with the real Flo logo (copied from icon-256.png)
- Tile assets: StoreLogo, Square44x44Logo, SmallTile, Square150x150Logo,
  Wide310x150Logo, LargeTile — all scale variants (100%-400%)
@itsbkm
itsbkm merged commit c9c6795 into FreeOpenSourcePOS:main Jun 11, 2026
khaira777 added a commit that referenced this pull request Jul 5, 2026
…ts/discounts/tax/loyalty)

Integration tests (7 files, 123 assertions):
- Happy path: order→discount→bill→payment→completed
- Tax correctness: India GST calculation and recalculation
- Payment integrity: split payments, wallet double-spend, zero-amount rejection
- Order lifecycle: status transitions, cancel with PIN
- Bill reconciliation: discount sync, add-items sync
- Loyalty points: cashback, idempotency, wallet balance
- Discount edge cases: combined discounts, max limits, addon prices

Bug fixes:
- #3: Add-items filters cancelled items from total recalculation
- #4: Add-items syncs bill after updating order total
- #5: Cancel/restore round-off formula corrected
- #6: Discount endpoint wrapped in withTxn (race condition)
- #7: Infinity passes validation (NaN/isFinite check)
- #8: Missing amount defaults to remaining balance
- #9: Amount capped at remaining balance
- #11: Removing discount restores original tax
- #12: Add-items preserves order-level discount
- #13: Cancel/restore preserves order-level discount
- #14: Item-level discount checks max settings
- #15: Item-level discount syncs to bill
- #17: Cancel/restore wrapped in withTxn
- #20: Cashback calculated on discounted subtotal
- #23: Quantity/price validation on order creation

Shared test helper (tests/helpers/test-setup.ts):
- Extracts 40 lines of boilerplate per test file
- Provides: initTestDb, createApp, startServer, seed helpers, assert utilities

Co-Authored-By: Claude <noreply@anthropic.com>
khaira777 pushed a commit that referenced this pull request Jul 6, 2026
fix(windows): add AppX tile assets to resolve Microsoft Store cert 10.1.1.11
itsbkm added a commit that referenced this pull request Jul 6, 2026
fix: prevent layout shift and re-render jank in POS customer search
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.

2 participants