-
Notifications
You must be signed in to change notification settings - Fork 58
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] ajoute la méthode findSnapshotByCampaignParticipationId (PIX-16339) #11284
[TECH] ajoute la méthode findSnapshotByCampaignParticipationId (PIX-16339) #11284
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 : |
ad36fc0
to
710280a
Compare
7d70f28
to
0cb19c2
Compare
408df7b
to
96b9f0f
Compare
e0022db
to
fe6aae2
Compare
4681ec6
to
8aa0641
Compare
bfe3898
to
8cb4682
Compare
...n-participation/integration/infrastructure/repositories/campaign-analysis-repository_test.js
Outdated
Show resolved
Hide resolved
api/tests/shared/integration/domain/services/placement-profile-service_test.js
Outdated
Show resolved
Hide resolved
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.
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.
tech review ok ✅
.../campaign-participation/infrastructure/repositories/participant-results-shared-repository.js
Outdated
Show resolved
Hide resolved
api/tests/shared/integration/domain/services/placement-profile-service_test.js
Outdated
Show resolved
Hide resolved
…nce in campaignAssessmentParticipationResult
…tSnapshotRepository method
fdb12d1
to
4672dd8
Compare
🥞 Problème
Dans le cadre de l'accès au
knowledge-element-snapshots
avec lecampaignId
;on souhaite éviter d'utiliser
findSnapshotBuUserId
qui mélange récupération des snapshot et des ke ) partir du userIdvu qu'on ne pourra plus récupérer les snapshot qu'avec un campaignParticipationId
🥓 Proposition
On ajoute une méthode pour récupérer les snapshot depuis un campaignParticipationId et on met à jour la où sont récupérer les snapshots pour utiliser un campaignParticiaptionId
Là où on attend que des résultats de knowledgeElements remontent sans qu'ils soient liés à une participation partagée, on va chercher les knowledge elements directement (sans faire un appel préalable aux snapshots, comme on faisait auparavant). Routes concernées :
/api/campaigns/{campaignId}/profiles-collection-participations/{campaignParticipationId}
'/api/campaigns/{campaignId}/assessment-participations/{campaignParticipationId}'
/api/campaigns/{campaignId}/assessment-participations/{campaignParticipationId}/results
🧃 Remarques
C'est essentiellement du refacto, on souhaite modifier à minima les tests (pour rajouter un campaignParticipationId au snapshots)
😋 Pour tester
RAS