yalelibrary / YUL-DC

Preliminary issue tracking for Yale University Libraries Digital Collections project
3 stars 0 forks source link

Add the PDF link to the manifest as part of the manifest generation process #764

Closed mark-dce closed 3 years ago

mark-dce commented 3 years ago

ACCEPTANCE

ACCEPTANCE NOTE The link does not need to function correctly until ticket #763 - Serve PDFs from a web service, is complete. For acceptance, we only need to validate that the link exists and has the expected form.

NOTE See https://wellcomelibrary.org/iiif/b20417081-0/manifest for an example manifest with downloadable PDF rendering. The main work page is here https://wellcomelibrary.org/item/b20417081#?c=0&m=0&s=0&cv=0&z=-0.6522%2C-0.0828%2C2.3042%2C1.6564

image.png

 "sequences": [
    {
      "@id": "https://wellcomelibrary.org/iiif/b20417081-0/sequence/s0",
      "@type": "sc:Sequence",
      "label": "Sequence s0",
      "rendering": [
        {
          "@id": "https://dlcs.io/pdf/wellcome/pdf-item/b20417081/0",
          "format": "application/pdf",
          "label": "Download as PDF"
        },
        {
          "@id": "https://wellcomelibrary.org/service/fulltext/b20417081/0?raw=true",
          "format": "text/plain",
          "label": "Download raw text"
        }
      ],
      "viewingHint": "paged",
      "canvases": [
        {
          "@id": "https://wellcomelibrary.org/iiif/b20417081-0/canvas/c0",
          "@type": "sc:Canvas",
          "label": " - ",
          "thumbnail": {
            "@id": "https://dlcs.io/thumbs/wellcome/5/b20417081_001_0001.jp2/full/67,100/0/default.jpg",
            "@type": "dctypes:Image",
mikeapp commented 3 years ago

Testing the manifest above, it does not show a download link in Mirador 3 (that I can find).

mikeapp commented 3 years ago

Note that Wellcome puts the rendering at the level of the sequence. More expected (and might possibly work in Mirador) is to put it at the root level of the Manifest, per the example in the spec at https://iiif.io/api/presentation/2.1/#c-example-manifest-response

maxkadel commented 3 years ago

Note that Wellcome puts the rendering at the level of the sequence. More expected (and might possibly work in Mirador) is to put it at the root level of the Manifest, per the example in the spec at https://iiif.io/api/presentation/2.1/#c-example-manifest-response

mikeapp commented 3 years ago

Mirador 3 test shows it is looking at Manifest level (at least):

Screen Shot 2020-11-09 at 1.32.11 PM.png

mark-dce commented 3 years ago

Looks good in UV and Mirador - see https://collections-uat.library.yale.edu/catalog/12222098

image.png

image.png