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

Updating metadata JSON via spreadsheet #349

Closed
amnonkhen opened this issue Jun 10, 2021 · 7 comments · Fixed by ebi-ait/ingest-ui#102
Closed

Updating metadata JSON via spreadsheet #349

amnonkhen opened this issue Jun 10, 2021 · 7 comments · Fixed by ebi-ait/ingest-ui#102
Assignees

Comments

@amnonkhen
Copy link
Contributor

No description provided.

@amnonkhen amnonkhen reopened this Jun 10, 2021
@amnonkhen
Copy link
Contributor Author

amnonkhen commented Jun 10, 2021

Acceptance Criteria

Scenario 1: Changing Accessions*

  • Given a spreadsheet
    • with uuids
    • and a list of files with no accessions
  • when the accessions is added to the spreadsheet
  • then the spreadsheet can be uploaded keeping uuids consistent

Scenario 2: ????

  • Given ...
  • when ...
  • then ...

@clairerye clairerye changed the title Simple Bulk Update Scenario - adding accessions to files Simple Bulk Update Scenario - usecase 2 adding accessions to files Jun 30, 2021
@aaclan-ebi
Copy link

aaclan-ebi commented Jul 2, 2021

Please note that I'm trying to work on implementation to support the general case.

  • Given a spreadsheet that accurately represents the project in ingest
    • with UUIDs
    • and updated values
  • when the spreadsheet is uploaded to Ingest
  • update the entities with the relevant changes in the metadata JSON

The use case for updating accessions should be supported with the general case implementation. Reusing the importer logic on how to construct the JSON should be able to support the general case for metadata JSON updates.

@aaclan-ebi aaclan-ebi changed the title Simple Bulk Update Scenario - usecase 2 adding accessions to files Updating metadata JSON via spreadsheet Jul 2, 2021
@aaclan-ebi
Copy link

aaclan-ebi commented Jul 2, 2021

previous title: Simple Bulk Update Scenario - usecase 2 adding accessions to files

I replaced the title with the more generic one.

@clairerye @amnonkhen, FYI.

@aaclan-ebi
Copy link

@ESapenaVentura
Copy link

I have found a small issue with this feature - Whenever you upload a spreadsheet to update the submission, if there is an error in the spreadsheet (e.g. missing required field official_hca_publication), it throws that error but doesn't update and doesn't say anything about any error updating

@ESapenaVentura
Copy link

ESapenaVentura commented Aug 19, 2021

I have tested the feature:

  • Addition of fields and modification of existing metadata works as intended *
    • *Aside from adding a list object, which adds it as a string instead of as an array
  • Deletion of metadata does not work, if I delete the field completely (3 types of fields) ingest does not detect it on upload

All details here: https://docs.google.com/spreadsheets/d/1hf6k7EUw5_l3mZEHVMi-iRZTRtCFyTDS1rOcIXNHBpo/edit#gid=0

@gabsie gabsie closed this as completed Nov 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants