docs: Localize onboarding images and update flow#106
Open
ChrisGammell wants to merge 6 commits into
Open
Conversation
Fast-forward the Golioth Docs submodule (dd7ee4c..80a1630) and correct the README: run npm commands from website/, and use `git submodule update --remote --merge` instead of the detached-HEAD- incompatible `git submodule foreach git pull`. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
….golioth.io - Bump @docusaurus/core and preset-classic 3.5.2 -> 3.10.1 (regenerates lockfile) - Add @easyops-cn/docusaurus-search-local for on-site full-text search - Move site URL golioth.github.io -> training.golioth.io - Add JSON-LD Course structured data, default OG social-card image, site-wide keywords, robots.txt, and llms.txt for LLM/search discoverability Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Replace the two submodule symlink partials (2-register, 3-manage-devices) on the Console page with local copies under website/, and vendor their 11 images into _partials/assets/console/. This decouples the page from submodule-goliothdocs so the cloud build serves these images without a golioth/docs update. Ten images are pulled from the current submodule working-tree edits; the credentials-panel image is restored pristine from submodule HEAD as a placeholder pending the intended replacement. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Drop the GoliothDeviceCreateConfirm import (trimmed from the device-creation row) and delete three images no longer referenced by the partials: console-quickstart-step2-confirmation.png, gettingstarted-console-createdevice.png, and gettingstarted-console-deviceview-credentialspanel.png. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
MDX partial files must not contain front matter. Locally this is only a warning, but with CI=true (GitHub Actions) Docusaurus escalates it to a hard error, failing the build. The id/title front matter is meaningless in these import-only partials, so drop it. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
ChrisGammell
force-pushed
the
feature/localize-console-page-images
branch
from
bf0c1f4 to
a450c28
Compare
|
Visit the preview URL for this PR (updated for commit a450c28): https://golioth-training--pr106-feature-localize-con-ebeux671.web.app (expires Wed, 22 Jul 2026 03:16:48 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: c5dc029ca233830e70690d9f5440a110931e16d2 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Decided to pull images in locally to reduce dependency on main docs page, treating this as a prototype for changes we need to make on the docs pages as well for onboarding.
I liked that I was able to trim things up easier and improve the flow.
I also updated all images with more modern captures that represent the experience of onboarding a new device on the Console in 2026.