Skip to content

WIP: Projet O : Api

DA Damien requested to merge semaine-flop/FlOpEDT:project into api-rest

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))
  • Nous avons remis les settings aux settings par défaut (base de données, ALLOWED_HOST)

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 ?

Quel est le nouveau comportement ?

  • Nettoyage des différentes requêtes : identification des requêtes nécessaires pour l'application mobile et retirer celles qui ne servent pas de la doc
  • Gestion de l'authentification sur les requêtes PUT, DELETE et POST : autoriser tout le monde a effectuer des requêtes de récupérations de données (GET, HEAD, OPTIONS) et autorisation aux tutor seulement de pour voir altérer les données (POST, PUT, PATCH, DELETE)
  • Destinée à : 1) développeurs de l'applications xFlop 2) potentiels développeurs externes

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 PRG

Merge request reports

Loading