Skip to content
This repository has been archived by the owner on Feb 18, 2021. It is now read-only.

Remise en forme du projet #1

Open
12 of 22 tasks
Lgt2x opened this issue Dec 20, 2020 · 1 comment
Open
12 of 22 tasks

Remise en forme du projet #1

Lgt2x opened this issue Dec 20, 2020 · 1 comment

Comments

@Lgt2x
Copy link
Member

Lgt2x commented Dec 20, 2020

Certains problèmes de conception de la base de code doivent être revus avant que le projet puisse être repris en PWEB au S2.

À un niveau général :

  • Mettre à jour les dépendances des modules Python et vérifier que ça ne casse rien
  • Faire en sorte de ne pas avoir un projet imbriqué dans un autre avec 2 README, 2 fichiers requirements etc.
  • Mettre en place un Dockerfile/docker-compose pour le déploiement
  • Préférer nginx à coffeescript pour la gestion des fichiers statiques
  • Mettre en place une Github Action pour vérifier le formatage du code : au moins pylint + flake8, après avoir lancé Black sur toute la codebase
  • Réorganiser les dossiers du projet pour avoir les templates, les fichiers de config du projet et l'app principale dans des dossiers séparés
  • Remplacer MySQL par Postgres

À un niveau plus spécifique à Django, pour être cohérent avec les bonnes pratiques du framework

  • Avoir un seul fichier settings.py unifié et utiliser des variables d'environnement pour le déploiement
  • Utiliser l'admin Django plutôt que de recommander de créer les objets via Django shell
  • Changer les URL pour ne pas avoir de préfixe sur toutes les pages
  • Mettre la vue de la page d'accueil dans l'application et pas dans les vues du projet pour unifier l'origine des vues
  • Standardiser les modèles : casse des noms de champ et modèles
  • Réduire les migrations pour en avoir plus que 2 : avant et après ces changement sur les modèles
  • Créer un nouveau fichier pour les choices des modèles et ne pas polluer ce fichier déjà long
  • Enlever certains duplicatas de modèles inutiles
  • Séparer les vues en plusieurs fichiers
  • peut-être d'autres incohérences que j'ai pas encore relevé

Enfin pour la gestion du projet :

  • Transformer le cahier des charges en issues et projects Github
  • Transformer la description de l'existant en Wiki Github
  • Mettre à jour ce Wiki suite aux modifs demandées par cette issue
  • Réécrire le readme pour refléter les nouveaux moyens de déployer et développer l'app
  • Faire une release/tag avant et après la PR qui va régler tout ça

Une PR devrait suivre

@titouan-joseph
Copy link
Member

Stuv regarder un coup, aux 24h on a fait un outil pour push des issues sur base d'US ici

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants