DGNum/gestioCOF
13
2
Fork 2
Code Issues 82 Pull requests 18 Projects Releases 12 Packages Wiki Activity Actions

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é

  
bug

Something is not working

  
duplicate

This issue or pull request already exists

  
enhancement

New feature

  
help wanted

Need some help

  
invalid

Something is wrong

  
question

More information is needed

  
wontfix

This won't be fixed

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
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
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?