Skip to content

Remove migrated AAP compatibility pages and repoint links#37654

Open
eliottness wants to merge 1 commit into
masterfrom
eliottness/cleanup-appsec-setup-compat
Open

Remove migrated AAP compatibility pages and repoint links#37654
eliottness wants to merge 1 commit into
masterfrom
eliottness/cleanup-appsec-setup-compat

Conversation

@eliottness

Copy link
Copy Markdown
Contributor

What does this PR do? What is the motivation?

The per-language App and API Protection (AAP) compatibility pages and the cross-language compatibility index under security/application_security/setup/ are now generated from the compatibility matrix and injected at build time. This PR removes the leftover in-repo copies and repoints all internal links to the consolidated /security/application_security/setup/compatibility/<lang> pages, matching the existing Go setup.

  • Removed setup/compatibility/_index.md and setup/{dotnet,java,nodejs,php,python,ruby}/compatibility.md
  • Repointed 47 references (41 content links + 6 shortcodes) from setup/<lang>/compatibility to setup/compatibility/<lang>
  • Corrected a copy-paste link on the nginx setup page that pointed to the PHP compatibility page

Merge instructions

Merge readiness:

  • Ready for merge

Additional notes

Translated content (content/es, content/ko) is managed externally and is not modified here.

The per-language compatibility pages and the cross-language compatibility
index under security/application_security/setup/ are now generated from the
compatibility matrix and injected at build time.

Remove the leftover in-repo copies (old setup/compatibility/_index.md and
setup/{dotnet,java,nodejs,php,python,ruby}/compatibility.md) and repoint all
internal links to the consolidated /setup/compatibility/<lang> pages, matching
the existing Go setup. Also correct a copy-paste link on the nginx setup page
that pointed to the PHP compatibility page.
@eliottness eliottness requested a review from a team as a code owner June 22, 2026 15:20
@github-actions github-actions Bot added the Architecture Everything related to the Doc backend label Jun 22, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Preview links (active after the build_preview check completes)

Removed or renamed files (these should redirect)

Modified Files

@datadog-official

Copy link
Copy Markdown
Contributor

Pipelines

Fix all issues with BitsAI

⚠️ Warnings

🚦 1 Pipeline job failed

DataDog/documentation | build_preview   View in Datadog   GitLab

Useful? React with 👍 / 👎

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: deca9a4 | Docs | Datadog PR Page | Give us feedback!

@rtrieu rtrieu self-requested a review June 22, 2026 18:34
@rtrieu

rtrieu commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

hi @eliottness thanks for this PR! it mostly looks good, but you will need to add aliases for the deleted pages so that users navigating to those pages won't 404. for example, /security/application_security/_index.md should have /security/application_security/threats/setup/compatibility/. content/en/security/application_security/setup/dotnet/docker.md should have /security/application_security/setup/dotnet/compatibility, etc. can you add the missing aliases and then request a re-review when ready? thanks!

@rtrieu rtrieu 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.

missing aliases

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Architecture Everything related to the Doc backend

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants