Skip to content

porch docs navbar redirect - #4611

Merged
liamfallon merged 1 commit into
kptdev:mainfrom
Nordix:docs-porch-navbar-redirect
Jul 1, 2026
Merged

porch docs navbar redirect#4611
liamfallon merged 1 commit into
kptdev:mainfrom
Nordix:docs-porch-navbar-redirect

Conversation

@Catalin-Stratulat-Ericsson

@Catalin-Stratulat-Ericsson Catalin-Stratulat-Ericsson commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Description: the porch navbar menu will redirect to the porch documentation directly rather than to the porch main menu.

Motivation: its the natural expectation to be redirected to the porch docs not the porch header page.

No AI was used / required.

Linked with kptdev/porch#1087 & kptdev/krm-functions-catalog#1270

Signed-off-by: Catalin-Stratulat-Ericsson <catalin.stratulat@ericsson.com>
Copilot AI review requested due to automatic review settings July 1, 2026 13:19
@dosubot dosubot Bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Jul 1, 2026
@netlify

netlify Bot commented Jul 1, 2026

Copy link
Copy Markdown

Deploy Preview for kptdocs ready!

Name Link
🔨 Latest commit 3c7cece
🔍 Latest deploy log https://app.netlify.com/projects/kptdocs/deploys/6a4513ec05179b00089c251d
😎 Deploy Preview https://deploy-preview-4611--kptdocs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@dosubot dosubot Bot added area/porch documentation Improvements or additions to documentation labels Jul 1, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the docs site navbar so the “Porch” menu item links directly to the Porch documentation landing page instead of the Porch home/header page, aligning navigation with the expected user destination.

Changes:

  • Changed the “Porch” navbar link from https://porch.kpt.dev/ to https://porch.kpt.dev/docs/.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@dosubot dosubot Bot added the lgtm label Jul 1, 2026
@liamfallon
liamfallon merged commit e793f7b into kptdev:main Jul 1, 2026
6 of 9 checks passed
OisinJohnston2005 pushed a commit to Nordix/kpt that referenced this pull request Jul 3, 2026
Signed-off-by: Catalin-Stratulat-Ericsson <catalin.stratulat@ericsson.com>
Signed-off-by: Oisin Johnston <oisin.johnston@ericsson.com>
efiacor pushed a commit that referenced this pull request Jul 9, 2026
* Allowed the default prefix to be specified by env variable or cobra flag

Signed-off-by: Oisin Johnston <oisin.johnston@ericsson.com>

* Added image-prefix as a cli option or env variable

Signed-off-by: Oisin Johnston <oisin.johnston@ericsson.com>

* It works this time

Signed-off-by: Oisin Johnston <oisin.johnston@ericsson.com>

* Moved to a cleaner implementation and made the same logic apply to eval as well as render

Signed-off-by: Oisin Johnston <oisin.johnston@ericsson.com>

* cleaned her up a bit

Signed-off-by: Oisin Johnston <oisin.johnston@ericsson.com>

* cleaned up duplicate code and removed redundant code

Signed-off-by: Oisin Johnston <oisin.johnston@ericsson.com>

* Adding rules to CONTRIBUTING.md (#4594)

* Adding rules to CONTRIBUTING.md

- Current CODEOWNER.md based rules for pr approvals.
  This will be changed ot the maintainer and contributor groups based rules
  once we agreed to apply them to all the repos.
- Rules for stale prs

Signed-off-by: Gergely Csatari <gergely.csatari@nokia.com>

* Apply suggestions from code review

Signed-off-by: Gergely Csatari <gergely.csatari@nokia.com>
Assisted-by: GitHub Copilot: Claude Haiku 4.5

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

---------

Signed-off-by: Gergely Csatari <gergely.csatari@nokia.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Oisin Johnston <oisin.johnston@ericsson.com>

* Fix basicpipeline-semver e2e expected output for set-labels v0.2.5 (#4605)

Signed-off-by: Aravindhan Ayyanathan <aravindhan.a@est.tech>
Signed-off-by: Oisin Johnston <oisin.johnston@ericsson.com>

* docs: update Nephio documentation link in FAQ

Signed-off-by: Oisin Johnston <oisin.johnston@ericsson.com>

* First section of chapter 3 proofreading.

Signed-off-by: Michael Greaves <michael.greaves@nokia.com>
Signed-off-by: Oisin Johnston <oisin.johnston@ericsson.com>

* Chapter 3 proofreading.

Signed-off-by: Michael Greaves <michael.greaves@nokia.com>
Signed-off-by: Oisin Johnston <oisin.johnston@ericsson.com>

* Fixing indentations in chapter 3.

Signed-off-by: Michael Greaves <michael.greaves@nokia.com>
Signed-off-by: Oisin Johnston <oisin.johnston@ericsson.com>

* Corrections done in chapter 3.

Signed-off-by: Michael Greaves <michael.greaves@nokia.com>
Signed-off-by: Oisin Johnston <oisin.johnston@ericsson.com>

* Apply suggestions from code review

Signed-off-by: Gergely Csatari <gergely.csatari@nokia.com>

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Oisin Johnston <oisin.johnston@ericsson.com>

* Apply suggestions from code review

Signed-off-by: Gergely Csatari <gergely.csatari@nokia.com>

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Oisin Johnston <oisin.johnston@ericsson.com>

* Apply suggestions from code review

Signed-off-by: Gergely Csatari <gergely.csatari@nokia.com>

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Oisin Johnston <oisin.johnston@ericsson.com>

* Breaking a long line

Signed-off-by: Gergely Csatari <gergely.csatari@nokia.com>
Signed-off-by: Oisin Johnston <oisin.johnston@ericsson.com>

* Fixing WordPress wording

Signed-off-by: Gergely Csatari <gergely.csatari@nokia.com>
Assisted-by: GitHub Copilot: Claude Haiku 4.5
Signed-off-by: Oisin Johnston <oisin.johnston@ericsson.com>

* Apply suggestion from @Catalin-Stratulat-Ericsson

Co-authored-by: Catalin Stratulat <159934629+Catalin-Stratulat-Ericsson@users.noreply.github.com>
Signed-off-by: Oisin Johnston <oisin.johnston@ericsson.com>

* porch docs navbar redirect (#4611)

Signed-off-by: Catalin-Stratulat-Ericsson <catalin.stratulat@ericsson.com>
Signed-off-by: Oisin Johnston <oisin.johnston@ericsson.com>

* security - fix Git argument injection via untrusted Kptfile ref enables file overwrite (#4604)

Signed-off-by: tar <gabor.tar.ext@nokia.com>
Signed-off-by: Oisin Johnston <oisin.johnston@ericsson.com>

* Specifying allowed alert types (#4612)

* Specifying allowed alert types

Signed-off-by: Dominika Schweier <dominika.schweier@nokia.com>

* Specifying allowed alert types

Signed-off-by: Dominika Schweier <dominika.schweier@nokia.com>

* Apply suggestions from code review

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

---------

Signed-off-by: Dominika Schweier <dominika.schweier@nokia.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Oisin Johnston <oisin.johnston@ericsson.com>

* Simplify CI workflows: remove legacy GOPATH checkout pattern (#4597)

Signed-off-by: Aravindhan Ayyanathan <aravindhan.a@est.tech>
Signed-off-by: Oisin Johnston <oisin.johnston@ericsson.com>

* Fixed copilot comments

Signed-off-by: Oisin Johnston <oisin.johnston@ericsson.com>

* mention the environment variable in the help for the --image-prefix flag

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Oisin Johnston <oisin.johnston@ericsson.com>

* Fixed copilot comments

Signed-off-by: Oisin Johnston <oisin.johnston@ericsson.com>

* Fixed copilot comments

Signed-off-by: Oisin Johnston <oisin.johnston@ericsson.com>

* Added prefix validation

Signed-off-by: Oisin Johnston <oisin.johnston@ericsson.com>

* added unit tests and tightened validation requirements

Signed-off-by: Oisin Johnston <oisin.johnston@ericsson.com>

---------

Signed-off-by: Oisin Johnston <oisin.johnston@ericsson.com>
Signed-off-by: Gergely Csatari <gergely.csatari@nokia.com>
Signed-off-by: Aravindhan Ayyanathan <aravindhan.a@est.tech>
Signed-off-by: Michael Greaves <michael.greaves@nokia.com>
Signed-off-by: Catalin-Stratulat-Ericsson <catalin.stratulat@ericsson.com>
Signed-off-by: tar <gabor.tar.ext@nokia.com>
Signed-off-by: Dominika Schweier <dominika.schweier@nokia.com>
Co-authored-by: Gergely Csatari <gergely.csatari@nokia.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Aravindhan Ayyanathan <aravindhan.a@est.tech>
Co-authored-by: dosubot[bot] <131922026+dosubot[bot]@users.noreply.github.com>
Co-authored-by: Michael Greaves <michael.greaves@nokia.com>
Co-authored-by: Catalin Stratulat <159934629+Catalin-Stratulat-Ericsson@users.noreply.github.com>
Co-authored-by: Tar Gábor Zsolt <gabor.tar.ext@nokia.com>
Co-authored-by: Schweier Dominika <dominika.schweier@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/porch documentation Improvements or additions to documentation lgtm size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants