is it possible to make ContactPerson be invalid without email?

[yarikoptic]

We have https://github.com/dandisets/000108/blob/draft/dandiset.yaml where contact person is Lee , but he is listed without email.

- affiliation: []
  includeInCitation: true
  name: Kamentsky, Lee
  roleName:
  - dcite:ContactPerson
  schemaKey: Person
- affiliation: []
  email: [email protected]
  identifier: 0000-0002-8209-0059
  includeInCitation: true
  name: Marx, Slayton
  roleName:
  - dcite:DataManager
  - dcite:DataCurator
  - dcite:Investigation
  - dcite:ProjectMember
  - dcite:Researcher
  - dcite:Software
  - dcite:Supervision
  - dcite:Visualization
  schemaKey: Person

PS I now made @slaytonmarx to be the contact person since AFAIK Lee is no longer with the project. Please adjust if that is not true

[yarikoptic]

@satra @candleindark - how to express such a requirement in pydantic ?

[candleindark]

@satra @candleindark - how to express such a requirement in pydantic ?

Change

dandi-schema/dandischema/models.py

Line 823 in 57b503a

email: Optional[EmailStr] = Field(None, json_schema_extra={"nskey": "schema"})

to

email: Optional[EmailStr] = Field(..., json_schema_extra={"nskey": "schema"}) will do.

or

email: EmailStr = Field(..., json_schema_extra={"nskey": "schema"}) will even be better since I don't see a situation that email will be None if you make it as a required field.

General reference for this can be found at https://docs.pydantic.dev/latest/concepts/models/#required-fields.

Just be aware this change will entails a change in the schema, making the field required in the schema as well, so you may need to issue a new patch version

[yarikoptic]

no -- it should be only if

roleName:
  - dcite:ContactPerson

[candleindark]

@yarikoptic I have submitted a draft PR to provide a model level validator to ensure that a contributor acting as a contact person has an email provided. Let me know if the PR is heading to the direction of solving the problem. If it is, I will finalize the PR by adjusting the tests, etc.

[yarikoptic]

closed by

• Add validator to ensure a contact person has email provided #235

next time @candleindark please use Closes #NUMBER in PR description.