Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • G gestioCOF
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
  • Issues 76
    • Issues 76
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 16
    • Merge requests 16
  • Deployments
    • Deployments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Commits
  • Issue Boards
Collapse sidebar
  • Klub Dev ENSKlub Dev ENS
  • gestioCOF
  • Merge requests
  • !137

Utilisation de django_custommail

  • Review changes

  • Download
  • Patches
  • Plain diff
Merged Martin Pepin requested to merge Kerl/use_django_custommail into master Dec 22, 2016
  • Overview 0
  • Commits 21
  • Pipelines 0
  • Changes 32

On utilise le package django_custommail, ça signifie que :

  • On installe le package depuis le dépôt COF-Geek
  • On supprime tous les fichiers texte des mails
  • On supprime le modèle CustomMail obsolète de gestioncof

Cette MR est marquée “Work In Progress” pour deux raisons :

  1. Ce n'est pas complètement testé
  2. Il reste à déterminer comment charger dans la BDD les mails nécessaires au bon fonctionnement de GestioCOF. Pour l'instant j'ai tout dans une fixture mais ça a plus sa place dans une data migration…
  3. J'aimerais écrire quelques tests unitaires.

UPDATE

Je retire le WIP

  • Normalement tout est testé et fonctionne
  • Le choix fait pour charger les mails dans la BDD est une fixture un peu particulière stockée dans gestioncof/management/data/ et qui est chargé par la commande python manage.py syncmails. Le chargement est moins strict que celui de loaddata : les pk sont ignorées et toutes les données déjà présentes dans la bdd ne sont pas rechargées.
  • Finalement l'écriture de tests me pose problème donc je préfère remettre ça à une prochaine MR

Il est à noter qu'il faudra ajouter la permission can_change_custommail au groupe COF en production et uniquement celle ci.

Fixes #93 (closed)

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: Kerl/use_django_custommail