Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Événements : nouvelle entité pour définir des types d'événements #822

Merged
merged 4 commits into from
Apr 18, 2023
Merged

Événements : nouvelle entité pour définir des types d'événements #822

merged 4 commits into from
Apr 18, 2023

Conversation

raphodn
Copy link
Member

@raphodn raphodn commented Apr 17, 2023 •
edited
Loading

Quoi ?

  • nouvelle entité EventKind
  • pouvoir list / new / edit / delete

Information supplémentaire

Nommage : pourquoi EventKind et pas plutôt EventType ? Car dans Symfony EventType a déjà une signification. Et le mot type est aussi utilisé dans php. kind a l'air plus neutre.

Captures d'écran

Page
Admin image
Liste des types d'événements image

@raphodn raphodn self-assigned this Apr 17, 2023
@raphodn raphodn requested a review from petitalb April 17, 2023 13:50
@raphodn raphodn force-pushed the raphodn/event-kind-entity branch from b7a31f0 to 130a3b7 Compare April 17, 2023 14:18
@raphodn raphodn mentioned this pull request Apr 17, 2023
Merged
@raphodn raphodn requested a review from symartin April 17, 2023 14:41
@raphodn raphodn force-pushed the raphodn/event-edit-improve branch from 7e1ce22 to 54ac17e Compare April 18, 2023 07:14
@symartin
Copy link
Collaborator

symartin commented Apr 18, 2023 •
edited
Loading

Hello,

En fait tu as mis EntityEventKind et pas: EntityKind comme décrit dans la PR. Mais comme ça on peut avoir des kind de qqc d'autre dans le future (bon après les Event c'est déjà un truc de Symfony 🤷‍♂️...).

Et oui type, c'est déjà utilisé pour plein de trucs aussi, donc kind c'est mieux 😀.

édit: j'avais pas bien suivi EntityEventKind et EntityKind ...

@symartin symartin added the Evénements et contenus dynamiques En lien avec les événements (Events) et les contenus dynamiques label Apr 18, 2023
Base automatically changed from raphodn/event-edit-improve to master April 18, 2023 17:16
@raphodn raphodn force-pushed the raphodn/event-kind-entity branch from 130a3b7 to 3917375 Compare April 18, 2023 17:23
@raphodn
Copy link
Member Author

raphodn commented Apr 18, 2023

en fait tu as mis EntityEventKind et pas: EntityKind

@symartin oui en effet je me suis complètement emmêlé dans la description de la PR. j'ai corrigé (à EventKind), et rebasé depuis master

@raphodn raphodn merged commit f8091fb into master Apr 18, 2023
@raphodn raphodn deleted the raphodn/event-kind-entity branch April 18, 2023 17:45
@raphodn raphodn linked an issue Apr 18, 2023 that may be closed by this pull request
Améliorer les événements #817
Open
OursDesCavernes pushed a commit to Les400Coop/gestion-compte that referenced this pull request Jan 20, 2024
@raphodn @OursDesCavernes
Événements : nouvelle entité pour définir des types d'événements (ele…
d2516a1 
…fan-grenoble#822)

* New entity EventKind

* EventKind: list, button in admin panel

* EventKind: new, edit, delete

* Add form errors in edit
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@symartin symartin symartin approved these changes

@petitalb petitalb Awaiting requested review from petitalb

Assignees

@raphodn raphodn

Labels
Evénements et contenus dynamiques En lien avec les événements (Events) et les contenus dynamiques
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Améliorer les événements
2 participants
@raphodn @symartin