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

CP V7 bug #12592: fix error 500 when unselectcontract management #1723

Conversation

mohatizaoui
Copy link
Contributor

Description

Description des modifications

Type de changement:

Indiquer le ou les types de changements

  • Build

  • PKI

  • Ansiblerie

  • Nouveau Code

  • Correction

  • Refactorisation de code

  • Autre

Documentation:

Indiquer la documentation mise à jour

[ ] Quels sont les nouvelles documentations ?

[ ] Quels sont les modifications existantes ?

[ ] Quels sont les documentations ou sections de documentations supprimés ?

Tests:

Indiquer comment le code à été testé (manuel, environnement, TU, etc)

manuel

environnement

TU

Migration:

Indiquer si les modifications apportées impliquent une migration sur l'existant et comment la faire

Checklist:

Sélectionner les éléments de la checklist

[ ] Mon code suit le style de code de ce projet.

[ ] J'ai commenté mon code, en particulier dans les classes et les méthodes difficile à comprendre.

[ ] J'ai fait les changements correspondant dans la documentation RAML.

[ ] J'ai fait les changements correspondant dans la documentation Métier.

[ ] J'ai fait les changements correspondant dans la documentation Technique.

[ ] J'ai rajouté les tests unitaires vérifiant mes fonctionnalités.

[ ] J'ai rajouté les tests de non régression vérifiant mes fonctionnalités.

[ ] Les tests unitaires nouveaux et existants passent avec succès localement.

[ ] Toutes les dépendances ont été mergées en priorité

Contributeur

Indiquer qui a développé cette fonctionnalité

VAS (Vitam Accessible en Service)

CEA (Commissariat à l'énergie atomique et aux énergies alternatives)

@mohatizaoui mohatizaoui added bug Something isn't working small pr embarquant peu de changements et à review rapide, ne nécessitant qu'un reviewer Cherry-Pick a cherry pick labels Mar 27, 2024
@mohatizaoui mohatizaoui changed the title bug #12592: fix error 500 when unselectcontract management CP7 bug #12592: fix error 500 when unselectcontract management Mar 27, 2024
@GiooDev GiooDev added this to the IT 133 milestone Mar 27, 2024
@marob marob changed the title CP7 bug #12592: fix error 500 when unselectcontract management CP V7 bug #12592: fix error 500 when unselectcontract management Mar 27, 2024
Copy link
Contributor

@marob marob left a comment

Choose a reason for hiding this comment

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

Il manque l'utilisation de la clé i18n dans le template :

<mat-option [value]="null"> {{ 'CONTRACT_MANAGEMENT.CONTRACTS_CREATION.SELECT_MANAGEMENT_CONTRACT' | translate }}</mat-option>

@mohatizaoui mohatizaoui force-pushed the CP7_BUG_12592_fix_error_500_unselect_management_contract branch from 4361cf3 to 0cc3ae4 Compare March 29, 2024 15:26
@mohatizaoui mohatizaoui requested a review from marob March 29, 2024 15:30
@mohatizaoui mohatizaoui merged commit 81aa95b into master_7.0.x Mar 29, 2024
@mohatizaoui mohatizaoui deleted the CP7_BUG_12592_fix_error_500_unselect_management_contract branch March 29, 2024 17:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Cherry-Pick a cherry pick small pr embarquant peu de changements et à review rapide, ne nécessitant qu'un reviewer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants