Skip to content

Agent redirect skill and small admin > company reorg#24856

Merged
akristen merged 15 commits intodocker:mainfrom
akristen:company-ia
Apr 24, 2026
Merged

Agent redirect skill and small admin > company reorg#24856
akristen merged 15 commits intodocker:mainfrom
akristen:company-ia

Conversation

@akristen
Copy link
Copy Markdown
Contributor

This PR captures a small chunk of work intended to test a new agent skill for handling redirect aliases, link references, and link anchors.

@akristen akristen self-assigned this Apr 22, 2026
@akristen akristen added the area/admin Relates to Docker Admin label Apr 22, 2026
@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 22, 2026

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit 7d47b67
🔍 Latest deploy log https://app.netlify.com/projects/docsdocker/deploys/69ea675d594c3500085f8bc6
😎 Deploy Preview https://deploy-preview-24856--docsdocker.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

@aevesdocker
Copy link
Copy Markdown
Contributor

/review

Copy link
Copy Markdown
Contributor

@docker-agent docker-agent Bot left a comment

Choose a reason for hiding this comment

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

Assessment: 🟡 NEEDS ATTENTION

Comment thread .agents/skills/migrate-content-ia/scripts/scope-pr-files.sh Outdated
Comment thread .agents/skills/migrate-content-ia/SKILL.md Outdated
Comment thread .agents/skills/migrate-content-ia/scripts/scope-pr-files.sh Outdated
The manage/ subfolder move left grid and next-step links on old paths and broke relative links to subscription and organization docs. Add published URL aliases on the new pages, point nav and new-company at the new URLs, and add one ../ segment to relative cross-references.

Made-with: Cursor
Add trailing newline to manage _index and correct 'manageemnt' in front matter keywords.

Made-with: Cursor
Keep the STYLE.md line on a clean line break; avoid a stray ** after 'learn' / split 'learn more'.

Made-with: Cursor
@akristen
Copy link
Copy Markdown
Contributor Author

/review

Copy link
Copy Markdown
Contributor

@docker-agent docker-agent Bot left a comment

Choose a reason for hiding this comment

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

Assessment: 🟢 APPROVE

This PR correctly reorganizes pages under content/manuals/admin/company/manage/ and handles redirects well. All moved pages (organizations.md, owners.md, users.md) include proper aliases: entries covering their old URLs, and internal cross-references are updated throughout.

Two very minor observations (not blocking):

  • content/manuals/admin/company/manage/_index.md: The new section stub is missing description: and keywords: front matter fields. Since render: never is set, there's no SEO impact, but project policy requires these fields on all content pages.
  • content/manuals/admin/company/new-company.md: The title was changed to Create new company, which is missing the article — consider Create a new company to match the first sentence of the page body.

No broken links, missing redirects, AI-isms, or formatting issues detected.

Point internal links and nav URLs to /manuals/admin/activity-logs and /manuals/admin/insights, update relative path in manage-products, and add an alias for the former /admin/organization/activity-logs/ URL.

Made-with: Cursor
@akristen akristen requested review from aevesdocker and removed request for craig-osterhout April 23, 2026 18:45
Copy link
Copy Markdown
Contributor

@aevesdocker aevesdocker left a comment

Choose a reason for hiding this comment

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

Nice! Looks like your skill works! ⭐

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.

question: would you consider grouping activity logs and insights under a 'Monitoring?'

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

That's a rly good idea. I have another PR for this area coming up next week and I'll do that ^

@akristen akristen merged commit 26ea39e into docker:main Apr 24, 2026
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/admin Relates to Docker Admin

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants