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

Add image loading (to fix breaking tests on Drupal 9.4.x). #872

Closed
wants to merge 3 commits into from

Conversation

rosiel
Copy link
Member

@rosiel rosiel commented May 17, 2022

GitHub Issue: Islandora/documentation#2093

  • Other Relevant Links : Tests are failing on 9.4.x on all Islandora PRs

What does this Pull Request do?

Tries to correct for the new image formatter setting by supplying it in the functional tests.

What's new?

This now passes the "image_loading" parameter. Hopefully it won't break on 9.3.x where this feature isn't available.

How should this be tested?

If the tests all pass, then this worked.

Documentation Status

  • Does this change existing behaviour that's currently documented? no
  • Does this change require new pages or sections of documentation? no
  • Who does this need to be documented for? no-one
  • Associated documentation pull request(s): ___ or documentation issue ___

Additional Notes:

Interested parties

@Islandora/8-x-committers

@rosiel rosiel changed the title Add image loading. Add image loading (to fix breaking tests on Drupal 9.4.x). May 17, 2022
@rosiel
Copy link
Member Author

rosiel commented May 17, 2022

Closing, my approach was wrong.

The ticket is solved by #869

@rosiel rosiel closed this May 17, 2022
@rosiel rosiel deleted the image_loading branch November 25, 2022 20:48
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.

1 participant