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

GRDI Template - isolated_by range is WhiteSpaceMinimizedString, but should be a menu. #15

Open
ejurga opened this issue Feb 29, 2024 · 2 comments

Comments

@ejurga
Copy link

ejurga commented Feb 29, 2024

The isolated_by field currently takes a string. However, the same field in the excel template over at the GRDI_AMR_One_Health repo is limited to its own isolated_by menu. This presumably should be incorporated here as well.

There appears to be an isolated_by menu in the corresponding YAML. I might suggest that, instead of having its own menu, this field take the same menu as sample_collected_by.

@cbarcl01
Copy link

@ejurga - this should be resolved with the most recent PGP. I'm going to add this over at GRDI/issues for easier tracking for release notes as well. Thanks for bringing up!

@cbarcl01
Copy link

@ddooley this has been resolved in the most recent PGP release and can be closed.

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

No branches or pull requests

2 participants