Bandeau pour les membres qui ne reçoivent pas les notifications - #38
Merged
Conversation
Un parent qui n'a jamais activé les notifications ne s'en rend pas compte : il croit simplement que l'association ne lui écrit pas. Un bandeau le lui dit en haut du tableau de bord, et propose l'activation en un clic, sans passer par les réglages. Il apparaît dès que le membre ne recevrait rien — appareil jamais activé, ou notifications coupées sur le compte — et disparaît sitôt les deux réunis. « Plus tard » le range pour un mois : assez pour ne pas harceler celui qui refuse, assez court pour rattraper celui qui remet à plus tard. Sur iPhone, où Safari n'ouvre les notifications qu'aux sites ajoutés à l'écran d'accueil, le bandeau explique la manipulation au lieu de proposer un bouton qui ne pourrait pas fonctionner. Il s'efface sur « Mon compte », où le réglage complet se trouve déjà, et pendant le guide de première connexion, pour ne pas disputer l'attention d'un membre qui découvre l'application. La logique d'abonnement, jusqu'ici enfermée dans le réglage du compte, est extraite dans un module partagé : une seule demande de permission, un seul enregistrement, deux points d'entrée qui ne peuvent plus diverger. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_014SfQYBU4xXTeSEHKhHQXdD
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Un membre qui n'a jamais activé les notifications ne le sait pas : il croit simplement que l'association ne lui écrit pas. Un bandeau le lui dit en haut du tableau de bord, avec l'activation en un clic.
Quand il apparaît
Il ne s'affiche pas sur « Mon compte » (le réglage complet y est déjà, juste en dessous), ni pendant le guide de première connexion, ni sur la partie publique du site.
Cas iPhone
Safari n'ouvre les notifications qu'aux sites ajoutés à l'écran d'accueil : l'API n'existe pas avant. Plutôt qu'un bouton qui échouerait, le bandeau explique la manipulation (Partager → « Sur l'écran d'accueil »).
Au passage
La logique d'abonnement était enfermée dans le réglage du compte ; elle passe dans
src/lib/push-device.ts, partagé par les deux points d'entrée — une seule demande de permission, un seul enregistrement, plus de divergence possible. Un signalwindowtient les deux à jour quand l'un des deux change.Vérifications
Instance réelle (PostgreSQL 16,
next build+next start), navigateur réel (Chromium/Playwright), faux service de push — le reste (API, base) est réel.push_enabled=true) et abonne l'appareilRendu relu en capture d'écran sur 390 px, 820 px et 1280 px : un premier jet repliait le texte mot à mot sur téléphone, corrigé avant livraison.
Generated by Claude Code