Thanks to visit codestin.com
Credit goes to github.com

Skip to content

feat(MedicalSpecialty):#4808

Merged
jlndr merged 3 commits into
schemaorg:mainfrom
jasonshanks:main
May 20, 2026
Merged

feat(MedicalSpecialty):#4808
jlndr merged 3 commits into
schemaorg:mainfrom
jasonshanks:main

Conversation

@jasonshanks
Copy link
Copy Markdown
Contributor

Added a missing MedicalSpecialty for Ophthalmologic. This differs from existing Optometric which is solely for the diagnose of, not surgical treatment of eye issues.

Added a missing MedicalSpecialty for Ophthalmologic. This differs from
existing Optometric which is solely for the diagnose of, not surgical
treatment of eye issues.
@jasonshanks
Copy link
Copy Markdown
Contributor Author

Apologies, before maintainer approves and merges I made a minor correction to the terminology to the more prevalent 'Ophthalmic'. Is there any known timeline on when this might occur?

@jasonshanks
Copy link
Copy Markdown
Contributor Author

Hello. I was what the status/timeline might be for getting this workflow approved and the PR merged into main branch?

rdfs:comment "A trial design in which the researcher knows the full details of the treatment, and so does the patient." ;
:isPartOf <https://health-lifesci.schema.org> .

:Ophthalmologic a :MedicalSpecialty ;
Copy link
Copy Markdown
Contributor

@jlndr jlndr May 19, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The term name and label should match. Should this be 'Ophthalmology'? See Dermatology, Pathology

Ophthalmology seems to be the relevant wikipedia entry

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes 'Ophthalmology' is correct. Do you need me to resubmit with change or will you?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Feel free to do it and I'll be able to merge

Copy link
Copy Markdown
Contributor Author

@jasonshanks jasonshanks May 20, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @jlndr . I've updated term and label now to: Ophthalmology

Comparing this more closely to the others, doesn't it seem like there is a mix of adjectives and nouns within MedicalSpecialty? For instance should these also change from:

  • Optometric -> Optometry
  • Osteopathic -> Osteopathy
  • Obstetric -> Obstetrics
  • Neurologic -> Neurology
  • Homeopathic -> Homeopathy
    several more, etc. etc.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

doesn't it seem like there is a mix of adjectives and nouns within MedicalSpecialty?

Yes definitely - seeing the differences between all them I'm not sure if we have/had any strict guidelines regarding this. Unfortunately it's hard to rename these once it's released and in use

- change term and label from ophthalmic to Ophthalmology
@jlndr jlndr merged commit 59947d1 into schemaorg:main May 20, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants