Comments arising from viewing Mr Reibey's entry (probably more for the compilers of the template than for thomaswwp):

Have a separate property for surname so that people can be listed by surname and so that other things can be done with surnames. Example: https://familypedia.fandom.com/wiki/Smith_(surname)/variants

Delete the "th" from "13th" so as to be more useful internationally.

Separate the day, month, and year if you ever want to tabulate everyone born on a specific day or in a specific month or year. Examples:
*Bottom of Familypedia's main page lists people who had events on the current day: https://familypedia.fandom.com/wiki/Family_History_and_Genealogy_Wiki
*https://familypedia.fandom.com/wiki/Concept:Born_in_New_South_Wales_in_1890

Separate the elements of the places if you ever want to list everyone born in Devon etc. - https://familypedia.fandom.com/wiki/Devon/bdm

If allowing more than two parents, why stop at 3? Adoption usually involves two more, and it is possible to be adopted more than once.

"Rev." should be part of a separate property. (people looking for James Haydock Reibey should not have to look under "Rev" if they don't know he became one - e.g. if they have his birth certificate.)

See (and copy if you like) https://familypedia.fandom.com/wiki/Template:Showfacts_person - yes, it is complex, but so is genealogy; the big commercial genealogy sites let people search by one or more of numerous minor details

You get "My test name (Genealogy parser function type not recognised: description=My test description) "

This looks like an error in person_template.wikitext, the parser function should have a pipe rather than an equals sign: {{#genealogy:description|Desc here}}. I'll fix the template file; @thomaswwp you'll need to update your template.

If allowing more than two parents, why stop at 3? Adoption usually involves two more, and it is possible to be adopted more than once.

The Genealogy extension doesn't limit it to three; the example template just has three to make it clear that it's not constrained to two. That template is not necessarily the best way to model things, it's just an example. The extension itself basically tries to do as little as possible while still making it easy to link between people etc. Modelling dates, places, etc. is mostly out of scope. (I do think approximate date modelling could be included, although perhaps as part of a separate extension. Places should not be, as they are adequately handled by Wikidata and are also not only applicable to genealogical research.)

Change #1129596 had a related patch set uploaded (by Samwilson; author: Samwilson):

[mediawiki/extensions/Genealogy@master] Fix description parser function syntax

https://gerrit.wikimedia.org/r/1129596

In T389439#10655295, @Samwilson wrote:

You get "My test name (Genealogy parser function type not recognised: description=My test description) "

This looks like an error in person_template.wikitext, the parser function should have a pipe rather than an equals sign: {{#genealogy:description|Desc here}}. I'll fix the template file; @thomaswwp you'll need to update your template.

Thanks - I confirm that fixed it immediately. I am entirely new to MediaWiki and templates so was unable to diagnose that myself. Thank you.

In T389439#10655252, @Robin_Patterson wrote:

Comments arising from viewing Mr Reibey's entry (probably more for the compilers of the template than for thomaswwp):

You have made some pretty bold assumptions about my use case @Robin_Patterson; I am sure you did not mean to come over as quite so patronising.

I would think those should go under "Request a Software Feature" not bugs though.

Have a nice day.

Change #1129596 merged by jenkins-bot:

[mediawiki/extensions/Genealogy@master] Fix description parser function syntax

https://gerrit.wikimedia.org/r/1129596