Skip to content

fix: pki approvals doc references#5258

Merged
carlosmonastyrski merged 1 commit intomainfrom
fix/approval-docs
Jan 23, 2026
Merged

fix: pki approvals doc references#5258
carlosmonastyrski merged 1 commit intomainfrom
fix/approval-docs

Conversation

@carlosmonastyrski
Copy link
Contributor

Context

Fix approval references to reflect the updated file path after the recent change.

Screenshots

Steps to verify the change

Type

  • Fix
  • Feature
  • Improvement
  • Breaking
  • Docs
  • Chore

Checklist

  • Title follows the conventional commit format: type(scope): short description (scope is optional, e.g., fix: prevent crash on sync or fix(api): handle null response).
  • Tested locally
  • Updated docs (if needed)
  • Read the contributing guide

@maidul98
Copy link
Collaborator

Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@carlosmonastyrski carlosmonastyrski changed the title fix: PKI approvals doc references fix: pki approvals doc references Jan 23, 2026
@greptile-apps
Copy link
Contributor

greptile-apps bot commented Jan 23, 2026

Greptile Overview

Greptile Summary

Updated documentation links to reflect the new file path for PKI approval policies. The approval policies documentation was moved from /documentation/platform/pki/certificates/approvals to /documentation/platform/pki/approvals, and this PR corrects all internal references across 3 documentation files.

  • Fixed link in certificates overview page
  • Fixed link in certificate lifecycle concepts page
  • Fixed two links in certificate approvals guide, including an anchor reference

All links now correctly point to the new location at docs/documentation/platform/pki/approvals.mdx. The anchor reference #guide-to-creating-an-approval-policy is valid and exists in the target file.

Confidence Score: 5/5

  • This PR is safe to merge with no risk
  • This is a straightforward documentation fix that updates broken internal links. All references have been correctly updated to point to the new file location, the target file exists, and the anchor reference is valid. No code changes, no security implications, and no breaking changes.
  • No files require special attention

Important Files Changed

Filename Overview
docs/documentation/platform/pki/certificates/overview.mdx Updated approval policy link from /certificates/approvals to /approvals to reflect new file location
docs/documentation/platform/pki/concepts/certificate-lifecycle.mdx Updated approval policy link from /certificates/approvals to /approvals to reflect new file location
docs/documentation/platform/pki/guides/certificate-approvals.mdx Updated approval workflow references from /certificates/approvals to /approvals, including anchor link

Copy link
Contributor

@scott-ray-wilson scott-ray-wilson left a comment

Choose a reason for hiding this comment

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

LGTM, verified each link

@carlosmonastyrski carlosmonastyrski merged commit 6f1a3af into main Jan 23, 2026
8 of 9 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.

3 participants