Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

resource_id and catalogue_item_id don't belong in submission.json #649

Open
csc-felipe opened this issue Nov 23, 2022 · 3 comments
Open
Labels
bug Something isn't working

Comments

@csc-felipe
Copy link
Contributor

Describe the bug

These lines don't belong in the submission schema. They should belong to a dataset object.

"resourceId": {
"type": "string",
"title": "REMS id of the resource created. Created automatically during the publication process"
},
"catalogueItemId": {
"type": "string",
"title": "REMS id of the catalogue created. Created automatically during the publication process"
}

They should be safe to remove, as no code adds those to a submission. It is instead added to datasets in the publication phase.

await obj_op.update_metadata_object(
ds["schema"],
ds["accession_id"],
{
"dac": {
"url": rems_url,
"workflowId": workflow_id,
"organizationId": org_id,
"resourceId": resource_id,
"catalogueId": catalogue_id,
},
},
)

Additional context

Thanks to @blankdots for figuring out the issue in the discussion in #648

@csc-felipe csc-felipe added the bug Something isn't working label Nov 23, 2022
@csc-jm
Copy link
Collaborator

csc-jm commented Nov 29, 2022

So should that dac (now renamed to rems) field and its properties be added to both EGA and BP json schemas?

@csc-felipe
Copy link
Contributor Author

I don't think those should be changed. Does @blankdots have an opinion?

One possible solution would be to put it in the submission's metadataObject field

@blankdots
Copy link
Contributor

One possible solution would be to put it in the submission's metadataObject field

I was thinking more in extraInfo > datasetIdentifiers as it is related to dataset the rems workflow, resource and catalogue item, however metadataObject works as well

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants