Skip to content

Update FAQ #329

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Update FAQ #329

wants to merge 1 commit into from

Conversation

phiro56
Copy link

@phiro56 phiro56 commented Jul 6, 2025

Restructure FAQ page with new template system

This PR restructures the FAQ page (topics/faq.md) to use a new template-based format that provides better organization and improved ui.

Template Dependencies

Coordination Required: This PR depends on corresponding valkey-io PR: #294

Template Functions Used:

  • {% faq_section(class="...") %} - Section wrapper with styling classes
  • {% end %} - Section closing tag

Note: The spellcheck warnings for faq_section are expected as this is a custom template function.

- Update FAQ Q&A
- Add Zola Shortcodes to section out categories

Signed-off-by: Daniel Phillips <[email protected]>
@stockholmux
Copy link
Member

@zuiderkwast How does this affect the generation of man pages? I see it has some template tags in it.

@zuiderkwast
Copy link
Contributor

Yeah, the template tags won't work in man pages. The docs are not only targeting our website so let's not add web templating dependencies. See the readme in this repo.

Maybe the FAQ is not needed in man pages and alike? If it's not really documentation (as in user manuals) we could consider moving pages like this out of docs and into the website repo...

Otherwise, just drop the template tags.

@stockholmux
Copy link
Member

@zuiderkwast Yep. I think it makes sense to move the FAQ into the website.

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