Skip to content

Merge main → env/staging (correctifs v1 jusqu'à #246) - #248

Merged
Oasis35 merged 6 commits into
env/stagingfrom
claude/merge-main-into-staging
Oct 1, 2026
Merged

Oasis35 merged 6 commits into
env/stagingfrom
claude/merge-main-into-staging

Conversation

@Oasis35

@Oasis35 Oasis35 commented Oct 1, 2026 •

Copy link
Copy Markdown
Owner

Before: main n'avait pas été mergé dans env/staging depuis plusieurs correctifs v1, contrairement au flux prévu par docs/refonte-v2/DEVELOPPEMENT.md.

After: env/staging contient tout main. Le contenu de #197, #199, #239 et #241 y était déjà ; seul #246 apporte un vrai changement au code v1 de la branche : renommage permis pour un morceau du défi du jour, avec un avertissement dans la modale.

Conflits résolus en gardant la version env/staging, plus avancée (tests front sous Vitest, modale app-modal, Signal Forms, doc à jour), puis en y reportant #246 :

  • admin-pool.service.ts et son spec : plus d'état locked, ni de garde renameLocked ;
  • edit-track-modal.component.html : avertissement « défi du jour », plus de message locked ;
  • CLAUDE.md racine et features/admin/CLAUDE.md : textes de Admin : renommer un morceau du défi du jour depuis le pool #246, pas de doublon du piège 45 ;
  • docker-compose.staging.yml et clipboard.service.spec.ts : version env/staging.

Vérifié en local : build back, tests unitaires back (426), build front, tests front de l'admin (Vitest, 190). Les tests d'intégration et E2E tournent en CI.

BrevoEmailSender remplace ResendEmailSender (API transactionnelle Brevo,
header api-key). Configuration Brevo:*, variable BREVO_API_KEY, page
confidentialité et doc mises à jour.
* Envoi des emails via Brevo au lieu de Resend (#196)

BrevoEmailSender remplace ResendEmailSender (API transactionnelle Brevo,
header api-key). Configuration Brevo:*, variable BREVO_API_KEY, page
confidentialité et doc mises à jour.

* Gel de série : retirer du compteur les gels déjà engagés sur une série protégée (#198)

La gélule du header affichait encore le stock en base (ex. 2) alors que l'accueil
annonçait qu'un gel avait sauvé la série : le stock n'est décrémenté qu'à la
complétion du défi. StreakView.Freezes déduit désormais les jours manqués en
statut protected, ce qui correspond au stock qui restera après la partie.
…ls cassé) (#239)

Tous les appareils d'un compte v1 partagent le même AuthToken : le faire
tourner à la déconnexion (M6, #179) déconnectait tous les autres appareils.
La déconnexion n'efface plus que le cookie de l'appareil courant.
… (#241)

Le bouton « Partager mon score » ne copiait parfois rien au premier clic :
navigator.clipboard.writeText rejette de façon intermittente (NotAllowedError,
« Document is not focused »). Sur rejet, ClipboardService retombe sur une copie
textarea + execCommand('copy') et remonte l'échec (sans le texte) à l'API.
* feat(admin): renommer un morceau du défi du jour depuis le pool

Retire le verrou RenameLock (409 track_locked) : le bouton ✎ est actif
sur toutes les lignes du pool, défi du jour compris. Les réponses déjà
enregistrées gardent leur verdict, les suivantes sont corrigées avec le
nouveau nom.

* feat(admin): avertissement dans la modale de renommage pour un morceau du défi du jour
…o-staging

# Conflicts:
#	CLAUDE.md
#	docker-compose.staging.yml
#	src/front/InSeconds.Client/src/app/core/services/clipboard.service.spec.ts
#	src/front/InSeconds.Client/src/app/features/admin/CLAUDE.md
#	src/front/InSeconds.Client/src/app/features/admin/components/edit-track-modal/edit-track-modal.component.html
#	src/front/InSeconds.Client/src/app/features/admin/services/admin-pool.service.spec.ts
#	src/front/InSeconds.Client/src/app/features/admin/services/admin-pool.service.ts
@Oasis35 Oasis35 self-assigned this Oct 1, 2026
@sonarqubecloud

sonarqubecloud Bot commented Oct 1, 2026

Copy link
Copy Markdown

@Oasis35
Oasis35 merged commit d5d63d7 into env/staging Oct 1, 2026
12 checks passed
@Oasis35
Oasis35 deleted the claude/merge-main-into-staging branch October 2, 2026 12:52
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