-
Notifications
You must be signed in to change notification settings - Fork 254
docs: tidy up tenant-onboarding guide and link to guides index#4401
docs: tidy up tenant-onboarding guide and link to guides index #4401SurbhiAgarwal1 wants to merge 2 commits intokptdev:main from
Conversation
✅ Deploy Preview for kptdocs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
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.
Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.
@liamfallon
liamfallon
left a comment
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.
Excellent work! Just one small suggestion from me.
- Add Tenant Onboarding to guides/_index.md listing - Add WIP notice at top of tenant-onboarding.md - Fix broken image path (/static/images/ -> /images/) - Rewrite summary to remove reference to non-existent next guide Fixes kptdev#4397 Signed-off-by: Surbhi <agarwalsurbhi1807@gmail.com>
Co-authored-by: Liam Fallon <35595825+liamfallon@users.noreply.github.com> Signed-off-by: Surbhi <agarwalsurbhi1807@gmail.com>
3179465 to
189e1d3
Compare
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.
Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.
@liamfallon
liamfallon
left a comment
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.
Thanks!
Thanks again for the review!
While this waits for the additional approval, I’d also be interested in contributing beyond documentation. If there are any beginner-friendly issues in other areas you’d recommend, I’d be happy to take a look.
liamfallon
commented
Feb 19, 2026
Sure, have a look at the "good first issue" labelled issues. We have been triaging issues on our weekly meetings recently and this one came up yesterday. It might be a good one to start with:
SurbhiAgarwal1
commented
Feb 20, 2026
Thanks for the guidance, @liamfallon! I'm happy to help with #1835. I'll head over to that issue now to get context and see what needs to be done.
Docs: tidy up tenant-onboarding guide and link to guides index
This PR addresses the issues identified by @liamfallon in #4393 and
tracked in #4397.
Changes:
guides listing at kpt.dev/guides/
guide is due for a rewrite and may not fully reflect the current state
of kpt
→ /images/tenant-onboarding.svg (the
/static/prefix isincorrect for Hugo's URL resolution)
non-existent "next guide" about scaling to hundreds of tenants.
@mozesl-nokia and agreed by @liamfallon in Fix/docs todo link #4393 )
static/ prefix when serving assets)
Fixes #4397