Add regional facets to Search Page

torrin47 commented 7 years ago

This comes out of the work on issue #53, which is complete and soon in production. The goal is to have check boxes (facets) allowing quick filtering of results to just EPA regions. If we get this working with a nice UI, it's quite likely Ana will want to add other facets to the search page in the future.

[x] Instead of entering owner=Region 0X into the search text, it would be more intuitive to leave the search box empty, and add checkboxes to the page that would filter the search results based on the regional owners. (In effect, this is creating the first "faceted search" functionality on the search page).
[ ] There would also be an "All Regions" checkbox that would be checked by default (all of the individual regions would be unchecked by default). If a user checks a box by a region, the "all regions" checkbox would toggle off. (We decided this has no value to end users.)
[x] A user may check multiple regions and the results would be additive.
[x] The search results would not be updated until a user clicks the "Search" button.
[x] If a user clicks the "See More" button for one of the regional tabs on the home page, the search page would appear with that region's box checked and the search results showing.

torrin47 commented 7 years ago

Here's the text to place above the regional checkboxes:

"Use the checkboxes below to limit the search results to metadata records contributed by specific EPA Regional offices."

torrin47 commented 7 years ago

Looks good in staging, one thing I'm noticing, though, when clicking "See More" for a region's collection on the home page, although the results on the search page are filtered for that region, the corresponding checkbox isn't checked. Is that something we can change?

Saisuma004 commented 7 years ago

Oh I have fixed that , it should be checked when we click on regions. Let me check if I missed anything to checkin.

On Tue, Mar 7, 2017 at 11:20 AM, Torrin Hultgren notifications@github.com wrote:

Looks good in staging, one thing I'm noticing, though, when clicking "See More" for a region's collection on the home page, although the results on the search page are filtered for that region, the corresponding checkbox isn't checked. Is that something we can change?

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/Innovate-Inc/EDG_metadata/issues/60#issuecomment-284827170, or mute the thread https://github.com/notifications/unsubscribe-auth/AQVVNiz4G1661sC__qmnUJZ8604kmv0Jks5rja4KgaJpZM4Lu5cY .

-- Sumalatha Malothu Innovate!, Inc. Cell: (318) 278-4228 http://www.innovateteam.com

Saisuma004 commented 7 years ago

I have missed adding code back to homeBody.jsp while resolving conflicts. Checking in the code shortly.

On Tue, Mar 7, 2017 at 12:08 PM, Suma Malothu smalothu@innovateteam.com wrote:

Oh I have fixed that , it should be checked when we click on regions. Let me check if I missed anything to checkin.

On Tue, Mar 7, 2017 at 11:20 AM, Torrin Hultgren <notifications@github.com

wrote:

Looks good in staging, one thing I'm noticing, though, when clicking "See More" for a region's collection on the home page, although the results on the search page are filtered for that region, the corresponding checkbox isn't checked. Is that something we can change?

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/Innovate-Inc/EDG_metadata/issues/60#issuecomment-284827170, or mute the thread https://github.com/notifications/unsubscribe-auth/AQVVNiz4G1661sC__qmnUJZ8604kmv0Jks5rja4KgaJpZM4Lu5cY .

-- Sumalatha Malothu Innovate!, Inc. Cell: (318) 278-4228 http://www.innovateteam.com

-- Sumalatha Malothu Innovate!, Inc. Cell: (318) 278-4228 http://www.innovateteam.com

torrin47 commented 7 years ago

Excellent, that did it, looks great!

On Tue, Mar 7, 2017 at 12:37 PM, Saisuma004 notifications@github.com wrote:

I have missed adding code back to homeBody.jsp while resolving conflicts. Checking in the code shortly.

On Tue, Mar 7, 2017 at 12:08 PM, Suma Malothu smalothu@innovateteam.com wrote:

Oh I have fixed that , it should be checked when we click on regions. Let me check if I missed anything to checkin.

On Tue, Mar 7, 2017 at 11:20 AM, Torrin Hultgren < notifications@github.com

wrote:

Looks good in staging, one thing I'm noticing, though, when clicking "See More" for a region's collection on the home page, although the results on the search page are filtered for that region, the corresponding checkbox isn't checked. Is that something we can change?

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/Innovate-Inc/EDG_metadata/issues/60# issuecomment-284827170, or mute the thread https://github.com/notifications/unsubscribe-auth/AQVVNiz4G1661sC__ qmnUJZ8604kmv0Jks5rja4KgaJpZM4Lu5cY .

-- Sumalatha Malothu Innovate!, Inc. Cell: (318) 278-4228 http://www.innovateteam.com

-- Sumalatha Malothu Innovate!, Inc. Cell: (318) 278-4228 http://www.innovateteam.com

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/Innovate-Inc/EDG_metadata/issues/60#issuecomment-284851030, or mute the thread https://github.com/notifications/unsubscribe-auth/ACB0vD5Yn5uze1wpdC9ojxHAQfdc-LsQks5rjcAUgaJpZM4Lu5cY .

-- Torrin Hultgren Senior Manager, Geospatial Solutions Innovate!, Inc. O: (703) 922-9090 x737

http://www.innovateteam.com

Innovate!, Inc. is a Virginia-based 8(a), Hispanic- and woman-owned business delivering Geospatial, Information Technology, Security, Business Consulting, Project Management and Executive Coaching solutions to our government and private sector clients. Check us out at www.innovateteam.com.

torrin47 commented 7 years ago

Hrmmm. I can't seem to get a normal text search working in conjunction with the checkboxes. It seems to be an either/or choice - either text search or regional checkboxes, but not both. Any way to allow both to work at the same time?

Saisuma004 commented 7 years ago

That scenario working on local, wonder why its not working on staging. Comparing code with local.

On Wed, Mar 8, 2017 at 11:41 AM, Torrin Hultgren notifications@github.com wrote:

Hrmmm. I can't seem to get a normal text search working in conjunction with the checkboxes. It seems to be an either/or choice - either text search or regional checkboxes, but not both. Any way to allow both to work at the same time?

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/Innovate-Inc/EDG_metadata/issues/60#issuecomment-285146522, or mute the thread https://github.com/notifications/unsubscribe-auth/AQVVNsb-k-GvnDaU-Y-jWcFElmtSC1dXks5rjwSFgaJpZM4Lu5cY .

-- Sumalatha Malothu Innovate!, Inc. Cell: (318) 278-4228 http://www.innovateteam.com

Saisuma004 commented 7 years ago

Checked in missing js file.

torrin47 commented 7 years ago

So facet criteria have to be form elements, and match REST API syntax (and be added to the REST URL example snippet in the form POST). Multiple criteria can be comma separated: https://edg.epa.gov/metadata/rest/find/document?f=json&owner=Region%203,Region%202

Example: frmSearchCriteria:scSelOwnerHidden:Region%203,Region%202 frmSearchCriteria:scSearchUrl:/metadata/rest/find/document?rid=local&ridName=This%20Site%20(the%20EDG)&rids=local&start=1&max=25&orderBy=relevance&owner=Region%203,Region%202&maxSearchTimeMilliSec=10000&f=searchpageresults

Innovate-Inc / EDG_metadata

Add regional facets to Search Page #60