Conversation
|
| type Props = { | ||
| source: LunaticSource; | ||
| data: LunaticData; | ||
| data: PdfRequestFromBody; |
There was a problem hiding this comment.
A retirer, laisser les lunaticData et ajouter un autre paramètre interrogationInfos ou renommer en objet "interrogation" pour plus de clarté. (cf ma remarque sur generateStream)
Donc par exemple, typer comme ceci
type Props = {
source: LunaticSource,
data: LunaticData,
interrogationInfo: InterrogationInfo
}
const data = data;
const surveyTitle = InterrogationInfo.collectionInstrumentId;
const usualSurveyUnitId = InterrogationInfo.usualSurveyUnitId;
const validationDate = InterrogationInfo.validationDate;
// avec InterrogationInfo un nouveau type comprenant tous l'objet interrogation (mais sans data)
/************************************************************************************************/
// ou bien
type Props = {
source: LunaticSource,
interrogation: Interrogation,
}
const data = interrogation.data;
const surveyTitle = interrogation.collectionInstrumentId;
const usualSurveyUnitId = interrogation.usualSurveyUnitId;
const validationDate = interrogation.validationDate;A choisir entre les 2.
Mais, remarque plus générale, à ce niveau de couche logiciel, on préfère être indépendant des objet de type "requêtes" http, qui est une couche plus haut.
| const data = readAndValidateLunaticData(res, interrogationData); | ||
| logger.info(`generate PDF with params: ${JSON.stringify(requestBody.interrogation)}`) | ||
| const data = readAndValidateLunaticData(res, requestBody.interrogation.data); | ||
| await generatePdfStream(res, source, data); |
There was a problem hiding this comment.
Adapter la méthode generatePdfStream
generatePdfStream(res, source, data, interrogationInfos)
// ou
generatePdfStream(res, source, interrogation)Car data (renvoyé par const data = readAndValidateLunaticData(res, requestBody.interrogation.data); contient uniquement les données Lunatic (LunaticData)
2 participants
No description provided.