-
Notifications
You must be signed in to change notification settings - Fork 4
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
[DUOS-2806] Fix empty columns for new study datasets #2429
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice! 👍🏽
6b0743e
to
60d0aa9
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍🏽
60d0aa9
to
5b7d4c8
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
New tests look good 👍🏽
5b7d4c8
to
5fa4f66
Compare
Addresses
https://broadworkbench.atlassian.net/browse/DUOS-2806
Summary
Fix empty columns for new study datasets by trying all available depositor and custodian fields.
Have you read Terra's Contributing Guide lately? If not, do that first.