Samvera Tech Call 2024-02-07
Meeting Logistics:
Time: 9:00am PDT / Noon EDT
Zoom Meeting URL: https://princeton.zoom.us/j/7739591625
(link will launch Zoom client – if you do not have Zoom, expand the instructions below)
Agenda (meeting notes below)
Discussion around the double_combo branch (@Jeremy Friesen )
SoftServ is reviewing the configuration of model classes and how to incorporate that more into the code (see https://github.com/samvera/hyrax/pull/6653 )
Hyrax PRs to main: @Jeremy Friesen
Discuss Hyrax app generation and provided dev environment (@Daniel Pierce @Bradley Watson )
Hyrax 5.0.0.rc3? @Daniel Pierce
Moderator: @Jeremy Friesen
Notetaker: @Bradley Watson
Attendees:
@Jeremy Friesen
@Shana Moore
@LaRita Robinson
@Heather Greer Klein
@Chris Colvard
@Daniel Pierce
Kirk Wang
@Bradley Watson
@Rebekah Kati
Meeting Process
Notes
Hyrax 5.0.0.rc3:
Double combo is not blocking 5.0.0 and is looking like it can be a minor version release; many PRs have been submitted to main regarding bugs discovered during the Hyku upgrade to Hyrax 5 (by way of the Double Combo PR)
Release candidate is the goal.
Jeremy found some things in Hyrax that didn’t work with Valkyrie. PRs are upcoming.
Hyku work is producing more errors locally versus Circle CI; SoftServ is working through these as part of the upgrades.
Jeremy has been documenting the upgrade of Hyrax in Hyku, found here: https://github.com/samvera/hyku/wiki/Updating-Hyku-6-with-Hyrax-5-Developer-Notes#access-control-list-acl-considerations
App generation has been ongoing to provide new Hyrax users a proper valkyrie environment to work with by Daniel:
Now using Lando for providing backend services (see Lando https://github.com/lando/lando ).
Zeitwerk fixes are also ongoing.
Lando spins up a Postgres environment and should work with Fedora as well (not tested yet).
What context do others need to help understand the PR?
This work could wait for future releases or RCs.
One discovered bug is that the creation of an admin set is hard-coded to use the Valkyrie model, ignoring the configuration. You could still create by hand an
AdminSet
object but the service does not provide that native support. SoftServ’s exploring what steps we can take regarding that.Unable to create a non-
Hyrax::AdminSet
via the service class; configuration implies this should work but there’s hard-coded logic that’s proving hard to disentangle.
Next week’s Moderator: @Daniel Pierce
Next week’s Notetaker: @Jeremy Friesen