Skip to content
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

Enable Geonames functionality for Location field in Works #5800

Closed
jlhardes opened this issue Aug 16, 2022 · 4 comments · Fixed by #6159
Closed

Enable Geonames functionality for Location field in Works #5800

jlhardes opened this issue Aug 16, 2022 · 4 comments · Fixed by #6159
Assignees
Labels
metadata valkyrization Work impacts the Work part of PCDM Model

Comments

@jlhardes
Copy link
Contributor

Descriptive summary

Geonames is not connected to Location field in the Edit Work form yet so Location metadata cannot be saved.

Expected behavior

Location should function in work editing on valkyrized hyrax.

55257372-ec633f80-521c-11e9-83b9-4cd1da1b522d

Actual behavior

Location field is not offering results from Geonames in Edit Work form and no value can be saved.

Steps to reproduce the behavior

  1. As a logged in user, create a new Work
  2. On the Descriptions tab of the Edit Work form, click Additional Fields and scroll to the Location field.
  3. Try searching for a location and no results are offered and no value can be saved in form field.

Related work

PR #5157 - Update PcdmCollection metadata to match existing ::Collection metadata (discussing what is needed for collections metadata)

@jlhardes jlhardes added metadata valkyrization Work impacts the Work part of PCDM Model labels Aug 16, 2022
@jessicahilt
Copy link

We're going to check configuration.

@tpendragon
Copy link
Contributor

We think this works in Nurax, but not in Nurax PG

@jessicahilt jessicahilt assigned eliotjordan and unassigned dlpierce Aug 21, 2023
@eliotjordan
Copy link
Contributor

This worked for me after setting a valid geonames service username:

config.geonames_username = 'username'

https://github.com/samvera/hyrax/blob/main/.koppie/config/initializers/hyrax.rb#L107

Screenshot 2023-08-21 at 4 29 56 PM

@eliotjordan
Copy link
Contributor

Closed by #6159

Tested in nurax-pg.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
metadata valkyrization Work impacts the Work part of PCDM Model
Projects
Development

Successfully merging a pull request may close this issue.

5 participants