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

Enable facet UI for Time Period Covered #5150

Closed
philippconzett opened this issue Oct 6, 2018 · 10 comments
Closed

Enable facet UI for Time Period Covered #5150

philippconzett opened this issue Oct 6, 2018 · 10 comments

Comments

@philippconzett
Copy link
Contributor

This issue may be related to #70.

Dataverse offers faceting for several date fields. However, faceting does not work for
Time Period Covered: timePeriodCoveredStart + timePeriodCoveredEnd (Citation Metadata)

It would be nice to make this field facetable just like this field in the Astronomy Metadata:
coverage.Temporal: coverage.Temporal.StartTime + coverage.Temporal.StopTime

Thanks!
Philipp

@pdurbin
Copy link
Member

pdurbin commented Oct 9, 2018

@philippconzett hi, I just wanted to confirm that you aren't being affected by the bug in #4906 which was fixed in Dataverse 4.9.3.

@philippconzett
Copy link
Contributor Author

@pdurbin hi, I we have just upgraded to 4.9.4, and I have tested this again, but there is no facet Time Period Covered popping up.

@pdurbin
Copy link
Member

pdurbin commented Oct 9, 2018

Thanks. I just took a quick look at scripts/api/data/metadatablocks/citation.tsv and "facetable" is already set to "true" so I'm not quite sure what's going on. Here's a screenshot:

screen shot 2018-10-09 at 9 32 47 am

This would need some more investigation.

@philippconzett
Copy link
Contributor Author

@pdurbin, hmm, I'm not sure, maybe enabling faceting of the field in row 62 helps?

@pdurbin
Copy link
Member

pdurbin commented Oct 9, 2018

@philippconzett I'm not sure if it would help or not. You are welcome to try it. 😄

@philippconzett
Copy link
Contributor Author

@pdurbin I'll ask our system admin. In the meantime, could you please send me a screenshot of the configuration of the three fields for coverage.Temporal?

@pdurbin
Copy link
Member

pdurbin commented Oct 9, 2018

@philippconzett sure, here you go:

screen shot 2018-10-09 at 10 14 19 am

You can find these TSV files at https://github.com/IQSS/dataverse/tree/v4.9.4/scripts/api/data/metadatablocks and we recently documented how they work (#3168): http://guides.dataverse.org/en/4.9.4/admin/metadatacustomization.html

@philippconzett
Copy link
Contributor Author

@pdurbin Thanks! I just tested this again, but this time in our production repo, and it works:
image

I just see here the chronological sort issue popping up again. I commented this in #4977

@pdurbin
Copy link
Member

pdurbin commented Oct 9, 2018

@philippconzett great! So it's working? Do you want to close this issue?

@philippconzett
Copy link
Contributor Author

Yes, please. Thanks!

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