Samvera Tech Call 2021-08-25
Meeting Logistics:
Time: 9:00am PDT / Noon EDT
ZOOM connection: https://notredame.zoom.us/j/94030214208
(link will launch Zoom client – if you do not have Zoom, expand the instructions below)
Agenda
(meeting notes below)
Hyrax routing questions (@Lynette Rayle)
Moderator: @Juliet Hardesty
Notetaker: @James Griffin
Attendees:
@Lynette Rayle (Cornell)
@tamsin johnson (UC Santa Barbara)
Kris Dekeyser (KU Leuven)
Meeting Process
Notes
Hyrax Routing (Lynette)
If you have an instance of the `Hyrax::AdministrativeSet`, `to_model_name` uses `singular_route_key`
`admin_set_path` is generated from this
`polymorphic_path` is called...uses this `singular_route_key` method
`ContentEventJob` raises a
NoMethodErrorHyrax::Monograph does work however
ContentEventJobuses the Rails::Routes::HelperActiveModel#name somehow keeps the proper path from being generated
What does `ActiveFedora::AdminSet` normally do?
The ActiveModel#name is most likely not being overridden properly
include ActionView::HelpersThis within
ContentEventJobis likely related to problems also
Larger problem: `link_to(some_admin_set)` is going to break
Hyrax::ResourceName is specific to the Valkyrie::Resource Models
This is necessary for generating the `ActiveModel#name` which is valid
This might be explored also
Exploring the User Interface on the
mainbranch, a bug was discovered for creating MonographsLynette created issue #5084 in response to this
Next Scheduled Tech. Call
Moderator: Tamsin
Notetaker: James
Samvera Help Follow-Up
Dan Kerchner (https://samvera.slack.com/archives/C0F9JQJDQ/p1629903187107700)
Tamsin: This is configured on the model directly, so there can be a variable indexer
self.indexershould then return the customIndexPresenter(This is from Blacklight upstream)
Pull Request Review
Hyrax
Many are still just drafts
5082 https://github.com/samvera/hyrax/pull/5082 should be merged to assist with reviewing and merging other pull requests
This is a parameterization of a previously-coded collaborator
The
git diffmay be difficult to visually parse, but hopefully there won't be much overlap
5077 https://github.com/samvera/hyrax/pull/5077
This should be closed, and contributors should plan to use a generic `SystemUser` for this case
Both of the EventJobs and notifications require some `User` to be attached
Should this be similar to the
AuditUser?Yes, this should work
This will be closed then
Call concluded at 12:31 EDT/09:31 PDT