Skip to content

WIP: Projet G : Proposition de Modification

M4trixFr requested to merge M4trixFr/FlOpEDT:dev into dev

Liste des choses à vérifier

Merci de vérifier is votre Merge Request (MR) satisfait bien aux conditions suivantes (mettre un 'x' pour cocher la case) :

  • Nous avons ajouté nos noms dans le fichier AUTHORS.md
  • Nous avons ajouté quelques mots d'anglais dans le CHANGELOG.md pour résumer notre travail
  • Si nous avons utilisé de nouveaux modules python, nous les avons ajouté dans le fichier requirements.txt
  • Nous avons effectué quelques tests (et les avons éventuellement ajouté au dépôt)
  • Nous avons ajouté, modifié et revu la documentation, si nécessaire. On peut effectuer une MR sur la page wiki de FlOpEDT, comme expliqué ici : https://stackoverflow.com/a/38537453
  • Nous n'avons pas ajouté d'affichage dans les consoles (ni dans celle de django (avec print), ni dans celle du mavigateur (avec des console.log))

Type de Merge Request

Merci d'indiquer le type de votre MR (mettre un 'x' pour cocher la case) :

  • Réparation de bug
  • Fonctionnalité
  • Mise à jour de style sur le code (formatage, renommage)
  • Refactoring (pas de changement fonctionnel, ni de changement d'API)
  • Autre (à décrire) :

Quel est le comportement actuel ?

Flop!EDT permet aux étudiant⋅e⋅s de faire une proposition de modification à valider par les profs concerné⋅e⋅s.

Quel est le nouveau comportement ?

Il s’agirait d'améliorer le contenu du mail, et de faire un système de validation par les enseignant⋅e⋅s concerné⋅e⋅s

L'objectif est que le professeur reçoivent un mail de confirmation avec 2 boutons où il pourra confirmer ou refuser la modification. En fonction de son choix un mail sera renvoyé a l'initiateur qui a fait la demande et le changement sera fait automatiquement. (Le cours bouge si il accepte le changement)

Travail restant

  • Debugger l'envoie de mail automatique
  • Il faut ajouter la fonctionnalité pour bouger le cours si le professeur clique sur valider

Est-ce que cette MR casse une autre fonctionnalité ?

  • Oui
  • Non

Est-ce que cette MR contient de nouvelles migrations de base ?

  • Oui
  • Non

Informations supplémentaires ?

Edited by M4trixFr

Merge request reports