search

CLOSER-Cohorts / archivist

Archivist is designed to be an easy to use editor that allows documention of questionnaires and data in DDI-Lifecycle
MIT License
18 stars 8 forks source link

Exporter not including all constructs #855

Closed HayleyMills closed 3 months ago

HayleyMills commented 5 months ago

xml export is missing question constructs e.g. Main alspac_covid_g0q4 - qc_covid_s5_6_a_2

Export attached as .txt

alspac_covid_g0q4.txt

This is for both exports.

simonreed commented 4 months ago

@HayleyMills do we know a specific construct/type of construct that's not being exported. Some constructs are but I'm not sure which ones aren't being exported.

HayleyMills commented 4 months ago

Question constructs e.g. alspac_covid_g0q4 qc_covid_s5_6_a_2

simonreed commented 4 months ago

@HayleyMills this has been deployed to staging.

HayleyMills commented 3 months ago

Thanks, unfortunately we cannot test this, as the same questionnaire is not on staging, and I don't know of an other example. Will need to test on main.

HayleyMills commented 3 months ago

Testing on live - exported and downloaded the instrument again (both version) and neither of them will load? https://closer-archivist.herokuapp.com/admin/imports/2789 https://closer-archivist.herokuapp.com/admin/imports/2790

Issue related to "No sequence found for instrument prefix" @simonreed

simonreed commented 3 months ago

@HayleyMills it's missing a sequence with the label of the instrument prefix, in the case of 2789 it's got alspac_covid_q4 but that sequence should have been called alspac_covid_g0q4_july.

simonreed commented 3 months ago

When I make that change it imports successfully https://closer-archivist.herokuapp.com/admin/imports/2799

HayleyMills commented 3 months ago

Tested on production, xml exports with all constructs and imports successfully.

simonreed commented 3 months ago

I've done the same for the xml which was failing in import 2790 but is successful here https://closer-archivist.herokuapp.com/admin/imports/2802