Guillermo Gutiérrez

Is there any other thing in the repo that should be reorganized?

Sure!

I think INFO for production is OK. Maybe we should review what gets logged and either remove some of these messages or downgrade them to DEBUG.

Could this be with non-finalized submissions that lack of an instanceID? That would explain the dupes...

So, after analyzing this issue, we have identified a particular scenario that we should solve separately: submissions without an instance ID. These submissions have always been excluded from the export...

The hardest problem to solve is when forms have repeat groups. Null IDs in this scenario wouldn't work because it would be impossible to link rows from a repeat group...

After discussing this with @yanokwa, and @lognaturel, we decided to redefine the validity rules of a submission: > This means that for a submission with repeat groups to be valid,...

I think I would have to check that with Aggregate's source code in front of me... I think it's an important thing we're doing here by documenting all these edge-case...

Hi @icemc! If you’re going to work on this, I’d want to ask you to create one PR per each task on the checkbox list, ok?

I think it's a mistake, sorry... Don't bother with the `PEMParser` class What we really need is to make sure that when the PEM file arrives at `ExportAction.readPemFile(Path pemFile)` it...