-
Notifications
You must be signed in to change notification settings - Fork 490
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
Custom Metadata: support boolean values #7961
Comments
Somewhat related to #5989 |
Hi @pdurbin |
@Asbjoedt no news. Let me mention @scolapasta and @cmbz to let them know you (and others) are interested. Is this something you (or others) might want to create pull request for? That often helps move things along. 😄 I agree it would be a nice feature. |
@Asbjoedt also, in the past I've encouraged installations to maintain a project/board of issues they care about. A great current example is https://github.com/orgs/IQSS/projects/15 maintained mostly by @DS-INRA. If you'd like a board like this, please open an issue at https://github.com/IQSS/dataverse-installations using the "request a project board" template. |
@jggautier Any connection here with the NIH GREI metadata recommendations? |
Hmm, yeah having the option to use a checkbox might help depositors provide information that those recommendations ask for. |
@pdurbin I can try to look at the source code. I know some Java and strictly speaking this is about implementing the Java boolean data type as a Dataverse data type for metadata blocks, if you agree on this approach? I've found the file "DatasetFieldType", may you provide me with a list of names of other files that are necessary to change to implement the boolean? I can't promise, I can create a PR for this, but I may take a look. |
@Asbjoedt great! For details on what files to edit (I'd need to look) maybe you can start a thread in #dev at https://chat.dataverse.org and we can figure it out together. |
@Asbjoedt Please comment here if you decide to make a PR. After discussion with @jggautier this issue appears to be aligned with our NIH GREI objectives. |
Here's the conversation so far: https://dataverse.zulipchat.com/#narrow/stream/379673-dev/topic/The.20boolean/near/405225599 To summarize, I agree with @poikilotherm that his commit at https://jugit.fz-juelich.de/fdm/dev/dataverse/-/commit/d4aece03aebcf9c2237aa7842c17c0d96af6a79a is a good one to study. |
It would be very useful to add a fieldType to support inclusion of true / false information. Some discussion with workarounds on the google group here.
The text was updated successfully, but these errors were encountered: