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

[BUG] Error from missing taxonomy term for fits derivative #13

Closed
nigelgbanks opened this issue Oct 19, 2022 · 5 comments
Closed

[BUG] Error from missing taxonomy term for fits derivative #13

nigelgbanks opened this issue Oct 19, 2022 · 5 comments
Assignees
Labels
bug Something isn't working

Comments

@nigelgbanks
Copy link
Contributor

I get the following error when attempting to add a PDF original file media entity to a Digital Document object.

Error message
Error generating event: Could not locate taxonomy term with uri: https://projects.iq.harvard.edu/fits

Seems to stem from the derivative action making a reference to a taxonomy term that does not exist. See the config references here.

Seems like this entry should be added to the migration tags, here? Though it looks like the intention is to handle it in the module from this pull request. The read me seem to reflect this. So perhaps the read me & composer files need updating?

@nigelgbanks nigelgbanks added the bug Something isn't working label Oct 19, 2022
@nigelgbanks nigelgbanks changed the title [BUG] [BUG] Error from missing taxonomy term for fits derivative Oct 19, 2022
@rosiel
Copy link
Contributor

rosiel commented Oct 19, 2022

nice catch!

@rosiel rosiel self-assigned this Oct 19, 2022
@adam-vessey
Copy link
Collaborator

adam-vessey commented Oct 19, 2022

Is... this really just that the islandora_fits_tags was left out of the README's Usage, step 6 dealio (fits was added back in later in the process, and was apparently left out, whoops)?

Both isle-dc and islandora-playbook should be handling running this migration?

How exactly did you get into this situation? Steps-to-reproduce, almost... at the environment level?

@nigelgbanks
Copy link
Contributor Author

Oh, I just was setting it up more or less manually in my little test repo here: https://github.com/nigelgbanks/starter-site-test, as I wanted to use it isolated from the other two (eventually sandbox will be based off it, and it doesn't use isle-dc or the islandora-playbook there are quite a few installations out there that just roll their own as well). You're most likely right that it's just missing from the read me, I'll test tomorrow.

@adam-vessey
Copy link
Collaborator

@nigelgbanks: With #14 having been merged, can this issue now be closed?

@nigelgbanks
Copy link
Contributor Author

@nigelgbanks: With #14 having been merged, can this issue now be closed?

Whoops yup 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants