DGNum/gestioCOF
13
2
Fork 1
Code Issues 77 Pull requests 14 Packages Projects Releases 12 Wiki Activity

Utilisation de django_custommail #441

Merged
mpepin merged 0 commits from Kerl/use_django_custommail into master 2017-02-11 16:29:37 +01:00
Conversation 16 Commits - Files changed -
mpepin commented 2016-12-22 04:11:16 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

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

On utilise le package [django_custommail](https://git.eleves.ens.fr/cof-geek/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
mpepin commented 2016-12-22 12:01:25 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

mentioned in merge request !99

mentioned in merge request !99
mpepin commented 2016-12-22 13:12:00 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Added 1 commit:

Added 1 commit: * 5f29caec - cleanup
mpepin commented 2016-12-23 16:08:52 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Added 4 commits:

  • 29801528 - cleanup et nouvelle implem de custommail
  • b39806e1 - Migration et chargement des emails
  • 8bf2f715 - Correction de bugs et nettoyage
  • a5b69019 - Merge branch 'Kerl/use_django_custommail' of git.eleves.ens.fr:cof-geek/gestioCO…
Added 4 commits: * 29801528 - cleanup et nouvelle implem de custommail * b39806e1 - Migration et chargement des emails * 8bf2f715 - Correction de bugs et nettoyage * a5b69019 - Merge branch 'Kerl/use_django_custommail' of git.eleves.ens.fr:cof-geek/gestioCO…
mpepin commented 2016-12-23 16:16:29 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Unmarked this merge request as a Work In Progress

Unmarked this merge request as a Work In Progress
mpepin commented 2016-12-23 18:53:59 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Added 1 commit:

  • 74abd6c8 - Fixe le comportement de syncmail
Added 1 commit: * 74abd6c8 - Fixe le comportement de syncmail
mpepin commented 2016-12-23 18:59:42 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Added 1 commit:

Added 1 commit: * 9482ab44 - Update provisioning
mpepin commented 2016-12-24 14:03:22 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Added 1 commit:

  • dd60c58b - Follows the changes in custommail
Added 1 commit: * dd60c58b - Follows the changes in custommail
mpepin commented 2016-12-24 15:58:20 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Added 1 commit:

Added 1 commit: * c72393d5 - Update readme
mpepin commented 2017-01-06 16:45:53 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Added 8 commits:

  • c72393d5...d6542d8b - 7 commits from branch master
  • cd3faa09 - Merge branch 'master' into Kerl/use_django_custommail
Added 8 commits: * c72393d5...d6542d8b - 7 commits from branch `master` * cd3faa09 - Merge branch 'master' into Kerl/use_django_custommail
mpepin commented 2017-01-13 01:06:08 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Added 66 commits:

  • cd3faa09...697511a8 - 65 commits from branch master
  • 993b8ae8 - Merge branch 'master' into Kerl/use_django_custommail
Added 66 commits: * cd3faa09...697511a8 - 65 commits from branch `master` * 993b8ae8 - Merge branch 'master' into Kerl/use_django_custommail
mpepin commented 2017-02-11 03:20:57 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Added 42 commits:

  • 993b8ae8...06eafdae - 41 commits from branch master
  • 6e559057 - Merge branch 'master' into Kerl/use_django_custommail
Added 42 commits: * 993b8ae8...06eafdae - 41 commits from branch `master` * 6e559057 - Merge branch 'master' into Kerl/use_django_custommail
mpepin commented 2017-02-11 15:14:04 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Added 1 commit:

Added 1 commit: * 0e759405 - Change the migration index
narmanli commented 2017-02-11 15:33:22 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Added 2 commits:

  • 62dc04c1 - cleaning ugly html
  • 5d35112c - Merge branch 'Kerl/use_django_custommail' of git.eleves.ens.fr:cof-geek/gestioCO…
Added 2 commits: * 62dc04c1 - cleaning ugly html * 5d35112c - Merge branch 'Kerl/use_django_custommail' of git.eleves.ens.fr:cof-geek/gestioCO…
mpepin commented 2017-02-11 16:16:17 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Added 2 commits:

  • b4bf4374 - Minor fixes
  • 27bf59d4 - Merge branch 'Kerl/use_django_custommail' of git.eleves.ens.fr:cof-geek/gestioCO…
Added 2 commits: * b4bf4374 - Minor fixes * 27bf59d4 - Merge branch 'Kerl/use_django_custommail' of git.eleves.ens.fr:cof-geek/gestioCO…
narmanli commented 2017-02-11 16:21:29 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Added 2 commits:

  • 924a289a - cleaning UGLY django template style
  • ec7012ee - Merge branch 'Kerl/use_django_custommail' of git.eleves.ens.fr:cof-geek/gestioCO…
Added 2 commits: * 924a289a - cleaning UGLY django template style * ec7012ee - Merge branch 'Kerl/use_django_custommail' of git.eleves.ens.fr:cof-geek/gestioCO…
narmanli commented 2017-02-11 16:29:37 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Status changed to merged

Status changed to merged
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
devtype -- backend

Au programme : du Python, du Django...

  
devtype -- docs

   
devtype -- frontend

   
devtype -- user interface

Au programme : templating, JS, CSS...

  
difficulty -- easy

   
difficulty -- hard

Difficile et/ou risqué

  
difficulty -- normal

   
Doing

   
domain -- bda

Concerne l'application BdA

  
domain -- bds

   
domain -- cof

Concerne l'application COF

  
domain -- core

Dépendances externes, CI, linting...

  
domain -- kfet

Concerne l'application K-Fêt

  
Good first issue

   
priority -- high

Urgent quoi

  
priority -- low

   
priority -- medium

   
priority -- staff-wanted

L'équipe dirigeante du domaine en question souhaite cette fonctionnalité.

  
status -- development

En cours de développement

  
status -- discussion

Avant de coder, c'est bien de discuter

  
status -- need review

En attente de code review

  
status -- production

L'ajout/la modification est présente en production

  
status -- ready to merge

Plus qu'à merger !

  
status -- todo

Le développement devrait bientôt commencer

  
To Do

   
type -- bug

Signalement ou correction de bug

  
type -- hygiene

Tout ce qui amène à du code plus propre et un repo mieux organisé.

  
type -- improvement

Ajout/Demande d'évolution mineure d'une fonctionnalité existante

  
type -- new feature

Ajout/Demande d'une nouvelle fonctionnalité

No labels
devtype -- backend
devtype -- docs
devtype -- frontend
devtype -- user interface
difficulty -- easy
difficulty -- hard
difficulty -- normal
Doing
domain -- bda
domain -- bds
domain -- cof
domain -- core
domain -- kfet
Good first issue
priority -- high
priority -- low
priority -- medium
priority -- staff-wanted
status -- development
status -- discussion
status -- need review
status -- production
status -- ready to merge
status -- todo
To Do
type -- bug
type -- hygiene
type -- improvement
type -- new feature
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
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: DGNum/gestioCOF#441
No description provided.
Delete branch "Kerl/use_django_custommail"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?