Améliore la mise en avant des nouveaux membres #742
Conversation
raphodn
changed the title
raphodn
changed the title
| It use the materialize modal class https://materializecss.com/modals.html | ||
| id = "modal-bucket" |
There was a problem hiding this comment.
| id = "modal-bucket" |
Coquille ?
src/AppBundle/Entity/Beneficiary.php
Outdated
| */ | ||
| public function isNew() | ||
| { | ||
| return $this->shifts->count() <= 1; |
There was a problem hiding this comment.
Je regarderai plutôt si le créneau est le premier créneau du beneficiaire.
La si tu réserves 2 créneaux, tu perds l'info. Tu perds aussi l'info dans le future.
There was a problem hiding this comment.
dans cette PR j'ai touché à la forme, pas le fond.
en effet je peux essayer de regarder pour remplacer le isNew() par un isFirstShift()
ou alors les faire cohabiter ? (pour afficher un badge sur la fiche membre aussi 🤔 )
le isNew pourrait plutôt regarder la date de première registration par exemple
There was a problem hiding this comment.
j'ai rajouté un commit : 3d95832
- qui créé
Shift.isFirstByShifterpour checker si c'est bien le "Premier créneau" - j'ai élargi un peu le
isNewà moins de 3 créneaux (mais faudra en rediscuter avec le BDM et tout)
There was a problem hiding this comment.
Parfait le Shift.isFirstByShifter, par contre je supprimerai le isNew. Je ne suis pas sur que ça apporte grand chose. A vouloir mettre trop de logo, ca va finir par être contre productif.
On pourrait très bien faire faire un Premiers créneaux (pluriel) si besoin.
There was a problem hiding this comment.
yep il faudrait pouvoir avoir "Deuxième créneau" et "Troisième créneau" si besoin.
j'ai gardé le isNew, ca concerne peu de membres, et c'est un axe d'amélioration de cette année (dont je me suis porté volontaire), donc je vais commencer par là et en discuter avec le BDM.
There was a problem hiding this comment.
j'ai gardé le isNew, ca concerne peu de membres, et c'est un axe d'amélioration de cette année (dont je me suis porté volontaire), donc je vais commencer par là et en discuter avec le BDM.
Le problème d'isNew, c'est que ça duplique la logique à 2 endroits. Pas top pour maintenir le code (qui déjà assez complexe)
There was a problem hiding this comment.
je pense que la règle qui défini isNew évoluera (en regardant la registrationDate par exemple, et en le déplacant sur le Membership), donc là c'est similaire mais ca reste séparé et pas complètement dupliqué..
There was a problem hiding this comment.
ah merde j'avais oublié de pusher un truc en fait (j'en ai profité pour rajouter un commentaire)
edit : j'ai pushé sur master du coup : 8a3a4ee
* Add new Shift.isFirstByShifter & Beneficiary.isNew * Add icon next to user / badge & small badge in admin user show & badge in modal content * Fix red color
Quoi ?
Mettre un peu en avant les nouveaux membres (plus exactement les nouveaux "bénéficiaires" qui sont inscrits à 0 ou 1 créneau)
Il existait déjà un badge "Premier créneau" sur l'écran d'accueil (badgeuse)
Modifications apportées :
beneficiary_new_icon= ★ (et basculébeneficiary_main_icon= ⚐ dans config as well)Beneficiary.isNew()qui renvoietruesi le bénéficiaire a moins de 3 créneauxShift.isFirstByShifter()qui renvoietruesi c'est le premier créneau du bénéficiaireCaptures d'écran
Reflexions
Qu'est-ce qu'un nouveau bénéficiaire ? Là avec cette implémentation, si le bénéficiaire est inscrit à 2 créneaux dans le futur, alors le badge disparaitra (car
beneficiary.shifts.count() = 2). Peut-être plutôt regarder du coté de la date de première registration ? + 1 mois ?