Conversation
✅ Deploy Preview for docsdocker ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
…ts in their respective home directories
…ce it contained two topics; moved domain management into provision since sso has its own instructions in setup
|
/review |
|
❌ PR Review Failed — The review agent encountered an error and could not complete the review. View logs. |
content/manuals/enterprise/security/single-sign-on/FAQs/troubleshoot-sso.md
Outdated
Show resolved
Hide resolved
|
/review |
![docker-agent[bot]](https://avatars.githubusercontent.com/in/2787757?s=60&v=4){kind=small}
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
|
/review |
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
|
/review |
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
Use Hugo source paths with .md or _index.md for cross-references from admin overview, organization pages, and company users. Align SSO, SCIM, roles, enforce sign-in, Settings Management, domains, billing paths, and related targets with content/manuals layout. Made-with: Cursor
Point billing FAQs, payment method, 3D Secure, and tax certificate cross-links to Hugo source paths under content/manuals/billing and subscription. Made-with: Cursor
Point Create a Docker ID to content/manuals/accounts/create-account.md. Made-with: Cursor
Use /manuals/admin/company/_index.md for Hugo ref resolution. Made-with: Cursor
Use /manuals/... and /reference/api/hub/latest.md paths for Hugo ref resolution on JIT, provision-scim, and migrate-scim. Made-with: Cursor
Use /manuals/ paths for security announcements, access tokens, JIT, FAQs, and troubleshoot; /reference/api/hub/latest.md for Hub API; repair security-announcements URL typo and Markdown link punctuation in troubleshoot-sso. Made-with: Cursor
Point domain and SSO connect anchors to /manuals/...; align CLI reference links with /reference/cli/...md forms used elsewhere in manuals. Made-with: Cursor
|
/review |
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
Point SCIM section links at scim/_index.md instead of nonexistent scim.md. Route supported-attributes anchor to provision-scim.md. Fix group mapping link in admin user management guide to use /manuals/... path. Made-with: Cursor
|
/review |
There was a problem hiding this comment.
Assessment: 🟢 APPROVE
This PR reorganizes SCIM and provisioning documentation with no critical issues found.
Changes reviewed:
- 42 files changed (primarily SCIM/JIT provisioning reorganization)
- File moves:
troubleshoot-provisioning.mdandtroubleshoot-sso.mdmoved into their respective feature directories - Deletion:
enterprise/troubleshoot/_index.md(empty directory index) - Link updates throughout to reflect new file locations
- Front matter formatting improvements (consistent YAML indentation)
Verification results:
All internal links verified and working correctly. The reorganization properly updates link targets to match the new file structure.
| - /security/for-admins/provisioning/group-mapping/ | ||
| weight: 30 | ||
| - /security/for-admins/provisioning/scim/group-mapping/ | ||
| weight: 20 |
There was a problem hiding this comment.
Missing redirect alias for moved file
The file group-mapping.md was moved from content/manuals/enterprise/security/provisioning/ to content/manuals/enterprise/security/provisioning/scim/.
After Hugo strips the /manuals prefix:
- Old published URL:
/enterprise/security/provisioning/group-mapping/ - New published URL:
/enterprise/security/provisioning/scim/group-mapping/
The aliases list in the front matter includes legacy redirects but is missing a redirect from the immediate previous location. Users visiting the old URL will get a 404.
Fix: Add this alias to the front matter:
aliases:
- /admin/company/settings/group-mapping/
- /admin/organization/security-settings/group-mapping/
- /docker-hub/group-mapping/
- /security/for-admins/group-mapping/
- /security/for-admins/provisioning/scim/group-mapping/
- /enterprise/security/provisioning/group-mapping/ # Add this line2 participants
just a lot of stuff that got out of hand idk. it started out as a nice reorg of provision and turned into a nightmare of my own making