[backend] feat:enable AEM export from a list of Missions #402
Conversation
… mission and action exist
aleckvincent
added
enhancement
Coverage Report for frontend
File CoverageNo changed files found. |
There was a problem hiding this comment.
@aleckvincent - essaie de checker les 2 scenarios, tu couvres pas encore tout :
- un unique fichier agrégé
- multiple fichiers séparés (probablement bundled en .zip)
A mon retour, je vais bosser sur les missions inter-services et j'aurai les 2 cas de figure à gérer, cad agrégé et séparé.
Tu peux voir les designs sur la homepage: https://xd.adobe.com/view/eec9382f-b6b5-4644-b965-81f5d6ba8b07-bb9a/
| fun fromMissionList(missions: List<MissionEntity>): List<AEMTableExport> { | ||
| val tableExports = mutableListOf<AEMTableExport>() | ||
|
|
||
| for (mission in missions) { | ||
| if (mission.actions != null) { | ||
| val tableExport = fromMissionAction(mission.actions ?: listOf(), mission.endDateTimeUtc) | ||
| tableExport.sovereignProtect?.nbrOfRecognizedVessel = mission.generalInfo?.nbrOfRecognizedVessel?.toDouble() | ||
| tableExports.add(tableExport) | ||
| } | ||
| } | ||
|
|
||
| return tableExports | ||
| } |
There was a problem hiding this comment.
Ca c'est bien si on veut exporter plusieurs fichiers séparés, ce qu'il nous faudra donc il faut garder ce bout de code.
Pour exporter un fichier agrégé, il nous faudrait une étape en plus qui zip toute cette List et retourne just un AEMTableExport. Je sais plus le nom de la fonction qui permet de passer d'une liste à une valeur agrégée, c;est genre zip, fold ou similaire.
donc en gros, 2 fonctions:
fun fromMissionList(missions: List<MissionEntity>): List<AEMTableExport>pour les fichiers séparésfun aggregateMissionList(missions: List<MissionEntity>): AEMTableExport : return zip(fromMissionList)
| import java.nio.file.StandardCopyOption | ||
|
|
||
| @UseCase | ||
| class ExportMissionListAEM( |
There was a problem hiding this comment.
Je suis pas convaincu de l'utilité de cette classe en l'état
Dans le cas d'un fichier agrégé, si tu ajoutes la fonction qui agrège qui retourne un AEMTableExport (cf mon commentaire précédent), je pense que tu peux réutiliser ExportMissionAEM sans souci, non ?
Dans le cas de multiple fichiers séparés, je pense que cette classe doit être retravaillée. Il faudrait créer plusieurs fichiers et puis en faire un .zip parce que le navigateur ne pourra télécharger qu'un seul fichier si je me trompe pas.
|
déso, j'ai posté la review avec mon autre compte github mais c'est moi |
|
Je pense que tu peux simplifier encore ton code car il y a 3 usecases qui font presque la même chose.
Je pense qu'on peut entierement virer
Et derniere chose, je suis pas contre renommer |
|
No description provided.