[FEATURE] Réferencer l'id du module plutôt que son slug (PIX-17403) #12033
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 : |
| const id = 'fake-uuid'; | ||
|
|
||
| // when & then | ||
| await expect(moduleDatasource.getById(id)).to.have.been.rejectedWith(LearningContentResourceNotFound); |
There was a problem hiding this comment.
Tiens je connaissais pas 🤩
Une alternative à catchErr ?
There was a problem hiding this comment.
ahh ouais p'tet bien, c'est du copié collé
| @@ -1,10 +1,10 @@ | |||
| import { requestResponseUtils } from '../../../shared/infrastructure/utils/request-response-utils.js'; | |||
|
|
|||
| const create = async function (request, h, { usecases, passageSerializer }) { | |||
| const { 'module-id': moduleId } = request.payload.data.attributes; | |||
| const { 'module-id': moduleSlug } = request.payload.data.attributes; | |||
There was a problem hiding this comment.
Remarque
Est-ce qu'il ne faudrait pas plutôt modifier le front pour passer un moduleId dans le champ request.payload.data.attributes.module-id ?
- Cela évite d'avoir
moduleSlugcôté controller puis lemoduleIdcôté usecase - Permet, dans le usecase
createPassage, d'utiliser la fonctionmoduleDatasource.getByIdau lieu demoduleDatasource.getBySlug - Fait que l'id est le seul identifiant technique qu'on utilise de bout en bout 😄
C'est pas bloquant et peut être fait dans une autre PR si besoin
There was a problem hiding this comment.
dans un monde idéal ouais mais ça nécessiterai de casser l'URL d'accès aux modules qui utilise le slug 😅 c'est une feature qu'on veut garder pour l'instant
pix-service-auto-merge
force-pushed
the
PIX-17403-passage-references-module-id-instead-of-slug
branch
from
8eac794 to
3c81d18
Compare
pix-service-auto-merge
deleted the
PIX-17403-passage-references-module-id-instead-of-slug
branch
🌸 Problème
Les slugs de module ont tendance à évoluer. C'est gênant pour nos données car on a un lien passage-module qui casse quand ça arrive.
🌳 Proposition
Dans les passages, le moduleId devient le champ
iddu module.🐝 Remarques
Impact imprévu : ça a cassé la vérification de réponses, voir les modifs dans
element-repository.js.🤧 Pour tester
Non reg passage de module sur RA.
S'assurer qu'il n'y a pas d'erreur en console.