Skip to content

Conversation

@BenjaminSere
Copy link
Collaborator

POD_V5 Project Initialization

This Pull Request corresponds to the very first building block of the POD_V5 project.
It initializes the technical base and lays the foundations of the new backend architecture.

Particular care has been taken with:

  • code structure
  • maintainability
  • architectural consistency
  • preparation for (CI/CD, deployment, documentation)

Objectives of this PR

  • Setup of the Django project structure

  • Initial configuration:

    • code
    • environment
  • Addition of basic and technical documentation

  • Initialization of:

    • CI/CD pipelines
    • deployment scripts

Priority review points

As this PR constitutes the base of the project, all feedback is expected, especially regarding:

  • Code architecture
  • Deployment logic
  • Documentation
  • CI/CD configuration
  • Unit tests

Documentation

Detailed documentation accompanies this initialization:

  • README.md at the root
  • docs/ directory

Implemented Endpoint

  • Authentication app

Checklist avant review

BenjaminSere and others added 30 commits November 27, 2025 11:28
… COPYING, SECURITY) and remove obsolete configuration files (nginx.conf, uwsgi.ini)
@Badatos
Copy link
Collaborator

Badatos commented Jan 16, 2026

Bonjour Benjamin,
Merci pour cette première PR V5.
N'hésites-pas à figer la branche et à retirer le statut "Draft" de la PR dès qu'elle est prête pour une relecture ;)

@BenjaminSere BenjaminSere marked this pull request as ready for review January 16, 2026 10:43
@Badatos
Copy link
Collaborator

Badatos commented Jan 16, 2026

@Benjamin-etu-lille je t'invite à lancer la commande suivante :
black . -l 90
(en installant au préalable black)
==> Elle permet automatiquement de mettre en forme le code python, de manière à ce qu'il soit relativement uniformisé.

Sur la v4, c'est fait automatiquement dans ce workflow :
https://github.com/EsupPortail/Esup-Pod/blob/main/.github/workflows/code_formatting.yml

Copy link
Collaborator

@Badatos Badatos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Voila déja une première vague de commentaires.
La suite la semaine pro ;)

| :--- | :---: | :--- | :--- |
| **`DJANGO_SETTINGS_MODULE`** | ✅ | `config.django.base` | Python path to settings module. (e.g. `config.django.dev.docker`) |
| **`SECRET_KEY`** | ✅ | *(None)* | Django security key. **Must be secret in production.** |
| **`VERSION`** | ❌ | `5.0.0-DEV` | Application version tag. |
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pourquoi placer le numéro de version en variable d'environnement ?
A priori, c'est le code qui définit la version, et non la config. 🤔

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

C'est pour que la version soit synchronisée avec la CI/CD et l'ensemble des services (DB et API). Cela évite de devoir faire un commit juste pour changer la version, et on peut même configurer la CI/CD pour qu'elle crée automatiquement les tags en cas de changement de version majeur.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants