Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove obsolete Crayfish parameter syn_config. #325

Merged
merged 1 commit into from
Feb 14, 2024

Conversation

alxp
Copy link
Contributor

@alxp alxp commented Feb 14, 2024

Related GitHub issue:

Islandora/Crayfish#181

From my understanding of Crayfish's UPGRADE.md file, this config key is no longer used and its presence is causing errors with Hypercube.

https://github.com/Islandora/Crayfish/blob/4.x/Hypercube/UPGRADE.md

It looks like the other config changes needed for the upgrade are already in place.

@jefferya
Copy link
Contributor

Tested locally. Fixes problem.
My test:

  1. Reproduce error: use Tag 3.1.2 and import a PDF via Islandora Workbench;
    • results: hypercube "syn_config" error and no OCR media
  2. Update to image produces by this PR and import PDF via Islandora Workbench;
    • results: no errors in the hypercube log; OCR media generated

@seth-shaw-asu seth-shaw-asu merged commit e663f41 into main Feb 14, 2024
76 checks passed
@rosiel rosiel deleted the hypercube-syn-config branch February 14, 2024 18:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants