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

External files ingest as "related" links (#958) and show them in sidebar (#959) #1002

Merged
merged 11 commits into from
Jan 2, 2024

Conversation

blms
Copy link
Contributor

@blms blms commented Dec 5, 2023

In this PR

Per #958:

  • In ingest, allow a related header in metadata, which should be populated with semicolon-separated URLs
  • Add each of these URLs as a RelatedLink related to the Manifest
  • Guess the MIME type based on the extension
  • Separate seeAlso (structured data per IIIF spec) from related (generally, other related links per IIIF spec) and use the latter for things populated by related in the metadata
    • Add new RelatedLink.is_structured_data to indicate whether it's structured data or not

Per #959:

  • Show both seeAlso and related in the sidebar

@blms blms requested a review from jayvarner December 5, 2023 20:20
@jayvarner jayvarner merged commit efb74aa into develop Jan 2, 2024
@jayvarner jayvarner deleted the feature/958-external-files branch January 2, 2024 13:26
jayvarner added a commit to ecds/readux-ingest-ecds that referenced this pull request Jan 30, 2024
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