June 8, 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: 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

• Rosie Le Faive (Chair)
• Alan Stanley
• Adam Vessey
• Melissa Anez
• Seth Shaw
• Yamil Suarez
• Alexander O'Neill
• Willow Gillingham
• Don Richards
• Jordan Dukart
• 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.

1. Issue Roundup - Link sorted by recently updated
2. PR Roundup- Link sorted by recently updated
3. Feel free to add more, or add something from Islandora Discussions

Minutes

Issue Roundup

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

• If it's an optional requirement would it still go in the composer requirements?
• Alexander: Depends how we want to have facets or not
• Facets as a dependency of islandora defaults and then the playbook, etc, don't have to keep track of that sort of thing - any reason we shouldn't update that
• Rosie: Is 1.6 the minimum compatibility right now?
• Seth: Before bumping to 2.02 is there any changes that would impact our configs?
• Seth: Check that optional configs still work with 2.2
• Alexander: Install it as normal with 1.6, then upgrade the module to 2.x
• Willow: at Born-Digital, bumped it up to facets 2.0. javascript rewritten in advanced search was done because of some slow down.

https://github.com/Islandora/jsonld/issues/61

• Seth: Publishing both the jsonld thats being published and the error popping up
• Error is associated with code recently added to deduplicate id
• Still works, but it's clogging the logs

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

• Rosie: we used to use a theme hook thats no longer being used in the OAI-PMH module
• Would it be possible to update it in the OAI-PMH module?
• Seth: The reason that theme hook went away is shift to plugin architecture - instead of overriding a theme, you would be addinga metadata plugin.
• Rosie: Make a plugin that will process the type relation field?
• Rosie: Will that create dependencies, or optional thing?
• Optional - if the field is of relationship field type, do the switch, but not presuming that any particular field is using it or not.

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

• It's a 32-bit problem rather than a GeoTiff problem
• Our version of image magic used by the playbook is a really old one that was compiled specially and uploaded to lyrasis paa
• Alexander: image magic was compiled specifically for jp2 support, this was a workaround for that
• It's used by Crayfish to generate derivatives of jp2 files
• Alex: Crayfish could check for ojp decompressed utility
• Seth: Question about maintenance
• Playbook currently on 20.04, not updated to 22.04
• Alex: Right now Cantaloupe looks for OpenJpeg and it can run that if it's there. Could crayfish look for either executable, if one doesn't support jp2 it could look for the other as a fallback?
• Seth: Simplest solution for now is updating the lyrasis ppa.
• Adam: At DGI, stops one of the compiler flags.

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

• Redundant content

https://github.com/Islandora-Devops/migrate_7x_claw/issues/40

• Alexander addressed and closed

Pull Request roundup

https://github.com/Islandora/.github/pull/1

• Check who else can merge on this repo
• Admin permissionsbc their admins on the islandora organization

https://github.com/Islandora-Devops/islandora-playbook/pull/218

• allowing dev el and it was causing the
• test on virtual box, check that it's drupal 9.3

https://github.com/Islandora/islandora_defaults/pull/66

• If there are other testers available, should just be a matter of re-importaing islandora defaults into a normal install and then checking for the manifest url.
• Alan can also check this out

https://github.com/Islandora-Devops/isle-dc/pull/253

• Don was working on this with Tom
• Running into issue with a .dev domain - may have to register another one?

Dependabot PRs - Bumping guzzle

• More composer files in the same repo generated these other dependabot PRs.
• It's been tested by Alexander.

https://github.com/Islandora-Devops/isle-dc/pull/263

• Merge conflict that needs to get resolved but it addresses 1.2 and 1.3 that gavin commented at the top.

https://github.com/Islandora-Devops/isle-dc/pull/213

• Addresses an old error and adds functionality to test if a view is broken - it adds the ability for it to fix the views
• A test pending - Yamil hoping to test for next tech call.

https://github.com/Islandora-Devops/isle-dc/pull/248

• Don: It built but saw something during build process wanted to check with Noah that this expected
• Otherwise testing worked fine

https://github.com/Islandora/islandora_ci/pull/10

• Failing on Drupal 10 and timing out because the version of drush that it was using doesn't support drupal 10.
• Now it fails because context doesn't exist for Drupal 10 yet.

https://github.com/Islandora/islandora/pull/878

• Rosie: looks like its checking for the Drupal version and allowing 7 and 8 as well
• Proceed with the CI testing PR
• An issue doesn't exist yet for Drupal 10 and contexts

https://github.com/Islandora/islandora/pull/877

• Expects an integer instead of the boolean
• Alexander to test

https://github.com/Islandora/documentation/pull/2096

• Seth: Other ACME PRs need to go in before this one, as it's documenting what's going on there.

https://github.com/Islandora/islandora_defaults/pull/63

• Seth: Question for Nat - if we should move his views tool that he pulled out of the default profile into an islandora repo
  • Should be in a place that the community maintains it.
• Rosie: We're also using fits right out of the roblib repository.

https://github.com/Islandora-Devops/islandora-playbook/pull/217

• Jordan: https://github.com/drupal-composer/drupal-project/commit/2c488fd3a12915f4083ca0e34c7e90917dc09ccb?

• Don: for those using ISLE DC we merged in a change that was kind of structural

  • All the docker compose files have been shifted into a directory cleaning up the folder,
  • https://github.com/Islandora-Devops/isle-dc/pull/223/files

• Documentation PR from Rosie to review https://github.com/Islandora/documentation/pull/2105