Skip to content

Commit

Permalink
Merge pull request #265 from dandi/enh-minor
Browse files Browse the repository at this point in the history
doc: A comment on current identification of URI and missing space into a description string
  • Loading branch information
yarikoptic authored Nov 18, 2024
2 parents 2fede03 + c25b43d commit 9812ced
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion dandischema/models.py
Original file line number Diff line number Diff line change
Expand Up @@ -567,6 +567,7 @@ def __get_pydantic_json_schema__(
if value.get("title") is None or value["title"] == prop.title():
value["title"] = name2title(prop)
if re.match("\\^https?://", value.get("pattern", "")):
# triggers only for ROR in identifier
value["format"] = "uri"
if value.get("format", None) == "uri":
value["maxLength"] = 1000
Expand Down Expand Up @@ -623,7 +624,7 @@ class PropertyValue(DandiBaseModel):
) # Note: recursive (circular or not)
propertyID: Optional[Union[IdentifierType, AnyHttpUrl]] = Field(
None,
description="A commonly used identifier for"
description="A commonly used identifier for "
"the characteristic represented by the property. "
"For example, a known prefix like DOI or a full URL.",
json_schema_extra={"nskey": "schema"},
Expand Down

0 comments on commit 9812ced

Please sign in to comment.