Chris Colvard comments

Results 53 comments of


                                            Chris Colvard

Add a configuration option for home page facets

I believe this could be handled with something like this: ``` config.add_facet_field 'format_facet', if: Proc.new {|context, _config, _opts| context.current_page?(main_app.root_path) } ```

Consider adding an aria-label to the "search" landmark

@barmintor Can this be closed by #2603?

In valkyrie/fedora ID are not in a pair tree

When I ran into this issue with Avalon I made an override to ActiveFedora to ensure the application was still creating and assigning ids instead of letting Fedora generate the...

Issue 557 - Assume client is requesting whole file if no range specified

Thanks for the clear writeup of the issue and this PR! It looks like the build is broken on an unrelated issue I think I saw on another samvera repo....

Run rubocop autocorrect for consistent styles

I think if we're going to consistently apply and require rubocop, I'd like to switch over to `bixby` for consistency with other samvera gems and for the insulation that it...

Update the supported rails and ruby versions

That sounds good to me. Do you mind making a ticket for any work that you're not planning on doing now?

Update dependencies

@gkellogg FWIW I updated the CircleCI build configuration in #133 (unmerged) and it is passing there. I wonder if the build will pass if you rebase on top of that...

Support Ruby 3.0.3

Draft PR: https://github.com/samvera/active_fedora/pull/1489 waiting on ActiveTriples support for ruby 3.

Test with newer versions of ruby and rails

Superseded by #1489

Third party auth scenarios that require non-cookie credentials

Avalon implements auth in our manifests using IIIF Auth v1 + probe service. Once a user has gone through the auth flow they then have a cookie which gets passed...