Skip to content

feat(catalog): tune the courier fee, triple the wares, show item icons - #15

Merged
arnaud-zg merged 0 commit into
docs/readme-rewritefrom
feat/catalog-expansion-v2
Sep 2, 2026
Merged

arnaud-zg merged 0 commit into
docs/readme-rewritefrom
feat/catalog-expansion-v2

Conversation

@arnaud-zg

Copy link
Copy Markdown
Owner

Stacked on #14.

Summary

  • courier fee numbers were leftover placeholders (a 300 copper threshold, a 25 copper
    fee) against items priced in the tens of gold, so delivery was always free and the fee
    was invisible on the rare cart that did pay it. Raised both to numbers that mean
    something at this catalog's prices
  • catalog grows from 12 to 30 items: Rhok'delar plus 17 Legion artifact weapons
    (Aluneth, Felo'melorn, Fangs of Ashamane, Scythe of Elune, Maw of the Damned,
    Apocalypse, Claws of Ursoc, G'Hanir, Strom'kar, Titanstrike, Thas'dorah, Talonclaw,
    Truthguard, T'uure, Xal'atath, Ulthalesh, The Silver Hand), each with a verified
    Wowpedia image, a verified lore video, and full EN/FR content
  • Product now carries a separate icon alongside its hero image, and the product card
    overlays it as a small badge, closer to how the game itself shows an item
flowchart LR
  A[12 items, free delivery always] --> B[30 items]
  B --> C[real courier threshold]
  A --> D[hero image only]
  D --> E[hero image + icon badge]
Loading

Test plan

  • pnpm typecheck
  • pnpm check
  • pnpm test:coverage, 22 files, 120 tests, thresholds pass
  • pnpm build, checked the shop page: icon badges on every card, courier fee
    shown/waived correctly, all 30 cards render with working video links
  • pnpm test:e2e

🤖 Generated with Claude Code

https://claude.ai/code/session_014BcGPRc3sgUJYKihGRXh5g

@arnaud-zg
arnaud-zg force-pushed the feat/catalog-expansion-v2 branch from 8e7297c to 1ef28c7 Compare September 2, 2026 09:58
@arnaud-zg
arnaud-zg merged commit 1ef28c7 into docs/readme-rewrite Sep 2, 2026
@arnaud-zg

Copy link
Copy Markdown
Owner Author

Note: this shows as merged because of a transient git topology state during a PR stack reorg, not an actual merge into main (verified: none of these commits are in main). Superseded by #19, which has the correct base and the same content.

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.

1 participant