Do we want to let Samples inherit Source material info to comply with the ENA minimal checklist? (collection date and geografical location)

[bedroesb]

Collection data and Geographical location is currently added to the source material, and not to sample, but ENA will look into the registered Sample in BioSamples. This throws an error since these are mandatory fields for ENA. Currently the MARS ENA repository API fills in default values (Not provided) but we might want to think about inheriting these values from Source material instead
Original post:

The ISA-JSON we are using actually contains those fields. I think we might want to revert this for the source sample.

collection date:

MARS/test-data/biosamples-input-isa.json

Lines 276 to 284 in ad636ca

	"category": {
	"@id": "#characteristic_category/collection_date_323"
	},
	"value": {
	"annotationValue": "01/01/2022",
	"termSource": "",
	"termAccession": ""
	}
	},

Geographic location:

MARS/test-data/biosamples-input-isa.json

Lines 306 to 314 in ad636ca

	"category": {
	"@id": "#characteristic_category/geographic_location_(country_and/or_sea)_326"
	},
	"value": {
	"annotationValue": "Belgium",
	"termSource": "",
	"termAccession": ""
	}
	},

Originally posted by @kdp-cloud in #69 (comment)