Skip to content

docs: group Terraform drift tutorials under a subsection#307

Merged
mbevc1 merged 2 commits into
mainfrom
mintlify/ec30df29
Jul 3, 2026
Merged

docs: group Terraform drift tutorials under a subsection#307
mbevc1 merged 2 commits into
mainfrom
mintlify/ec30df29

Conversation

@mintlify

@mintlify mintlify Bot commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Groups the two drift-detection tutorials under a new Detecting Terraform drift subsection inside the Security tutorials group:

  • tutorials/detecting_unexpected_statefile_changes
  • tutorials/detecting_non_terraform_changes

tutorials/rotating_api_keys stays at the top level of Security.

Page review

Both pages are in good shape — mirrored structure (intro → mechanism → prerequisites → kosli-dev/tf setup → hardening → checklist → related), symmetric cross-links, and each explicitly names the drift type the other page covers. No content changes needed.


Generated by Mintlify Agent.

Requested by: marko@kosli.com via Slack
Mintlify session: slack_1783096596.150999_C0AJTQ5633K

@mintlify mintlify Bot requested a review from a team as a code owner July 3, 2026 16:38
@claude

claude Bot commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Claude finished @mintlify[bot]'s task in 1m 7s —— View job


PR Review

  • Gather context — read diff, changed files, and CLAUDE.md
  • Review navigation changes in config/navigation.json
  • Verify deleted file tutorials/unauthorized_iac_changes.md
  • Check for broken links or navigation issues
  • Post review findings

Overall: This PR looks good. Two clean, low-risk changes — a nav restructure and a file deletion — with no issues found.

config/navigation.json

The two Terraform drift tutorials are correctly nested under a new "Detecting Terraform drift" sub-group inside "Security", and tutorials/rotating_api_keys stays at the top level. All three referenced pages (detecting_unexpected_statefile_changes, detecting_non_terraform_changes, rotating_api_keys) exist as .mdx files. Structure and JSON syntax are valid. No issues.

tutorials/unauthorized_iac_changes.md (deleted)

  • The page was already absent from config/navigation.json on main, so this deletion is cleanup of an orphaned file.
  • A redirect from /tutorials/unauthorized_iac_changes/tutorials/detecting_unexpected_statefile_changes already exists in config/redirects.json (line 7), so any existing links or bookmarks will still resolve.
  • No other .md/.mdx files reference unauthorized_iac_changes.
  • The images referenced in the deleted page (authorized-iac-change.png, unauthorized-iac-change.png) don't exist in the repo, so no orphaned assets are left behind.

No critical issues, improvements, or suggestions.

@mbevc1 mbevc1 merged commit 05acae2 into main Jul 3, 2026
6 checks passed
@mbevc1 mbevc1 deleted the mintlify/ec30df29 branch July 3, 2026 16:47
@mintlify

mintlify Bot commented Jul 3, 2026

Copy link
Copy Markdown
Contributor Author

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
kosli 🔴 Failed Jul 3, 2026, 4:52 PM

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.

1 participant