Skip to content

feature/seed-v3 : add 1 seed file for mysql and 3 files for mongodb#16

Open
Ibrah131 wants to merge 3 commits intodevelopmentfrom
feature/seed-v3
Open

feature/seed-v3 : add 1 seed file for mysql and 3 files for mongodb#16
Ibrah131 wants to merge 3 commits intodevelopmentfrom
feature/seed-v3

Conversation

@Ibrah131
Copy link
Copy Markdown
Collaborator

@Ibrah131 Ibrah131 commented Oct 7, 2025

Branche : feature/seed-v3 (créée depuis development)
Scope : Ajout de seeds Mongo + SQL. Aucun fichier existant modifié.

La table de logs n’existait pas : j’ai proposé un exemple de table activity_log+ 15 logs métier (changement de statut, affectation, SLA, etc.). A voir.

@Ibrah131 Ibrah131 changed the title add 1 seed file for mysql and 3 files for mongodb feature/seed-v3 : add 1 seed file for mysql and 3 files for mongodb Oct 7, 2025
Copy link
Copy Markdown
Owner

@julie-belrose julie-belrose left a comment

Choose a reason for hiding this comment

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

Globalement l'ensemble c'est niquel, il y a quelques petits erreurs de logique de type d'attribut. Par contre, tu n'es pas obligé de faire énormément de données seed 😉

Comment thread backend/src/database/mysql/seeds/seed.sql
(13,13,'sarah.mu','Lyon 6e - Foch',99),
(14,14,'thomas.c','Oullins',77),
(15,15,'ines.lm','Bron',74),
-- Agents 16..30
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

ici ce ne sont que des citoyens

@Pcedric95
Copy link
Copy Markdown
Collaborator

Rien à signaler — seeds clairs et cohérents.
À exécuter une seule fois sur base vide (non idempotents, normal dans notre contexte).

"recipient_user_id": 32,
"recipient_role": "citizen",
"title": "Prise en charge",
"content": "Votre signalement est en cours par l’Équipe Vaise B.",
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Il y a une incohérence avec le MCD :

🔑 _id: ObjectId [1]            |
| user_id: int [1]                |
| title: string [1]               |
| content: string [1]             |
| target_type: string [0..1]      |
| target_id: ObjectId [0..1]      |
| read: boolean [1]               |
| created_at: datetime [1]

"agent_id": 201,
"assigned_by": 1,
"assigned_team": { "id": 301, "name": "Équipe Gerland A", "center_id": 1 },
"status": "ouvert",
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

il manque juste le champs, updated_at , oublier dans le MCD mais nécessaire pour les report et les logs 😉

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