docs(model-serving): capture endpoint detail ready-alert and replica-filter screenshots#7459
Conversation
|
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
How to use the Graphite Merge QueueAdd either label to this PR to merge it via the merge queue:
You must have a Graphite account in order to use the merge queue. Sign up using this link. An organization admin has required the Graphite Merge Queue in this repository. Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue. This stack of pull requests is managed by Graphite. Learn more about stacking. |
There was a problem hiding this comment.
Pull request overview
This PR removes screenshot TODO placeholders in the model-serving user manual after the corresponding screenshots (endpoint_detail_ready_alert.png and replica_status_filter.png) were captured for all four locales. It is a small follow-up to PR #7443.
Changes:
- Removes two
<!-- TODO: Capture screenshot — ... -->HTML comments per locale, now that the referenced images exist. - Applied identically to en/ja/ko/th versions of
model_serving.md.
Reviewed changes
Copilot reviewed 4 out of 12 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| packages/backend.ai-webui-docs/src/en/model_serving/model_serving.md | Removes two screenshot TODO comments |
| packages/backend.ai-webui-docs/src/ja/model_serving/model_serving.md | Removes two screenshot TODO comments |
| packages/backend.ai-webui-docs/src/ko/model_serving/model_serving.md | Removes two screenshot TODO comments |
| packages/backend.ai-webui-docs/src/th/model_serving/model_serving.md | Removes two screenshot TODO comments |
…filter screenshots
a9a6165 to
c26480f
Compare

Follow-up polish that resolves the screenshot
TODOplaceholders introduced by #7443 inmodel_serving.md. No separate issue (small completion of #7443's own scope).Summary
endpoint_detail_ready_alert.pngandreplica_status_filter.pngfrom the live WebUI on a healthy deployment, in all 4 locales (en/ja/ko/th), at 2× zoom viawindow.switchLanguageperSCREENSHOT-GUIDELINES.md.<!-- TODO: Capture screenshot — ... -->comments insrc/{en,ja,ko,th}/model_serving/model_serving.md(8 lines total). Image references and surrounding blank lines unchanged.Intentionally left as-is
endpoint_detail_shm.pngTODO kept. The live Endpoint Detail Page has no "Service Info" card (it renders "Basic Information") and surfaces no Shared Memory (SHM) field for the available deployment (verified via DOM scan). Capturing it would require fabrication — and it signals a doc↔UI mismatch in docs(model-serving): update for deployment-only create modal, revision UX, and detail page polish #7443's prose that should be addressed in docs(model-serving): update for deployment-only create modal, revision UX, and detail page polish #7443, not faked here.TODO(Deployment Presets link) kept.deployment_presets.mdis added by docs(model-serving): add Deployment Preset management documentation #7444, which is stacked above docs(model-serving): update for deployment-only create modal, revision UX, and detail page polish #7443; restoring the link here would create a broken internal link. Belongs in docs(model-serving): add Deployment Preset management documentation #7444's scope.Notes for #7443 author
Test plan
src/{en,ja,ko,th}/images/endpoint_detail_ready_alert.pngandreplica_status_filter.pngrender in the manual.model_serving.md.🤖 Generated with Claude Code