AcadeNice/corentin_wakdo
7
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

feat(borne): modale allergenes generale (14 INCO) sur carte et fiche + harnais tests JS (P3) #36

Merged
Corentin merged 1 commit from feat/p3-borne-allergenes into dev 2026-06-17 12:10:47 +02:00
Conversation 0 Commits 1 Files changed 12 +1015 -4
Corentin commented 2026-06-17 12:09:36 +02:00
Owner
Copy link
No description provided.
Corentin added 1 commit 2026-06-17 12:09:36 +02:00
feat(borne): modale allergenes generale (14 INCO) sur carte et fiche + harnais tests JS (P3)
All checks were successful
CI / secret-scan (pull_request) Successful in 7s
Details
CI / php-lint (pull_request) Successful in 19s
Details
CI / static-tests (pull_request) Successful in 39s
Details
CI / js-tests (pull_request) Successful in 19s
Details
CI / secret-scan (push) Successful in 8s
Details
CI / php-lint (push) Successful in 18s
Details
CI / static-tests (push) Successful in 39s
Details
CI / js-tests (push) Successful in 16s
Details
CI / auto-merge (pull_request) Successful in 4s
Details
CI / auto-merge (push) Has been skipped
Details
eb2891238f 
PR-C du lot P3. Icone "i" allergenes sur la borne ouvrant une modale GENERALE
listant les 14 allergenes a declaration obligatoire (UE INCO 1169/2011). Info
generale, PAS un calcul par produit (mapping ingredient_allergen reste differe).

- data/allergens.json : les 14 INCO (liste fixe data borne). loadAllergens()
  ajoute a data.js avec point de swap P4 documente (-> /api/allergens).
- allergens.js : module CSP-safe (DOM API, aucun handler inline). buildAllergenInfoButton
  (bouton "i", stopPropagation sur la carte), openAllergenModal (idempotent),
  fermeture par bouton / clic overlay / touche Echap.
- Integration : carte produit (page-products.js) ET fiche produit simple
  (page-product.js) ; CSS modale + badge reutilisant le pattern overlay du composer.

Harnais de tests front (premier du depot) : node:test + jsdom. tests/js/allergens.test.js
couvre les 14 INCO, la construction du bouton, l'ouverture/listing/fermeture et
l'idempotence (7 tests). Scoping ESM local (src/public/borne/assets/js + tests/js)
pour garder la racine en CommonJS (hooks, _byan, bin inchanges). Job CI js-tests
(Node 20 epingle) ajoute aux checks requis de l'auto-merge.
Corentin scheduled this pull request to auto merge when all checks succeed 2026-06-17 12:09:37 +02:00
Corentin merged commit 1ecd78324c into dev 2026-06-17 12:10:47 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
auto-merge

Fusion automatique sur CI verte

No labels
auto-merge
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: AcadeNice/corentin_wakdo#36
No description provided.
Delete branch "feat/p3-borne-allergenes"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?