-
Notifications
You must be signed in to change notification settings - Fork 13
Open
Labels
Description
Expected behaviour:
According to the Presentation 3 specs, it is allowed to have a metadata key on a Range object. See: https://iiif.io/api/presentation/3.0/#a-summary-of-property-requirements
Observed behaviour:
When trying to add metadata to a Range, the Pydantic validation returns:
File "pydantic/main.py", line 363, in pydantic.main.BaseModel.__setattr__
ValueError: "Range" object has no field "metadata"
Potential bug location(s):
Adding metadata: Optional[Metadata] = None to the Range class in skeleton.py would fix this.
iiif-prezi3/iiif_prezi3/skeleton.py
Line 388 in 7c64e36
| class Range(Class): |
Any other comments:
I guess this then should also apply to Annotation, AnnotationPage, AnnotationCollection and what the spec calls 'Content Resources'? And for the other properties described in the spec table.