Trey Pendragon

Results 122 comments of Trey Pendragon

@mathewjordan Yay! I think our use case is closer to https://iiif.io/api/cookbook/recipe/0485-contentstate-canvas-region/. Our manifests already have a `start`, so this would be like...ignore what `start` says, open it here.

@mathewjordan Yeah, that's kinda where we're at too. If you'd prefer we could probably instantiate it with a prop and control the URL ourselves, but in general we just think...

@mathewjordan I owed you some work: Manifest: https://figgy.princeton.edu/concern/ephemera_folders/b51ee428-e508-4e13-97e4-2acbf716d756/manifest Content state: ```json { "id": "https://figgy.princeton.edu/concern/ephemera_folders/b51ee428-e508-4e13-97e4-2acbf716d756/manifest/canvas/9ec7f5da-80af-43b0-a0a5-b3d2fec4e3da", "type": "Canvas", "partOf": [ { "id": "https://figgy.princeton.edu/concern/ephemera_folders/b51ee428-e508-4e13-97e4-2acbf716d756/manifest", "type": "Manifest" } ] } ``` Encoded: `JTdCJTIyaWQlMjIlM0ElMjJodHRwcyUzQSUyRiUyRmZpZ2d5LnByaW5jZXRvbi5lZHUlMkZjb25jZXJuJTJGZXBoZW1lcmFfZm9sZGVycyUyRmI1MWVlNDI4LWU1MDgtNGUxMy05N2U0LTJhY2JmNzE2ZDc1NiUyRm1hbmlmZXN0JTJGY2FudmFzJTJGOWVjN2Y1ZGEtODBhZi00M2IwLWEwYTUtYjNkMmZlYzRlM2RhJTIyJTJDJTIydHlwZSUyMiUzQSUyMkNhbnZhcyUyMiUyQyUyMnBhcnRPZiUyMiUzQSU1QiU3QiUyMmlkJTIyJTNBJTIyaHR0cHMlM0ElMkYlMkZmaWdneS5wcmluY2V0b24uZWR1JTJGY29uY2VybiUyRmVwaGVtZXJhX2ZvbGRlcnMlMkZiNTFlZTQyOC1lNTA4LTRlMTMtOTdlNC0yYWNiZjcxNmQ3NTYlMkZtYW5pZmVzdCUyMiUyQyUyMnR5cGUlMjIlM0ElMjJNYW5pZmVzdCUyMiU3RCU1RCU3RA` Example:...

We've implemented our way out of this problem.

I hope CI can run these tests, because I couldn't get molecule running locally. Something about it wanting a sudo password.

@beechesII Sorry to ping, but going by recent merges I think you're active on this project - is there something I can do to get this in?

I'm not sure if I can answer that. Is Fedora 5 unsupported? Does nobody use it in Samvera-land?

If nobody's using it and it's unsupported, I'm okay removing it. It probably means a major version to do that though

@dchandekstark Yeah let's just drop it, I'm tired of CI failing.

@dmolesUC I'd merge a PR that uses the deprecation gem. Great suggestion!