Skip to content

Conversation

@stakovicz
Copy link
Contributor

@stakovicz stakovicz commented Jan 18, 2026

Refacto de la liste des inscriptions aux évènements vers Symfony.

J'ai ajouté des tests et des fixtures pour couvrir le maximum de cas.
Cela a mis en évidence que l'on a cassé le lien de paiement dans le cas ou une cotisation n'est plus à jour.

J'ai également ajouté une notion de TicketOffer qui mix les différents type de tarifs entre les constantes et ceux enregistrés en base. À discuter, naming is hard !

Issue liée : #2089

Avant :

avant

Après :

apres

$sessions = $this->talkRepository->getByEventWithSpeakersAndVotes(
event: $event,
search: $data['q'] ?? '',
search: $data['filter'] ?? '',
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Pour unifier le paramètre des filtres ente les controllers ?filter={mySearch}

@stakovicz stakovicz self-assigned this Jan 18, 2026
@stakovicz stakovicz marked this pull request as ready for review January 20, 2026 20:28
@stakovicz stakovicz requested review from Mopolo and vgreb January 20, 2026 20:28
Then The page "1" of the PDF should not contain "Total HT"
Then The page "1" of the PDF should not contain "Total TVA"
Then The page "1" of the PDF should contain "TOTAL 250 €"
Then The page "1" of the PDF should contain "TOTAL 275 €"
Copy link
Contributor

Choose a reason for hiding this comment

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

C'est le changement vers du chèque qui change ce montant ?

Y'a toujours indiqué 250 € 3 lignes au dessus, et c'est un test avant la TVA donc je n'arrive pas à comprendre.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Le paiement est passé de CB à CHQ, il y a donc des frais de 25€ en plus. J'ai complété le test pour le mettre en évidence.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants