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 copy manifest functionality, what is iiif link? #278

Merged
merged 1 commit into from
May 5, 2023

Conversation

mathewjordan
Copy link
Member

@mathewjordan mathewjordan commented May 5, 2023

What does this do?

This adds three action items in the IIIF Manifest section of the Download and Share modal for works.

image

This addresses:

How should we review?

See https://preview-3322-iiif-whats-this.d2v1qbdeix3nr2.amplifyapp.com/items/62149fe9-b3f8-4713-b18a-caf09d1978f6 in the preview branch in AWS Amplify.

@mathewjordan mathewjordan self-assigned this May 5, 2023
@davidschober
Copy link

This looks awesome

Copy link

@davidschober davidschober left a comment

Choose a reason for hiding this comment

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

looks great!

@davidschober
Copy link

ps I'm loving feature previews.

@mathewjordan mathewjordan force-pushed the preview/3322-iiif-whats-this branch from 46464ef to aa23b6d Compare May 5, 2023 20:49
@mathewjordan mathewjordan merged commit 7a7ecf5 into deploy/staging May 5, 2023
@mathewjordan mathewjordan deleted the preview/3322-iiif-whats-this branch May 5, 2023 20:50
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