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)
- Notes: https://confluence.cornell.edu/x/HuQIG
- 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
NoMethodError
- Hyrax::Monograph does work however
ContentEventJob
uses the Rails::Routes::Helper- ActiveModel#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::Helpers
- This within
ContentEventJob
is likely related to problems also
- This within
- 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
main
branch, a bug was discovered for creating Monographs- Lynette 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.indexer
should 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 diff
may 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
- Should this be similar to the
- This will be closed then
- Hyrax
Call concluded at 12:31 EDT/09:31 PDT