-
Notifications
You must be signed in to change notification settings - Fork 53
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
[TECH] Ajouter un commentaire interne jury sans recréer de nouvelle entrée dans assessment-results (PIX-11049) #9374
Conversation
Une fois les applications déployées, elles seront accessibles via les liens suivants :
Les variables d'environnement seront accessibles via les liens suivants : |
b449b1a
to
7682449
Compare
27f727a
to
6c986c8
Compare
api/src/certification/session-management/domain/models/AssessmentResultJuryComment.js
Outdated
Show resolved
Hide resolved
.../session-management/infrastructure/repositories/assessment-result-jury-comment-repository.js
Outdated
Show resolved
Hide resolved
.../session-management/infrastructure/repositories/assessment-result-jury-comment-repository.js
Outdated
Show resolved
Hide resolved
admin/app/controllers/authenticated/certifications/certification/informations.js
Outdated
Show resolved
Hide resolved
* @param {number} juryId | ||
*/ | ||
constructor({ id, commentByJury, juryId } = {}) { | ||
this.id = id; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Je suis perturbé par l'id
qui ne fait pas partie de l'entité métier AssessmentResultJuryComment
(car c'est l'id de l'assessment result) mais qui sert plutôt techniquement à découper au sein du usecase la modification du commentaire de l'assessment result en deux parties (deux appels aux repository avec getLatestAssessmentResult
et save
).
Est-ce pertinent de le retirer et de n'avoir qu'un seul et unique appel vers le repository (par exemple avec une fonction updateLatestAssessmentResultJuryComment
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A moins de renommer id
en assessmentResultId
mais dans l'absolu, le fait de tout faire en une méthode de répo ne me paraît pas mal
6c986c8
to
23f2ada
Compare
.../session-management/infrastructure/repositories/assessment-result-jury-comment-repository.js
Outdated
Show resolved
Hide resolved
c8fedbf
to
79d48f8
Compare
a709936
to
ca8e5ee
Compare
ca8e5ee
to
74f759b
Compare
74f759b
to
77a4119
Compare
77a4119
to
78de1b8
Compare
78de1b8
to
24204cd
Compare
close car sujet dépriorisé. Je ne delete pas la branch car on souhaite la réutilisé quand le sujet sera repris. |
🦄 Problème
A chaque MAJ du commentaire jury, on recree un
assessment-results
ainsi que lescompetence-marks
associé.🤖 Proposition
Mettre à jour l'assessment-results courant
🌈 Remarques
certification-courses
plutot queassessment-results
assessment-results
, on ne peut pas ajouter de commentaire avant le 1er scoring💯 Pour tester
Dans admin sur le detail d'une certif, modifier plusieurs fois le commentaire jury.
S'assurer qu'il est mis à jour
Verifier en base qu'il n'y a pas de nouveaux assessment-result
Relancer un scoring
S'assurer que le commentaire est conservé