Skip to content

Check Modif and JSON creation

Will Touz requested to merge Fugyud1/flop-edt-dnf: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) :

  • [ x ] Le code et les commentaires sont en anglais
  • 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és 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 navigateur (avec des console.log))

Nouvelles dépendances

Des dépendances nouvelles ont-elles été ajoutées ?

  • Oui
  • [ x ] Non

Si oui, lesquelles ?

Licences

Si de nouvelles dépendances ont été ajoutées, leurs licences sont compatibles avec une inclusion dans un logiciel sous AGPL.

  • Tout à fait
  • J'ai regardé, mais j'ai un doute. Franchement, les licences, c'est compliqué.
  • Non, mais il existe une alternative.

Si non, quelle(s) alternative(s) ?

Type de Merge Request

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

  • Réparation de bug
  • [ x ] 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 ?

Un script php extérieur vient vérifier les données modifiées et envoie un mail.

Quel est le nouveau comportement ?

-Un script, "check_modif.py" permet de stocker une sauvegarde des champs importants de la BD et de faire la vérification entre les anciennes et nouvelles données pour les stocker dans un fichier JSON (A paufiner). Un bot discord est aussi présent pour faire l'envoie des messages suivant les modifications (toujours en cours)

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

  • Oui
  • [ x ] Non

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

  • [ x ] Oui
  • Non

Informations supplémentaires ?

La fonctionnalitée nécéssite encore des modifications. On pourrait donner le nombre de semaine que l'on veut stocker. Il faut modifier la requête de récupération des cours pour prendre en compte la fin de l'année. Peut etre mettre en place un système de transaction pour la backup pour éviter de tout perdre si il y a un probleme. Il faut rajouter les supp_tutor dans la vérification des modifications pour l'envoie potentiel de mail.

Il faut aussi continuer le bot pour afficher les messages.

Nous allons continuer le projet en dehors de la semaine FLOP.

Edited by Will Touz

Merge request reports