-
Notifications
You must be signed in to change notification settings - Fork 283
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
Docsy theme #244
Docsy theme #244
Conversation
First pass at integrating etcd into a shell docsy themed hugo site. * Moving content/docs to content/en/docs * Updating netlify.toml, config.toml, and Docker config * first pass at versions dropdown menu for docs pages * Updating hugo version to 0.81.0 (ext) in Dockerfile and netlify.toml * updating preview-build command * updating package.json * requiring lts node version * Adding content to en/_index.html, updating config.toml, updating navbar-version-selector to ensure it works with multiple languages. * updating package.json & removing package-lock.json * changing from npm ci to npm install as we removed the package-lock.json file Signed-off-by: Nate W <4453979+nate-double-u@users.noreply.github.com>
* Drop About from docsy MVP, and make link checker happy * Homepage: remove data-proofer-ignore
(For sake of consistency)
Looks amazing thank you @nate-double-u and team. LGTM |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm, great look and feel as we discussed! Thanks @nate-double-u @chalin
Wonderful, thanks for the feedback @hexfusion and @spzala! |
@nate-double-u - can you merge this today (either before or after #203), given that this PR has been open for a week? While I might not get to doing etcd work today, if you can merge before the EOD, I'll be able to make some progress at least as of tomorrow morning (ET). |
Followup changes will be tracked via #247 |
Conversion to the Docsy theme is complete.
You'll find the preview next and screenshots below:
Preview: https://docsy--etcd.netlify.app
@spzala @ptabor @jberkus - PTAL and let @nate-double-u and I know if you find that this is good enough to use as a base for continued development.
If there are no objections, we'd like to merge this by Thursday (04/22) at the latest so that we can then move ahead with the implementation of the new IA, and any further changes that you'd like made to the site.
Screenshots
Homepage:
Site page: