Ti 135 doc highlighting

lucidworks / lucidworks-view

Create custom user experiences for your Fusion-powered apps.

https://lucidworks.com/view

Apache License 2.0

37 stars 23 forks source link

Ti 135 doc highlighting #25

Closed arielisaacs closed 8 years ago

arielisaacs commented 8 years ago

Fix highlighting function in document list
Add highlighting to slack and jira, as well as regular documents
Change default fields for Jira
Add styling for highlighting, currently background color highlight-color will be applied to results to be highlighted
NB highlighting for twitter does not currently work. Please refer to TI-142.

JoshEllinger commented 8 years ago

@arielisaacs Left a few comments, we should try and be consistent with one time bindings where we can ( i didn't comment on each one), and there were a few missing doc ids.

JoshEllinger commented 8 years ago

Doctypes:

[x] web datasource**
[x] jira datasource issue
[x] jira datasource project
[x] slack datasource
[ ] ~~twitter datasource~~
[x] local file datasource**
[x] generic datasource (not working) -- could just need steps to reproduce not sure what im looking for.

key \ = default display will not show anything in some cases due to how solr works

Note: to test jira projects you have to index projects, a setting on the datasource. Also it is recommended to use a facet on jira_content_type_s so you can focus on just projects.

i was unable to get web datasource to work with highlighting in view

arielisaacs commented 8 years ago

Web is working, I'm not sure how to replicate what you are seeing. A lot of times it seems as though the custom fields that the highlight is showing up on is not one of the fields in view. Since fields are customizable, I don't feel comfortable adding every possible field that could have highlighting. Especially in looking at CSV and other datasources without standardized fields, this becomes an issue. Furthermore, since we are not using the prefs API, there isn't a way to change what fields are showing up. Body link anchor would not show up by design in the response in view. Hence your issue.

arielisaacs commented 8 years ago

All datasources save for twitter are tested and working. Closing this PR