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

Follow-up on caching Height and Width on the media: Add config schema for new Action, and adjust wording. #1044

Merged
merged 1 commit into from
Aug 8, 2024

Conversation

rosiel
Copy link
Member

@rosiel rosiel commented Jul 17, 2024

GitHub Issue: n/a

  • Other Relevant Links (Google Groups discussion, related pull requests,
    Release pull requests, etc.)

#1029

What does this Pull Request do?

A little housekeeping, creating a config schema, and some rewording

What's new?

  • Config schema.

  • Wording changes

    • In the Actions config page, there's a blurb at the top talking about how there are "simple" and "advanced" actions. Since this is the terminology used by Drupal, I changed "complex" to "advanced".
    • When talking about the Media Use to do this action on, I removed the word "Source". It implies a "destination" and more confusingly, implies that the destination is not the source. What we're actually selecting is "the Media to do the thing to", being both Source and Destination.
    • Removed hard coded references to Original File (in comments/documentation) since you could do this to any media. (Admittedly, in our standard config, Original File makes the most sense)
  • It looks like there was a stray 'media_use_term' that was later renamed to 'source_term_uri'.

  • Does this change add any new dependencies? nope

  • Does this change require any other modifications to be made to the repository
    (i.e. Regeneration activity, etc.)? nope

  • Could this change impact execution of existing code? nope

How should this be tested?

Due to the nature of the all-string config schema, as well as the useful dropdown list to choose from, you could do what I did and mess with the schema (change one value to Boolean) then inspect the saved config in Devel's Config Editor or however you like to inspect your configs. In my case, the value stored was "True".

Documentation Status

  • Does this change existing behaviour that's currently documented? this changes documentation about existing behaviour.
  • Does this change require new pages or sections of documentation? Yeah maybe there could be a page in the docs.
  • Who does this need to be documented for? site managers getting frustrated with IIIF slowness.
  • Associated documentation pull request(s): ___ or documentation issue ___

Additional Notes:

Any additional information that you think would be helpful when reviewing this
PR.

Interested parties

Tag (@ mention) interested parties or, if unsure, @Islandora/committers

@aOelschlager aOelschlager self-requested a review July 17, 2024 17:11
Copy link

@aOelschlager aOelschlager left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just tested this and it works. I had to update the database because of schema change.

@aOelschlager aOelschlager merged commit 2b9b320 into Islandora:2.x Aug 8, 2024
27 checks passed
@rosiel rosiel deleted the iiif-additions branch August 8, 2024 17:54
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.

2 participants