search

samvera-labs / nurax-pg

This is a Hyrax application configured to use Postgres for metadata storage instead of ActiveFedora.
Apache License 2.0
0 stars 0 forks source link

Editing a collection raises `undefined thumbnail_id` as the edit form tries to load #27

Closed elrayle closed 2 years ago

elrayle commented 3 years ago

Description

When using the postgres adapter in nurax-pg, editing a collection raises undefined thumbnail_id as the edit form tries to load.

Expected

Dashboard -> Collections -> click Action menu beside existing collection -> Edit collection -> opens the edit form populated with the collection's values.

Actual

Dashboard -> Collections -> click Action menu beside existing collection -> Edit collection -> raises an exception

image

Steps to reproduce the behavior

Prereq: Create a User Collection on nurax-pg.

  1. Login to nurax-pg
  2. Dashboard -> Collections
  3. Click the action menu beside an existing collection
  4. Select Edit collection

Related work

Issue https://github.com/samvera/hyrax/issues/5133 - Val MVP: Edit PcdmCollection as a valkyrie resource through UI

Issue https://github.com/samvera/hyrax/issues/5286 - specific to this undefined thumbnail_id exception

elrayle commented 2 years ago

PR https://github.com/samvera/hyrax/pull/5287 adds a check to skip the code that allows a user to select a thumbnail for the collection. When this issue is resolved, the skip should be removed.

jlhardes commented 2 years ago

Thumbnails for collections is not available in nurax-pg and is also not available in locally built Valkyrized Hyrax (koppie). There is a hyrax issue (#5286) for the fact that thumbnail selection is not working for collections in Valkyrized Hyrax. There is also an issue to redo the collection thumbnail selection and move the current selection workflow behind a feature flipper (https://github.com/samvera/hyrax/issues/5764). Closing this issue as a won't fix for nurax-pg and approaching collection thumbnails for Valkyrized Hyrax through the redo issue instead.