Mettre un peu en avant les membres sans adhésions #729
Conversation
raphodn
force-pushed
the
raphodn/member-status-warnings
branch
from
4016ece
to
f565ff0
Compare
|
Pour éviter les confusions, je mettrais plutot une $ (que un €). C'est plus parlant je pense. |
src/AppBundle/Entity/Beneficiary.php
Outdated
| } | ||
| if (!$this->getMembership()->hasValidRegistration()) { | ||
| return true; | ||
| } |
There was a problem hiding this comment.
j'ai mis un if global finalement : 32502c3
| } | ||
| // for dispensed beneficiary 🏵 | ||
|
|
There was a problem hiding this comment.
il peut y avoir plusieurs status en même temps
| @@ -48,7 +48,7 @@ public function getSearchForm($formBuilder, $type = null) { | |||
| ] | |||
| ]); | |||
| $formBuilder->add('exempted', ChoiceType::class, [ | |||
| 'label' => 'exempté', | |||
| 'label' => '☂ exempté', | |||
There was a problem hiding this comment.
Pas possible d'utiliser une méthode du type getStatusIcon pour éviter de dupliquer les icones à plusieurs endroits ?
There was a problem hiding this comment.
le getStatusIcon dépend du bénéficiaire. il faudrait plutôt basculer tout ca dans un fichier de config (un peu comme les noms & icones des rôles dans config.yml)
- l'icon html
- l'icon materialize
- le background color de la fiche membre
There was a problem hiding this comment.
j'ai fait un début de PR ici : #732
mais pas possible dans l'état de récupérer un 'parameter' dans une Entity, donc ca va être un peu plus compliqué
raphodn
force-pushed
the
raphodn/member-status-warnings
branch
from
e704535
to
585798a
Compare
* Add extra warnings: isCurrentlyExemptedFromShifts & not hasValidRegistration * Also display symbol on member list & show * Add legend in Period page * Use $ instead of €. Cleanup symbols (remove hex)
Quoi ?
Dans la page "Semaine type"
Dans la page "Liste des membres" et "Membre"
Pourquoi ?
Pour indiquer les membres "actifs" mais qui n'ont pas encore (ré-)adhéré. Cela peut poser problème si ils sont inscrits à des créneaux fixes, il y a un risque qu'ils ne viennent pas.
Captures d'écran
edit : remplacé le € par un $