search

IIIF / website

IIIF website context (except for API and model specifications)
http://iiif.io
8 stars 35 forks source link

Several versions of JSON-LD context? #273

Open aisaac opened 4 years ago

aisaac commented 4 years ago

While checking the implementation of the new mapping for "profile" and related elements I found that the official JSON context is at https://iiif.io/api/presentation/3/context.json but there is still an old context at https://iiif.io/api/presentation/3/combined-context.json which actually doesn't reflect PR IIIF/api#1722 (considering that this PR seems to be for the 'combined context' file).

azaroth42 commented 4 years ago

Thanks Antoine!

This seems to be a deployment issue, as the combined-context file is not in the master branch: https://github.com/IIIF/api/tree/master/source/api/presentation/3

Transferred the issue to website and assigned to Glen for investigation.

glenrobson commented 4 years ago

Hi Rob, is this just a case of deleting the combined-context.json? (just checking I've understood correctly).

azaroth42 commented 4 years ago

Yes ... but that's just a symptom of the issue. The file isn't in the master branch any more and hence shouldn't be being deployed. Does the deployment script only update files, and not check for deletions?

glenrobson commented 4 years ago

Your right the deployment script only adds and updates not delete. I've manually deleted it for now but will look into fixing the deployment scripts.