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

[BO - Signalement] Ajout d'une liste de motifs de refus d'affectation ou signalement #1861

Merged
merged 4 commits into from
Nov 3, 2023

Conversation

emilschn
Copy link
Collaborator

Ticket

#946

Description

Ajout de liste de motifs de refus pour

  • la modale de refus d'affectation
  • la modale de refus de signalement

Pour l'instant, c'est un enregistrement, on n'affiche la sélection nulle part

Changements apportés

  • Ajout d'un champ motifRefus dans la table affectation et la table signalement

Pré-requis

make load-migrations

Tests

Refuser une affectation

  • Refuser une affectation avec un partenaire
  • Vérifier la liste de motifs
  • Vérifier qu'on ne peut pas valider le formulaire sans sélectionner
  • Vérifier l'enregistrement en BDD

Refuser un signalement

  • Refuser un signalement avec un RT
  • Vérifier la liste des motifs
  • Vérifier qu'on ne peut pas valider le formulaire sans sélectionner
  • Vérifier l'enregistrement en BDD

@emilschn emilschn requested a review from sfinx13 October 31, 2023 14:22
src/Manager/AffectationManager.php Outdated Show resolved Hide resolved
@sfinx13 sfinx13 self-requested a review November 2, 2023 09:54
Copy link
Collaborator

@sfinx13 sfinx13 left a comment

Choose a reason for hiding this comment

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

Pour l'instant, c'est un enregistrement, on n'affiche la sélection nulle part
@emilschn

Pourquoi ne pas l'afficher dans le suivi ?

@sfinx13 sfinx13 self-requested a review November 2, 2023 09:55
Copy link
Collaborator

@sfinx13 sfinx13 left a comment

Choose a reason for hiding this comment

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

Mettre à jour les fixtures de signalement ou affectation refusé avec un motif de cloture

Copy link
Collaborator

@sfinx13 sfinx13 left a comment

Choose a reason for hiding this comment

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

@emilschn Bizarre, on ne peut plus valider de signalement sur cette branche, sur develop c'es ok

@emilschn
Copy link
Collaborator Author

emilschn commented Nov 2, 2023

Pour l'instant, c'est un enregistrement, on n'affiche la sélection nulle part
@emilschn

Pourquoi ne pas l'afficher dans le suivi ?

Parce que c'est ainsi que l'a dit Mathilde :)

@emilschn
Copy link
Collaborator Author

emilschn commented Nov 2, 2023

@emilschn Bizarre, on ne peut plus valider de signalement sur cette branche, sur develop c'es ok

Tu as bien joué les migrations ?

Copy link

sonarcloud bot commented Nov 3, 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 1 Code Smell

No Coverage information No Coverage information
0.0% 0.0% Duplication

@sfinx13 sfinx13 merged commit b935226 into develop Nov 3, 2023
4 checks passed
@emilschn emilschn deleted the feature/946-categories-refus branch November 3, 2023 14:34
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.

None yet

2 participants