Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Back] Traitement des fichiers lors de la soumission d'un signalement #1940

Merged
merged 4 commits into from
Nov 17, 2023

Conversation

sfinx13
Copy link
Collaborator

@sfinx13 sfinx13 commented Nov 14, 2023

Ticket

#1668

Description

Liaison des fichiers au signalement lors de la validation nouveau formulaire

Changements apportés

  • Ajout de nouvelles colonnes dans la table file pour stocker le type de document et le slug du désordre
  • Ajout d'un nouveau type afin de catégoriser les documents
  • Traitement des fichiers dans le builder

Pré-requis

Tests

  • Soumettre avec les différents type de profil un signalement avec des fichiers via le nouveau formulaire
  • Vérifier que les photos s'affichent dans la fiche signalement
  • Vérifier que les nouvelles info sont bien en base

image

@sfinx13 sfinx13 changed the title [Back] Traitement des fichiers lors de la soumission d'un signalement #1668 Formulaire 2.0 [Back] Traitement des fichiers lors de la soumission d'un signalement Nov 14, 2023
@numew
Copy link
Collaborator

numew commented Nov 15, 2023

tests et relecture ok

@emilschn
Copy link
Collaborator

En dehors de mon message, test et relecture OK. On voit en direct !

Copy link

sonarcloud bot commented Nov 16, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 2 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@emilschn emilschn merged commit eebee3d into develop Nov 17, 2023
4 checks passed
@sfinx13 sfinx13 deleted the feature/1668-handle-file-new-form branch November 21, 2023 23:50
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