koka/sanctuary
1
0
Fork 0
Code Issues 4 Pull Requests 1 Actions Packages Projects Releases Wiki Activity

fix(sanctuary): compter rubis et saphirs sur toutes les rangees d'inventaire #8

Closed
chris wants to merge 0 commits from fix/shop-currency-overflow-rows into main
pull from: fix/shop-currency-overflow-rows
merge into: :main
Conversation 0 Commits - Files Changed -
+10 -13
chris commented 2026-08-27 07:20:46 +00:00
Collaborator
Copy Link
Copy Source

Objectif

Le Shop ne comptait que l'inventaire vanilla (36 cases). Les rangées overflow 5 et 6 — visuellement la première et l'avant-dernière quand la hotbar est en bas — étaient ignorées pour les rubis et saphirs.

Périmètre

  • Modules touchés : sanctuary
  • APIs ou fichiers partagés touchés : ShopService réutilise BlackMarketService.countPlayerItem / removePlayerItem (SanctuaryPlayerInventoryView)
  • Dépend d'une autre branche/PR : non

Compatibilité inter-mods

  • chaque donnée/règle reste dans son module propriétaire
  • IDs, sauvegardes et protocole inchangés
  • tests adaptés (shopDeliveryTimingSmoke, verifyBlackMarket)
  • version module non incrémentée : les pins alpha.200 exigent encore sanctuary 0.0.0-alpha.115 ; à traiter dans le prochain lot

Test plan

  • Inventaire à 6 rangées, hotbar en bas
  • Mettre des rubis/saphirs sur la 1ère ligne et l'avant-dernière
  • Ouvrir le Shop : la bourse et Acheter/Sauvegarder reflètent le stock
  • Un achat / déblocage consomme bien ces piles
## Objectif Le Shop ne comptait que l'inventaire vanilla (36 cases). Les rangées overflow 5 et 6 — visuellement la première et l'avant-dernière quand la hotbar est en bas — étaient ignorées pour les rubis et saphirs. ## Périmètre - Modules touchés : `sanctuary` - APIs ou fichiers partagés touchés : `ShopService` réutilise `BlackMarketService.countPlayerItem` / `removePlayerItem` (`SanctuaryPlayerInventoryView`) - Dépend d'une autre branche/PR : non ## Compatibilité inter-mods - [x] chaque donnée/règle reste dans son module propriétaire - [x] IDs, sauvegardes et protocole inchangés - [x] tests adaptés (`shopDeliveryTimingSmoke`, `verifyBlackMarket`) - [ ] version module non incrémentée : les pins alpha.200 exigent encore `sanctuary` `0.0.0-alpha.115` ; à traiter dans le prochain lot ## Test plan - [ ] Inventaire à 6 rangées, hotbar en bas - [ ] Mettre des rubis/saphirs sur la 1ère ligne et l'avant-dernière - [ ] Ouvrir le Shop : la bourse et Acheter/Sauvegarder reflètent le stock - [ ] Un achat / déblocage consomme bien ces piles
chris added 1 commit 2026-08-27 07:20:47 +00:00
fix(sanctuary): compter rubis et saphirs sur toutes les rangees d'inventaire
Verification 26.2 / check (pull_request) Canceled after 0s
Details
3deeb90ffc 
Co-authored-by: Cursor <cursoragent@cursor.com>
chris closed this pull request 2026-08-27 08:22:36 +00:00

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 labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
chris koka
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: koka/sanctuary#8