search

scientist-softserv / adventist_knapsack

Apache License 2.0
1 stars 0 forks source link

:bug: Location dropdown doesn't load #212

Open ShanaLMoore opened 7 months ago

ShanaLMoore commented 7 months ago

Story

This was spotted when we were QAing. The location facet wasn't seen because none of the current data has a location value. When we tried to add it we were unable to:

Spotted on knapsack staging

Acceptance Criteria

testing instructions

Create a new Dataset work type Navigate to the location field and type at least two characters to activate the options.

Screenshots / Video

image

Notes

This works in adventist proper/production when creating a work:

image
ShanaLMoore commented 7 months ago

I believe part of the issue is config/initializer/hyrax.rb needs a geoname_username defined.

ShanaLMoore commented 4 months ago

Even adding the same geonames_username (jcoyne) doesn't seem to resolve this. For some reason the access is denied. yet the same path works in production: https://adl.b2.adventistdigitallibrary.org/

Knapsack staging

https://adl.adventist-knapsack-staging.notch8.cloud/authorities/search/geonames?q=sandiego&_=1707785405764

Image

Production

https://adl.b2.adventistdigitallibrary.org/authorities/search/geonames?q=sandiego&_=1707785405764

Image

ShanaLMoore commented 4 months ago

Each tenant must set their own geoname in their account. Setting and env variable, HYKU_GEONAMES_USERNAME, doesn't matter. ~I will revert all the code I've submitted and add help text to guide the user.~

I am setting HYKU_GEONAMES_USERNAME as the default value to honor HYKU's readme

ShanaLMoore commented 4 months ago

deployed and confirmed that staging it working:

Image

ShanaLMoore commented 4 months ago

Image

Image

QA PASS ✅