Some style comments if you have the energy to fix them:
When using the code macro you could always specify the language as otherwise 2 problems occur: 1) performance is dropped a lot since pygments needs the analyze the content to try to infer the language and 2) in a lot of cases the wrong language will be inferred. Thus for velocity content for ex, always use language='velocity' for velocity content (you’ll see that it’ll remove all the red boxes for example ;)). Added to https://dev.xwiki.org/xwiki/bin/view/Community/DocGuide#HMacros
When using links please always use wiki link syntax and not URLs since 1) this means that if we rename pages in the future links specified as URL won’t be refactored automatically and 2) URL links are indicated as external links which is not correct since the links are internal. See https://dev.xwiki.org/xwiki/bin/view/Community/DocGuide#HLinking
Using ##…## everywhere would be great as it makes the code more readable IMO. I learnt this with my editor at Manning when I wrote my JUnit in Action book
One difference is that the FAQ tutorial involves a very trivial object: a FAQ object contains just two things: a “question” and an “answer”, while this example uses a Person object that has more properties (##name##,##age##,##sex##, etc.)
language=‘velocity’ didn’t work for me. Could it be that ‘velocity’ is not a supported language? I’ve changed it to ‘none’ for now.
I’ve used xwiki link syntax where I could, but can’t seem to get that to work for links like: https://www.xwiki.org/xwiki/bin/view/ScriptingDocumentation/
I also added to your style guide page, telling users to look in the Information tab for the “id” of a page. Suggestion: would be nice to also list the wiki name in the Information tab. Also, for some reason, it doesn’t seem to work when I use parameters. For example, this link doesn’t work: xwiki:Documentation.UserGuide.Features.XWikiSyntax.WebHome?syntax=2.1§ion=Groups