Skip to content

Groupement des évènements par années #1715

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Apr 9, 2025

Conversation

Mopolo
Copy link
Contributor

@Mopolo Mopolo commented Apr 8, 2025

J'ai repris l'idée du comparateur d'events mais pour tous les endroits où la liste est affichée.

Avant

image

Après

image

@Mopolo Mopolo self-assigned this Apr 8, 2025
@Mopolo Mopolo force-pushed the event-lists-group-annees branch from 5645577 to 992c3c4 Compare April 8, 2025 18:54
@Mopolo Mopolo marked this pull request as ready for review April 8, 2025 19:06
@Mopolo Mopolo requested review from agallou and stakovicz April 8, 2025 19:06
Copy link
Contributor

@stakovicz stakovicz left a comment

Choose a reason for hiding this comment

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

C'est cool, mais est-ce qu'il n'y aurait pas moyen de factoriser ça avec ce form ?
https://github.com/afup/web/blob/master/sources/AppBundle/Event/Form/EventCompareSelectType.php

Et peut être généraliser la liste des événements utiliser un peu partout ?

@Mopolo Mopolo force-pushed the event-lists-group-annees branch from 992c3c4 to 4658bc4 Compare April 9, 2025 11:33
@Mopolo
Copy link
Contributor Author

Mopolo commented Apr 9, 2025

C'est cool, mais est-ce qu'il n'y aurait pas moyen de factoriser ça avec ce form ? master/sources/AppBundle/Event/Form/EventCompareSelectType.php

Bonne idée ! J'ai extrait le plus possible dans une classe dédiée et ajouté quelques tests unitaires.

Et peut être généraliser la liste des événements utiliser un peu partout ?

Il ne reste que peu de pages qui n'utilisent pas EventSelectType, ça ferait une bonne "first issue" je trouve nan ?

@Mopolo Mopolo force-pushed the event-lists-group-annees branch from 4658bc4 to 6dc3462 Compare April 9, 2025 11:46
@stakovicz
Copy link
Contributor

Il ne reste que peu de pages qui n'utilisent pas EventSelectType, ça ferait une bonne "first issue" je trouve nan ?
Oui très bonne idée.

@Mopolo Mopolo merged commit d5278ae into afup:master Apr 9, 2025
5 checks passed
@Mopolo Mopolo deleted the event-lists-group-annees branch April 9, 2025 12:09
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