Skip to content

Conversation

@apriltuesday
Copy link
Contributor

Besides the diff, feel free to review the entire validation overview doc and suggest any additional changes.

@apriltuesday apriltuesday self-assigned this Nov 13, 2025
Copy link
Contributor

@Dona094 Dona094 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks really good!! I have a few suggestions to add to the document:

Metadata Spreadsheet section :

  1. We could mention the Helper tabs of the spreadsheet

  2. We could mention about the fields marked in green indicate an either/or requirement. It’s quite common for users to fill in both sections—for example, both the Novel Samples and Pre-existing Samples sections.

  3. Hold Date: We could mention the Hold Date field if users want to keep the data under embargo.

Under Common Errors Seen with Metadata Checks,

  1. Date fields of the metadata spreadsheet not following the YYYY-MM-DD format.

  2. CV Fields: It’s very common for users to modify these fields and enter their own data

Copy link
Contributor

@Dona094 Dona094 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great!! Many thanks

@apriltuesday apriltuesday merged commit bfa8f05 into EBIvariation:main Nov 20, 2025
1 check passed
@apriltuesday apriltuesday deleted the EVA-3995 branch November 20, 2025 12:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants