1597 - Side PPA by celine-m-s · Pull Request #1612 · MTES-MCT/Docurba

celine-m-s · 2025-11-18T13:29:21Z

À faire :

Après la MEP :

Créer le groupe Écriture et assigner les bonnes permissions.
Changer les commentaires des colonnes en base de données.

Accorder les droits PPA

Se connecter à l'admin Django et cliquer sur « Profils ». Chercher un utilisateur par email ou grâce aux filtres de droite. Cliquer sur le nom pour modifier.
Champ departements : La liste des départements auquel peut accéder la région doit être du format suivant : 09,11,12 etc. Les départements d'un chiffre doivent être commencés par un zéro, il ne faut pas mettre de guillemets et il faut séparer les valeurs par une virgule.
Champ departement : indiquer le département par défaut de l'utilisateur (pareil que pour les DDT). Il doit être du format 09 (un chiffre) ou 11 (deux chiffres).

Tests

Lien vers la recette jetable

## Utilisateurs de test

Communiqués en privé.

Cas de test

En tant qu'administrateur, je me connecte à l'admin Django puis je modifie un utilisateur qui devrait être PPA.
En tant que PPA, je me connecte grâce à mon identifiant et mot de passe sur la page de connexion État. Je devrais être redirigé vers le tableau de bord du side État.

Notes Céline :

Données : Side : 'ppa', Poste : différent de 'DDT'
Recette : ajouter les compartiments S3 doc-events-attachements, projects-annexes (?) et text-images (?)

# Admin Django

Anthony, j'ai ajouté un groupe « Écriture » pour gérer les profils :

J'ai aussi modifié le groupe « Lecture seule » :

Cas d'usage et comportement souhaité

Appliquer les droits et l’affichage suivant :

masquer les rubriques “Trame de PAC départementale” et “Mes PAC”
droits sur les procédures :
- lire les événements publics
- Ne peut pas inviter
- Ne peut pas ajouter, modifier ou supprimer d'événement (public ou privé)
- ne peut pas créer de procédure
- ne peut pas créer de procédure secondaire
- ne peut pas supprimer de procédure

Droits en tant qu'invité :

Appliquer le périmètre géographique :

code Région (similaire DREAL - pour cette V1 : région IDF)
avec un filtre par département (cf autre sujet ci-dessus)→ après connexion, affichage de la page Mes Collectivités sur le périmètre d’un département de la région (pour la v0)

Afficher dans la liste des collaborateurs d’une feuille de route :

Créer un nouveau poste : “Région”
Prénom NOM
Région (= poste) Île-de-France (= nom de la région correspondant au code Région)

#1597
#1625
#1626
https://www.notion.so/docurba/Filtre-d-partement-22cbe4d354c0800ea023f19396a225c7

HermanceGauth · 2025-12-05T14:58:30Z

ne peut pas modifier de procédure : à voir avec Camille car je ne sais pas ce qui est demandé. Ou alors le « ne peut pas créer, modifier ou supprimer » concerne les procédures secondaires seulement ?

Effectivement à voir avec Camille, car si c'est "ne peut pas modifier les paramètres de la procédure" => aujourd'hui personne ne peut le faire.

celine-m-s · 2025-12-16T17:02:44Z

Merci pour vos retours @cami-na , @CeliaVermicelli et @Octaviedebs !
Octavie, je t'ai répondu dans la carte dédiée au souci. J'ai modifié le statut en KO|OK. La priorisation sera à discuter.

HermanceGauth · 2025-12-17T15:24:16Z

J'ai testé l'export avec l'autre PR qui donne accès au tableau de bord et ça marche nickel!

Sur l'impossibilité de créer une procédure : c'est effectivement pas possible, mais je me demande, si cela est rapide, si on peut mettre un petit message d'explication en même temps que la redirection, je ne sais pas ce que tu en penses @cami-na pour éviter de faire penser à un bug?

Sur le parcours, je ne sais pas où mettre ce retour parce que je l'ai vu en utilisant l'autre PR qui donne accès au tableau de bord mais cela concerne plus le side PPA que le filtre en lui-même @celine-m-s n'hésite pas à me dire s'il faut que je décale :

J'ai remarqué que si on clique sur une FRP et qu'on revient sur la page collectivité, c'est une page collectivité type "side collectivité" et non type "side Etat" et qu'il n'y a donc plus de retour vers le tableau de bord.

Page Collectivité side PPA en cliquant directement sur la collectivité (OK)

Page Collectivité side PPA en cliquant sur FRP puis retour à la collectivité (manque le retour vers le tableau de bord)

Avec ce même dernier parcours side Etat (prod et recette jetable) OK

Octavie avait fait un retour un peu similaire dans cette PR : #1540 (comment)

rik

Très cool d'avoir centraliser en partie la gestion des permissions et d'avoir ajouter des vérifications de permissions au chargement des pages elles-mêmes.

Et très cool d'avoir modéliser profiles pour y avoir accès dans Django Admin.

django/users/models.py

django/users/apps.py

django/users/enums.py

django/users/models.py

nuxt/plugins/user.js

nuxt/plugins/utils.js

celine-m-s · 2025-12-18T16:05:39Z

Merci pour la revue ! J'ai adressé la majorité des commentaires. Ceux restés sans réponse feront l'objet d'un correctif à la rentrée.

celine-m-s · 2026-01-06T18:16:57Z

@rik J'ai fini de traiter tes retours. Ils sont dans des commits séparés.
Je viens de tester un nouvel outil, git absorb, donc c'est un peu le bazar dans les commits. Désolée !
Les commits à prendre en compte sont ceux qui ne commencent pas par Django ou Nuxt. Ils seront tous fusionnés dans les commits d'origine.

rik

Il me semble indispensable de bien gérer le cas de Profile.created_at avant de mettre ça en production pour éviter des pertes de données.

J'aimerais aussi que soit traité le cas des erreurs de linter empêchant de travailler en local.

Pour les autres sujets, je te laisse en responsabilité.

django/users/migrations/0001_initial.py

django/users/models.py

nuxt/layouts/ddt.vue

django/users/models.py

nuxt/pages/collectivites/_collectiviteId/procedures/add.vue

nuxt/pages/ddt/_departement/collectivites/_collectiviteId/procedure/add.vue

nuxt/plugins/user.js

Documentation du schéma actuel de la base de données. Ajout d'une application pour distinguer la gestion des utilisateurs du reste de l'applicatif. Ajout des entrées dans l'interface d'administration Django.

Les règles de suppression sont actuellement éparpillées. Le besoin actuel est de modifier le comportement à la marge pour cacher le bouton aux sides PPA, or je pense que l'harmonisation des règles nécessiterait une PR en soi. Pour l'instant, faisons simple.

La clé primaire du modèle Collectivite est de la forme "{code_insee}_{type}", or le format qui est dans la colonne Profile.collectivite_id est "{code_insee}". En attendant de corriger ce souci, affichons dans l'admin le contenu de la colonne mais en lecture seule.

sonarqubecloud · 2026-01-09T22:43:27Z

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

celine-m-s · 2026-01-09T23:16:47Z

@rik Pour info, j'ai ajouté un commit pour corriger un bug. Le dernier commit (move me) sera supprimé.

celine-m-s self-assigned this Nov 18, 2025

celine-m-s linked an issue Nov 18, 2025 that may be closed by this pull request

Sizer les scénarios pour la création du side PPA et le filtre département #1597

Closed

celine-m-s force-pushed the side-ppa branch from d707717 to 06b4063 Compare December 2, 2025 18:22

celine-m-s changed the base branch from main to xo-linter December 2, 2025 18:31

celine-m-s force-pushed the xo-linter branch from a407c54 to 66dad0c Compare December 2, 2025 18:33

celine-m-s force-pushed the side-ppa branch 2 times, most recently from 807dcd2 to ca01bdd Compare December 3, 2025 17:18

celine-m-s force-pushed the side-ppa branch 4 times, most recently from 69d7f77 to 248ca7f Compare December 10, 2025 09:02

celine-m-s force-pushed the xo-linter branch from 66dad0c to 29cb543 Compare December 10, 2025 09:15

celine-m-s changed the base branch from xo-linter to celinems/local-supabase December 10, 2025 09:16

celine-m-s changed the base branch from celinems/local-supabase to review_apps December 10, 2025 09:16

celine-m-s force-pushed the side-ppa branch from 248ca7f to 56cea5a Compare December 10, 2025 09:19

celine-m-s force-pushed the review_apps branch from 72c5cd4 to c5b7aae Compare December 10, 2025 11:47

celine-m-s added recette-jetable Crée une recette jetable and removed recette-jetable Crée une recette jetable labels Dec 10, 2025

celine-m-s closed this Dec 10, 2025

celine-m-s reopened this Dec 10, 2025

celine-m-s force-pushed the review_apps branch from 033477b to 32abccb Compare December 10, 2025 13:16

celine-m-s added recette-jetable Crée une recette jetable and removed recette-jetable Crée une recette jetable labels Dec 10, 2025

celine-m-s closed this Dec 10, 2025

celine-m-s removed the recette-jetable Crée une recette jetable label Dec 10, 2025

celine-m-s reopened this Dec 10, 2025

rik mentioned this pull request Dec 18, 2025

Remplacer Retool par Django Admin #1688

Closed

rik suggested changes Dec 18, 2025

View reviewed changes

This was referenced Jan 6, 2026

Django admin : pouvoir sélectionner la collectivité #1709

Closed

Technique Nuxt : refondre is_admin #1710

Open

rik suggested changes Jan 7, 2026

View reviewed changes

Nuxt : suppression de code commenté

b9e9abe

rik approved these changes Jan 8, 2026

View reviewed changes

celine-m-s added 16 commits January 8, 2026 15:50

Django : ajout des modèles User et Profile

b612ea4

Documentation du schéma actuel de la base de données. Ajout d'une application pour distinguer la gestion des utilisateurs du reste de l'applicatif. Ajout des entrées dans l'interface d'administration Django.

Django : ajout du side PPA à ProfileSideType

d7d782a

Django : ajout de la colonne "users.departements"

8cf22a6

Nuxt : les side PPA peuvent accéder au tableau de bord

f7e2e8f

Nuxt : les PPA ne peuvent pas créer de procédure

c5ea35d

Nuxt : les sides PPA ne peuvent pas créer de procédure secondaire

2f4669d

Django : Users — ajout du poste Région

441ade7

Nuxt : affichage du poste Région + nom de la région

3ea290c

Nuxt : séparer la validation du profil de son side

6683acc

Nuxt : utilisation du code 302 pour indiquer une redirection

912398e

Nuxt : éviter une erreur locale dans la console

5bc3472

Django : re-génération de la migration initiale de users

13fdde9

Nuxt : affichage du lien vers le tableau de bord des DDT pour les PPA

d0f98b7

move me

6196a53

celine-m-s mentioned this pull request Jan 12, 2026

Side PPA : mise en prod #1716

Merged

Conversation

celine-m-s commented Nov 18, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Accorder les droits PPA

Tests

Cas de test

Notes Céline :

Cas d'usage et comportement souhaité

Uh oh!

HermanceGauth commented Dec 5, 2025

Uh oh!

celine-m-s commented Dec 16, 2025

Uh oh!

HermanceGauth commented Dec 17, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

rik left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

celine-m-s commented Dec 18, 2025

Uh oh!

celine-m-s commented Jan 6, 2026

Uh oh!

rik left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

sonarqubecloud bot commented Jan 9, 2026

Quality Gate passed

Uh oh!

celine-m-s commented Jan 9, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

celine-m-s commented Nov 18, 2025 •

edited

Loading

HermanceGauth commented Dec 17, 2025 •

edited

Loading

celine-m-s commented Jan 9, 2026 •

edited

Loading