Trey Pendragon comments

Results 122 comments of


                                            Trey Pendragon

`Reform::Rails` doesn't work with Rails 6.1 well

Y'all are the best, thank you!

Ensure that OS package conflicts can be handled properly

Oh yeah I had the same problem with Rocky, and just added a curl-minimal variable in https://github.com/ansible-community/ansible-nomad/pull/197

Allow adding GenericFile to GenericWork in memory (without saving either object)

I think it's more than just tests - you should be able to build things up in memory before persisting, and right now you can't. Seems related to https://github.com/projecthydra/active_fedora/pull/858

Deleting Works not removing list_source and indirect_containers from Solr

So the problem is that the solr document is left behind, yes? Because that indirect container should be very deleted from Fedora.

Deleting Works not removing list_source and indirect_containers from Solr

This is a much more generic problem than Hydra:PCDM. Effectively this is a failure in the sync of Fedora -> Solr. When I delete a root node in Fedora, it...

Deleting Works not removing list_source and indirect_containers from Solr

The list source being in solr is used for a query, I think, but maybe it doesn't have to be?

Roles may be returning registered for guest users

This still a problem @jcoyne?

Valkyrie::StorageAdapter::File - insert a higher abstraction to support cloud providers?

@dchandekstark Do you have a reference library that handles file abstraction the way you're proposing? We've leaned on IO because it's how folks expect to handle files, and on `#disk_path`...

Valkyrie::StorageAdapter::File - insert a higher abstraction to support cloud providers?

@dchandekstark Have you looked at `valkyrie-shrine` for S3 support? That's largely what we've done.

Valkyrie::StorageAdapter::File - insert a higher abstraction to support cloud providers?

@dchandekstark It's pretty simple. In figgy we have this: https://github.com/pulibrary/figgy/blob/c575619bfeb42a35ed0545644b21034214f62a75/config/initializers/valkyrie.rb#L129-L145 then that adapter works just like the disk one.