You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
It looks like it would almost work already, except that the current code is waiting for arbitrary JSON within a string, and we'd rather want the aforementioned specific JSON, as JSON.
This might be the only necessary change for the current code to accept that:
diff --git a/regovar/core/managers/pipeline_manager.py b/regovar/core/managers/pipeline_manager.py
index d22478d..0f55619 100644
--- a/regovar/core/managers/pipeline_manager.py+++ b/regovar/core/managers/pipeline_manager.py@@ -242,7 +242,7 @@ class PipelineManager:
log(data)
manifest = json.loads(data)
manifest = self.check_manifest(manifest)
- pipeline.developpers = manifest.pop("contacts")+ pipeline.developpers = str(manifest.pop("contacts"))
pipeline.manifest = manifest
# list documents available
… but I'm not even sure it's required, as the call to str() might be implicit when writing to the database.
If after verification, that change appears to not be required, then it'd probably be better to leave the code unchanged, to keep the structured (ie. JSON) variable instead of a simple string.
I don't know if we really need this information in the manifest., as this information is not mandatory for the usage of the pipeline, and can be just handle by the about.html document...
We could use a list of lists as:
The text was updated successfully, but these errors were encountered: