Skip to content

DEV: restructure command pages #1648

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

Merged
merged 1 commit into from
Jun 4, 2025
Merged

DEV: restructure command pages #1648

merged 1 commit into from
Jun 4, 2025

Conversation

dwdougherty
Copy link
Collaborator

@dwdougherty dwdougherty commented Jun 2, 2025

DOC-5298

This PR contains more extensive command page simplifications in preparation for command versioning. Currently, each command page has the following layout (using the SET command to illustrate):

/content/commands/set/index.html
/content/commands/set/syntax.svg

After this change, each page will be a separate Markdown page under /content/commands; e.g.,

/content/commands/set.md

The SVG files have been removed because (1) they weren't complete/usable and (2) they weren't used at all on the site.

Simplification is the watchword here. :)

This work is part of the DEV: add versioning for command pages epic.

@dwdougherty dwdougherty self-assigned this Jun 2, 2025
@dwdougherty dwdougherty added the dev label Jun 2, 2025
@dwdougherty dwdougherty requested a review from a team June 2, 2025 15:39
Copy link
Contributor

@andy-stark-redis andy-stark-redis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Repo is now much neater and leaner (2.7MB vs 5.1MB with all the SVGs, etc), and links work, a few test pages I tried look exactly the same, as they should... Basically, LGTM - nice improvement :-)

@dwdougherty
Copy link
Collaborator Author

Thank you, @andy-stark-redis!

@dwdougherty dwdougherty merged commit a4f3ff6 into main Jun 4, 2025
5 checks passed
@dwdougherty dwdougherty deleted the DOC-5298 branch June 4, 2025 13:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants