Membership dans season #2

Merged
fabien merged 1 commits from michel/GOLEM:michel_adhesions into master 2018-01-19 07:53:43 +01:00
Contributor

Affichage des membership uniquement.

Affichage des membership uniquement.
Owner

Très bien. En revanche, avant de fusionner, je vais attendre les deux autres points d'améliorations des adhésions :

  • Reprise des dates par défaut
  • Sortie liste des contacts du module / menu principal membres
Très bien. En revanche, avant de fusionner, je vais attendre les deux autres points d'améliorations des adhésions : * Reprise des dates par défaut * Sortie liste des contacts du module / menu principal membres
Owner

Sur la reprise des dates et donc le fichier golem_season/models/golem_season.py :

  • la méthode onchange est la bonne, il faudrait un for season in self plutôt qu'activity ;
  • tu peux ne tester la présence que du date_start et appliquer les deux dates je pense ;
  • enfin, un test unitaire sur ce sujet serait une bonne chose.

Pour le reste, j'attends la sortie du menu contacts et le fait que l'on ne puisse pas créer à la volée de nouvel article d'adhésion sur le formulaire des saisons.

Merci !

Sur la reprise des dates et donc le fichier `golem_season/models/golem_season.py` : * la méthode onchange est la bonne, il faudrait un `for season in self` plutôt qu'activity ; * tu peux ne tester la présence que du date_start et appliquer les deux dates je pense ; * enfin, un test unitaire sur ce sujet serait une bonne chose. Pour le reste, j'attends la sortie du menu contacts et le fait que l'on ne puisse pas créer à la volée de nouvel article d'adhésion sur le formulaire des saisons. Merci !
fabien closed this pull request 2018-01-19 07:53:43 +01:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: Yaltik/golem#2
No description provided.