Skip to content
This repository has been archived by the owner on Aug 18, 2020. It is now read-only.

Issue 572 #50

Merged
merged 3 commits into from
Jul 19, 2017
Merged

Issue 572 #50

merged 3 commits into from
Jul 19, 2017

Conversation

whikloj
Copy link
Contributor

@whikloj whikloj commented Jun 29, 2017

GitHub Issue: Islandora/documentation#572 and Islandora/documentation#665

What does this Pull Request do?

Downloads and installs the required openseadragon libraries, module and configures Cantaloupe to serve files from the Drupal filesystem.

Also updates system config to ensure ActiveMQ starts with system startup.

What's new?

This gets OpenSeadragon installed, it does NOT however enable it as a field formatter. See the testing instructions for more details.

How should this be tested?

You must go to Admin -> Structure -> Media Bundles, choose Manage Display from the Edit dropdown menu beside OBJ, and choose Openseadragon as the formatter for the File field.

Save

You may need to do this on both Default and Content tabs, depending if you change the View Mode below.

You then need to go to Admin -> Structure -> Content types, choose Manage Display beside Islandora Image.

Change the Format dropdown beside OBJ, from Label to Rendered Entity.

I'd also recommend clicking the little gear icon and changing the View Mode to Content for consistency.

Save

Create an Islandora Image and put an image (Jpeg, Gif, Png, or Tiff) in the OBJ field.

Remember to check "Include file in display" before you click "Create Media"

Save and publish

Interested parties

@dannylamb @jonathangreen

@dannylamb
Copy link
Contributor

@whikloj vagrant up

@dannylamb
Copy link
Contributor

:shipit: Had to clear my cache after trying to view, but totally works as expected. Thanks @whikloj

@dannylamb dannylamb merged commit be41a78 into islandora-deprecated:master Jul 19, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants