search

bcgov / ckan-ui

CKAN UI - VueJS
GNU Affero General Public License v3.0
14 stars 4 forks source link

Review the search terms included on Landing Page (DDS-422) #273

Open SteveChapmanBCDX opened 3 years ago

SteveChapmanBCDX commented 3 years ago

The Landing Page includes the following search terms:

These have always been placeholders. Let's see if we can come up with 4-5 terms that would be more topical.

Review with @yuisotozaki.

SteveChapmanBCDX commented 3 years ago

Reviewed with @yuisotozaki. This could also be used to direct users to specific data records. Stick with most popular search terms.

ghost commented 3 years ago

@SteveChapmanBCDX , what is to be tested here? Is there a place in the new catalogue where these terms are editable? is it in code, or in an interface available to a specific profile (sys admin vs admin). Not sure what to do here?

I did have a look at https://github.com/bcgov/ckan-ui/issues/352 but it appears to be closed. Any insight would be great. thanks.

SteveChapmanBCDX commented 3 years ago

Brandon added new functionality that will eventually allow us to use a plug-in to dynamically populate the search terms on the landing page using analytics (e.g., most popular searches).

In the meantime, Brandon refactored the code to allow us the add the desired search terms in a configuration file. The search terms for the Landing Page can now be found here => backend/config/default.json

@sheilamv - We only needed to test that the existing search terms worked the same way they did before Brandon made this change. I've tested them in Toyger and they work correctly.

SteveChapmanBCDX commented 3 years ago

The purpose of this ticket was for the team to review the existing search terms and replace them if required. That has not been done yet, so returning this ticket to the Backlog.

ghost commented 2 years ago

https://dpdd.atlassian.net/browse/DDS-422