Skip to content

Prolonger la validité d'un token visiteur (#1747)#2166

Open
BySplashGm wants to merge 5 commits intoafup:masterfrom
BySplashGm:feature/1747-prolonger-token-visiteur
Open

Prolonger la validité d'un token visiteur (#1747)#2166
BySplashGm wants to merge 5 commits intoafup:masterfrom
BySplashGm:feature/1747-prolonger-token-visiteur

Conversation

@BySplashGm
Copy link
Contributor

@BySplashGm BySplashGm commented Mar 19, 2026

Description

Ajout d'un bouton permettant de prolonger la validité d'un token visiteur de 3 jours depuis la liste des tokens dans l'administration des événements.

Changements

  • Nouvelle route admin_event_special_price_extend (/event/special_price/{id}/extend)
  • Nouveau controller ExtendSpecialPriceAction qui décale la date de fin du token
  • Constante EXTEND_DAYS = 3 dans SpecialPriceAction partagée entre le controller et le template
  • Ajout du bouton "prolonger" avec tooltip dans le template (avant la colonne "Lien")
  • Fix : ajout de primary: true manquant dans les métadonnées du TicketSpecialPriceRepository (nécessaire pour que get($id) fonctionne)

Tests

  • Nouveau scénario Behat : création d'un token puis prolongation via le bouton

Screenshots

Capture d’écran 2026-03-19 à 09 48 52

Permet de prolonger la validité d'un token visiteur de 3 jours depuis
la liste des tokens dans l'administration des événements.
@BySplashGm BySplashGm marked this pull request as ready for review March 19, 2026 08:49
@BySplashGm
Copy link
Contributor Author

Je n'étais pas trop sûr de comment définir la durée de prolongation; dans l'issue il est dit "disons 3 de façon totalement arbitraire" j'ai donc mis une constante, mais je pense qu'il serait possible de faire autrement ?

Copy link
Contributor

@Mopolo Mopolo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Je pense que la constante comme ça c'est très bien pour le moment.

Si vraiment à l'usage on se rend compte que ça serait bien que ce soit paramétrable on verra.

Merci pour la contribution :)

@BySplashGm
Copy link
Contributor Author

Avec plaisir, si jamais la constante est modifiée il faudra penser à modifier le test TokensVIsiteurs.feature 😉

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.

3 participants