-
-
Notifications
You must be signed in to change notification settings - Fork 135
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use different var for meta author #68
Comments
You are correct @mattcen. Your suggestion is wise & reasonable. I will implement this when I get some time. |
Depending how much it annoys me, I may also give this a shot soon. I think it'll require a custom frontmatter field because there isn't a "standard" author field but that shouldn't be a big deal. |
(Oh, the other thing I haven't checked is whether the linked data name field is meant to be a full name, handle, or otherwise. Will need to do that, too.) |
@mattcen, I think it makes sense to use full name for the name field. Finally, we should have: <a href="link">Full name</a> If someone wishes to use any handle (let's say twitter), they can set the parameters (in frontmatter or in site config) such that we have: <a href="https://twitter.com/handle">@handle</a> This way, one can easily use handle(s). |
You are correct @mattcen. |
This issue has been marked stale due to inactivity. It will auto-close in 7 days without an update. |
This issue was auto-closed due to inactivity. |
I see here:
hugo-blog-awesome/layouts/partials/meta/post.html
Line 39 in ef3ce22
That the JSON linked data uses
Site.Params.github
as the name of any given post's author. This means that, if this var is unset (which is likely, since the docs don't say to set it), there is no author name, and if it is set, it is set for all posts, even those which set a different author in their page metadata.I think this should probably use per-page metadata if present, and fall back
Site.Params.author.name
. Does this seem reasonable?The text was updated successfully, but these errors were encountered: