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

Overhaul the Tech Docs website #198

Merged
merged 69 commits into from
Dec 13, 2024
Merged

Overhaul the Tech Docs website #198

merged 69 commits into from
Dec 13, 2024

Conversation

brianzelip
Copy link
Contributor

@brianzelip brianzelip commented Nov 11, 2024

This PR overhauls the Tech Docs website to use Astro for static site generation instead of Jekyll. The big upgrade here is the change in the website theme, from the Jekyll 'Minimal' theme to Astro 'Starlight'. The Starlight theme provides many UX improvements, including better navigation, search (!!), and design.

TechDocsAstro

About the docs content

The docs content is now found in src/content/docs/.

All of the docs content was left largely intact, save for syntax formatting, internal link adjustments, image path adjustments, and frontmatter changes to accommodate the new framework. New documentation was also added to src/content/docs/about/ to help with authoring Tech Docs content and maintaining the Tech Docs application.

How the new Astro site works

See src/content/docs/about/development.md for further info.

Jekyll site deprecation

The Jekyll site has been deprecated via the TD-18-deprecated-jekyll branch which contains the GitHub Actions workflow that published the final iteration of the jekyll codebase as a set of static files with all source code removed.

⚠️ Don't delete the branch after merging! ⚠️

Don't delete the astro-starlight branch after merging! We’ll need to make some Cloudflare settings changes and a commit to the codebase after it’s merged before we can delete the branch and start serving the new site from master.

Add js script for md file transformations
Add prettier for formatting via cli
Update config to recognize Footer component and show Starlight credit
Replace dirs in /architecture with only index.md files
If a dir contained only index.md file, then rename index.md
after the dir name, mv the .md file up a level, and rmdir empty dir

Rename architecture/backend/index.md --> backend.md
Then mv architecture/backend/*.md ../../ and rmdir

Flatten /administration and fix some relative links
Assert an opinion about directory order
Order each directory's content via its index.md content
Add missing Architecture files
Nest an Upgrading section under Administration
Reorder auth files under Customization
Fix credits broken from upgrade in custom footer
Add README_FEATURES_1.5.0.md content to upgrading_1_5_0.md and delete the README
Add aside syntax
Clean up headings, etc.
@brianzelip brianzelip marked this pull request as draft November 11, 2024 21:27
Copy link

cloudflare-workers-and-pages bot commented Nov 12, 2024

Deploying archivesspace-tech-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: ccd665a
Status: ✅  Deploy successful!
Preview URL: https://dd67aedc.archivesspace-tech-docs.pages.dev

View logs

Pass env vars to Cypress via npm scripts
Update about dev docs
Make import aliases explicit per directory
Update Sidebar nav link hover styles
@brianzelip brianzelip marked this pull request as ready for review November 29, 2024 11:51
@brianzelip brianzelip requested a review from thimios November 29, 2024 11:51
@brianzelip brianzelip changed the title Rewrite Tech Docs from Jekyll to Astro Overhaul the Tech Docs website Dec 3, 2024
@brianzelip brianzelip changed the title Overhaul the Tech Docs website Upgrade the Tech Docs website Dec 3, 2024
@brianzelip brianzelip changed the title Upgrade the Tech Docs website Overhaul the Tech Docs website Dec 3, 2024
@brianzelip brianzelip merged commit 8365484 into master Dec 13, 2024
4 checks passed
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