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
5584 Filtering of Files on Dataset Page, Search using Solr #5820
Conversation
(facets are not working yet) (#5584)
- added an indexed flag, for the published files removed from the current draft; - backward compatibility, if talking to a solr server with an older schema; - added check for solr being down - reverting back to searching in the db if it is. (#5584)
…le that's been removed from the current draft. (#5584)
…t to bundle [ref #5584]
…adata on dataset pg [ref #5584]
Per discussion with @mheppler, looking at the rendering rules in the fragment - specifically things like where it says "no files in the dataset" where it should really be saying "your search found nothing". |
…n the dataset" and "search found no files" situation, and paging. (#5584)
Updated the "no results found" msg for the files table on the dataset pg and moved it to the bundle. Render logic added by @landreev looks good. |
// This 2nd search() is for populating the facets: -- L.A. | ||
// TODO: ... | ||
// This 2nd search() is for populating the "type" ("dataverse", "dataset", "file") facets: -- L.A. | ||
// (why exactly do we need it, again?) |
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.
I believe it's to get the counts (since it shows those counts even if that facet is not checked, ie those types are not returned by the search).
New Contributors
Welcome! New contributors should at least glance at CONTRIBUTING.md, especially the section on pull requests where we encourage you to reach out to other developers before you start coding. Also, please note that we measure code coverage and prefer you write unit tests. Pull requests can still be reviewed without tests or completion of the checklist outlined below. Thanks!
Related Issues
Pull Request Checklist