Skip to content

Espace « Mes partages » pour l'invité (#273)#359

Merged
ronan-develop merged 5 commits into
mainfrom
feature/273-mes-partages
Jul 23, 2026
Merged

Espace « Mes partages » pour l'invité (#273)#359
ronan-develop merged 5 commits into
mainfrom
feature/273-mes-partages

Conversation

@ronan-develop

Copy link
Copy Markdown
Owner

Résumé

  • Nouvelle page /mes-partages listant, pour l'utilisateur connecté, les Share actifs (non révoqués, non expirés) dont il est le destinataire (guest) — jusqu'ici la seule porte d'entrée était le lien contenu dans l'email de notification.
  • ShareRepository::findActiveByGuest() : requête filtrée guest + active, triée par date décroissante. Aucune migration Doctrine (l'index idx_share_lookup couvrait déjà le besoin).
  • MyReceivedSharesWebController (lecture seule) + templates/web/my_received_shares.html.twig, réutilisant le pattern resolveResourceName/redirectUrlFor existant dans ShareWebController.
  • Lien de navigation conditionnel {% if app.user.isGuest %} ajouté à la sidebar.
  • Décisions de conception : page séparée de /partages (dédiée à l'owner) ; partages expirés/révoqués exclus (pas grisés) ; route accessible à un compte full mais toujours liste vide (pas de 403).
  • Correction d'un commentaire obsolète dans ShareWebController qui affirmait à tort qu'un guest n'avait pas accès à /partages.

Closes #273

Test plan

  • ./vendor/bin/phpunit --filter ShareRepositoryTest — 7 tests (isolation guest, exclusion expiré/révoqué, vide, tri)
  • ./vendor/bin/phpunit --filter MyReceivedSharesWebTest — 11 tests (accès non autorisé, happy path, cas limites, lien nav)
  • Suite complète ./vendor/bin/phpunit — 940 tests, GREEN
  • composer build-assets (Tailwind) relancé après modification des templates

@ronan-develop ronan-develop added feature Nouvelle fonctionnalité api Modifications ou ajouts concernant l’API (routes, contrôleurs, ressources…). frontend CSS, Twig, Stimulus, JS labels Jul 23, 2026
@ronan-develop
ronan-develop merged commit b561d74 into main Jul 23, 2026
6 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api Modifications ou ajouts concernant l’API (routes, contrôleurs, ressources…). feature Nouvelle fonctionnalité frontend CSS, Twig, Stimulus, JS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Espace « Mes partages » pour l'invité — regrouper tous ses accès reçus

1 participant