Versions Compared

Key

  • This line was added.
  • This line was removed.
  • Formatting was changed.

...

Agenda

(meeting notes below)

...

  • 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
    • 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
  • 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
        • This will be closed then


Call concluded at 12:31 EDT/09:31 PDT