October 26, 2022

Time/Place

This meeting is held virtually via Zoom, with an open channel for chatter on Slack. Anyone is welcome to join. The hosting can be claimed by anybody with the host ID. The Host ID is in the header of the TAG slack channel.

Time: Wednesday, 1:00pm Eastern Time US (UTC-5)
Zoom link: https://zoom.us/j/97345581392
Passcode: 919241
Join us on the Slack Islandora Channel https://islandora.slack.com/archives/CM5PPAV28

Chair Roster

These are the core members of TAG and will take on chair of the meeting if there are no volunteers, and will support volunteers who are chairing for the first time:

Don 🪑
Seth
Rosie
Willow

Attendees

Don Richards (Chair)
Seth Shaw
Alan Stanley
Alexander O'Neill
Willow Gillingham
Rebel Cummings-Sauls
Annie Oelschlager
Rosie Le Faive
Yamil Suarez
Isabella Nikolaidis (notes)

Agenda

Note that links are to the document repository only. New issues in other repositories or organizations will not appear in this list, and should be added by interested parties directly to the agenda.

PR Roundup- Link sorted by recently updated
Issue Roundup - Link sorted by recently updated
Issue Catch-Up from last week by repo:
Review of mailing list
Feel free to add more, or add something from Islandora Discussions

Minutes

PR Roundup

https://github.com/Islandora/islandora-starter-site/pull/12

Seth: Jared mentioned that composer.json had an Islandora FITS patch - but I don't see an update to the islandora composer reference to say that.
- Rolling in a dev update that was merged - did that patch get rolled in?
Don: Adam's question?
Rosie: Yes, those two (linked previously in the README) have been merged. Fits has no releases. if it did, i'd make the composer requirement for ^(release).
Rosie: Ready to test and merge
Alexander: The stuff generated by the starter site isn't really human code.
- Don't know if code reviews is a coherent thing to make someone do - Drupal writes out configs in different order.
- If you know what change you're making you can verify the change.
- Testing - here's the change, go make sure the change expected happens. Also some automatic CI thing would be a good idea.
Seth - further testing instructions?
- In playbook - https://github.com/Islandora-Devops/islandora-playbook/blob/dev/vars/starter_dev.yml
- In ISLE - https://github.com/Islandora-Devops/isle-dc/blob/development/Makefile#L543

https://github.com/Islandora/Crayfish/pull/170

Seth: Want to make sure that updates are done before testing.

Issue Roundup

https://github.com/Islandora/islandora_mirador/issues/10

Tested by Alan - this ticket can be closed.

https://github.com/Islandora/islandora/issues/902

Is features being phased in or phased out by Drupal?
Alexander: It's not part of core, so it's not being phased in or out.
- Seems to be a thing that people on the Vanguard group are updating lately.
Seth: When Danny introduced it, seemed like a good idea, but with practical use it didn't work as well as we wanted it to.
- Especially with movement to the starter site, where we're bringing in a bunch of config in the first place using Drupal's built in config setup.
Alexander: There's not much to do to remove - delete the features yaml file.

https://github.com/Islandora-Devops/isle-buildkit/issues/245

Seth: Referenced in the solr docker file using ARG instead of ENV.
Don: Related to this PR: https://github.com/Islandora-Devops/isle-buildkit/pull/244
If you have thoughts about using ARGs in the docker file, please comment.

https://github.com/Islandora/documentation/issues/2185

Display large tiled images
Don followed up with Cantaloupe API endpoints and caching links

https://github.com/Islandora-Devops/isle-buildkit/issues/225

Seth: Another ISLE buildkit release conversation

https://github.com/Islandora-Devops/isle-buildkit/issues/254

Same as above.

https://github.com/Islandora-Devops/isle-dc/issues/299

Same as above.

https://github.com/Islandora-Devops/isle-dc/issues/296

There are a number of services in ISLE where they're not super obvious how to get to them. Some aren't exposed by default.
Rosie: Outdated table in docs.
Don updated.

https://github.com/Islandora-Devops/isle-buildkit/issues/248

Alexander: Likely a wider bug than the one case it's found in.
Rosie: is it just ISLE, or is it any time you make a circular reference?
Don: Think every time you make a circular reference, not sure.

https://github.com/Islandora-Devops/isle-buildkit/issues/249

Crayfish rebuilt as Symfony, new development going on for 3x branch.

https://github.com/Islandora-Devops/isle-buildkit/issues/250

Seth: Another to do for the 2.0 release.

https://github.com/Islandora/controlled_access_terms/issues/86

Seth: This may have come up in the Metadata IG? Like Rosie says, this is just how Drupal works.
Ex. Updating the formatter to, if the entity is missing, hide the reference until drupal cleans itself up - a UI improvement.
Rosie: The functionality of "cleanly deleting" an entity and its reference it would be very desirable for metadata!
- Not just a UI consideration, as right now if you delete a taxonmy term (because you don't want it as a tag) then all your nodes/media will still point to the term as they did before, and all their jsonld metadata will expose that old data.
Marking as enhancement instead of bug.
Don: Should we fix at the point where it goes to look up the resources, or is that just filtering at the end rather than addressing the bigger issue?

https://github.com/Islandora/documentation/issues/2162

This issue will close when we merge the open PR.

https://github.com/Islandora/documentation/issues/2181

Configuration options for flysystem vs fcrepo urls.
Seth: Based on slack discussion - really talking about Crayfish, allow it to pull directly from fedora rather than flysystem.
- It's bypassing flysystem entirely when in fedora.
- If you add an option to the actions to admit the event, part of the event that it sends is the flysystem URI to the file so Crayfish knows where to get the file to do the derivative.
- Instead of sending flysystem URI, take the configured fedora destination for flysystem and swap that part of the flysystem URI so it's the actual fedora URL and Crayfish will get the fedora URL to pull that file.
- If you have a system where your Crayfish is just as close, you don't have to put a request to Drupal to pull the file from Fedora, because then Fedora has to send to Drupal, Drupal/Apache? has to cache in temporary file system, and then send to Crayfish to stream through. This saves a lot of networking and temporary disc caching.
Don: Should this be a github discussion?
Alexander: It's a real performance issue, and architecturally kinda silly.
- The discussion is, should it be the default? Adding the option should happen.
Rosie: Is there any reason to want to use the Drupal URL?
Alex: Access control from the outside world. But that doesn't affect your derivative generation.
Seth: System flexibility? Where you may not necessarily want your Crayfish to know where your Fedora is.
Rosie: Would both options work if your file is not in fedora?
Seth: A lot of these sites no longer use fedora. If they use Amazon S3, there is no flysystem url to use.
- The option would work if your file was not in Fedora. The option needs to make sure that if it's not Drupal. If it's using flysystem, to fix that, take whatever flysystem configured and redirect. It isn't just Fedora. Option needs to not just pull the fedora url, but also the flysystem configured destination is (could be fedora, might be s3, etc).
Seth: need to double check the flysystem api.

Other

Rosie's Modularization update from yesterday's open meeting
Jamboard from community discussion: What does Islandora do for you? What do you wish it could do?

🏠 islandora-community wiki home · 📆 community calendar · islandora website