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 : nouveau champ 'end' #821

Merged
merged 5 commits into from
Apr 18, 2023
Merged

Événements : nouveau champ 'end' #821

merged 5 commits into from
Apr 18, 2023

Conversation

raphodn
Copy link
Member

@raphodn raphodn commented Apr 17, 2023 •
edited
Loading

Quoi ?

  • Nouveau champ Event.end (facultatif)
    • ajouté le champ au formulaire de création d'événement
    • afficher la durée de l'événement si le champ est rempli
  • Améliorations d'affichage sur le formulaire de modification d'un événement :
    • descendu les champs "procuration" + mis dans une section "nécessite un vote ?"
    • éviter que l'image prennent trop de places

Information supplémentaire

  • il faudrait aussi renommer Event.date en Event.start, mais j'ai peur de tout casser...
  • il faudrait aussi rajouter des check pour s'assurer que end > start

Capture d'écran

Avant Après
image image
image Screenshot from 2023-04-17 16-16-41

@raphodn raphodn self-assigned this Apr 17, 2023
Base automatically changed from raphodn/event-list-cleanup to master April 18, 2023 07:13
@symartin
Copy link
Collaborator

Hello,

il faudrait aussi renommer Event.date en Event.start, mais j'ai peur de tout casser...

ba a mon avis il vaut mieux faire le changement dans la BDD maintenant, que plus tard qd on utilisera les événements plus d'une fois par an...

sinon, petit bug d'affichage, si la date de fin est ds plus d'un mois, ça fait un mauvais décompte des jours :

image

et si end > start (ce qui ne devrait pas être possible, comme tu le dis) ça met rien :-/ :
image

le 2eme n'est pas très important mais le premier et qd mm gênant...

@symartin symartin added Interface utilisateur En lien avec l'ergonomie et le GUI/frontend Evénements et contenus dynamiques En lien avec les événements (Events) et les contenus dynamiques labels Apr 18, 2023
@raphodn
Copy link
Member Author

raphodn commented Apr 18, 2023

sinon, petit bug d'affichage, si la date de fin est ds plus d'un mois, ça fait un mauvais décompte des jours

déjà merci d'avoir testé, et pour l'explication 🙌
j'ai corrigé ici - 311ddf1 - en rajoutant mois & année(s)

@raphodn
Copy link
Member Author

raphodn commented Apr 18, 2023 •
edited
Loading

il faudrait aussi rajouter des check pour s'assurer que end > start

bon en fait Symfony ça a du bien parfois, c'est très simple à rajouter ce check en fait : bdb02a4
(ça avait déjà été fait coté MembershipShiftExemption)

@raphodn raphodn merged commit e4933aa into master Apr 18, 2023
@raphodn raphodn deleted the raphodn/event-edit-improve branch April 18, 2023 17:16
@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 : nouveau champ 'end' (elefan-grenoble#821)
69bcb65 
* Event edit: improve form display, reduce img size

* New field Event.end (optional)

* Cleanup new & edit form

* Fix duration: add year & months

* Check that start < end
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 Interface utilisateur En lien avec l'ergonomie et le GUI/frontend
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