Skip to content
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

Add related content links to pages #163

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Add related content links to pages #163

wants to merge 3 commits into from

Conversation

divinerites
Copy link

No description provided.

@divinerites
Copy link
Author

Need a recent version for use of =

May be put the request on one line if you want to keep with old versions.

@Pilskalns
Copy link

Hi @divinerites,

Please also add also documentation example to this PR - what is related shortcode and how to use it.

P.S. As there are many commits, in the end will merge with squash, just FYI.

@divinerites
Copy link
Author

divinerites commented Oct 26, 2018

gohugo has interesting feature : related content who can fit perfectly with a lot of use case for documentation.

This partial implement this feature for all page and sections with a keywords parameter in their front-matter.

---
keywords: [technical, go]
---

This partial will be implemented on all the pages (due to #164 )
On a side note, if this is not wanted for all cases, we can add a general true/false parameter to activate or not the feature.

@divinerites
Copy link
Author

P.S. As there are many commits, in the end will merge with squash, just FYI.

Yes, As I directly edited on GitHub I couldn't find a way to make several commits an only one PR with those commits. Sorry.

@Pilskalns
Copy link

Ohh.. I see what you mean here, this is not a shortcode despite the PR title.

I think good global variable could be something like relatedLinksCount with default to 3-5 or something and if the one wants to disable the feature, settings count to 0/false/nil (?) should disable it at all.

The variable should allow per-page override, like if you want somewhere to disable them at all or show more.

Also, more granual features could be achieved by elaborating Hugo built in config

@Pilskalns Pilskalns changed the title Add related shortcode Add related content links to pages Oct 26, 2018
@divinerites
Copy link
Author

Ohhh I see too :) this is a partial !!! Sorry.
So OK, thanks for your message, I'll make the proposed changed asap.

thanks.

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.

2 participants