Fix Azure getting started page: UI rendering and remove unsupported Terraform step - #39727
Open
perzycharles wants to merge 1 commit into
Open
Fix Azure getting started page: UI rendering and remove unsupported Terraform step#39727perzycharles wants to merge 1 commit into
perzycharles wants to merge 1 commit into
Conversation
…rraform step - Escape + in ui shortcode to prevent markdownify treating it as a list bullet - Remove Configure log collection step from Terraform setup (not supported) Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Contributor
Preview links (active after the
|
Contributor
Author
|
/review |
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
🤖 Automated review by Claude. AI-generated; verify before acting.
Small formatting/scope change in hugo/content/en/getting_started/integrations/azure.md. No blockers found — the \+ escape aligns with the pattern used elsewhere in the repo (e.g., content/*/actions/app_builder/variables.md) for the same UI shortcode, and the removed Terraform log-collection step appears to be an intentional scope change.
Reviewed 18d162e378b65406689b0a44013da37952660711 — workflow run
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What does this PR do? What is the motivation?
Two fixes to the Azure getting started page (
/getting_started/integrations/azure).Live page: https://docs.datadoghq.com/getting_started/integrations/azure/?tab=createanappregistration#terraform-setup
1. UI rendering fix
The numbered list instructions in the Quickstart and Terraform tabs were broken. The
{{< ui >}}shortcode pipes its content throughmarkdownify; when the content starts with+(e.g.,+ Add New App registration), the+is treated as a markdown list bullet, which wraps the text in<ul><li>and breaks the surrounding<ol>. All subsequent numbered steps were swallowed into the bullet.Screenshots of the broken UI:
Fix: Escape the
+as\+in both the Quickstart and Terraform tab instructions.2. Remove unsupported Terraform step
Removed the "Configure log collection" step (step 6) from the Terraform "Create an app registration" tab — this flow is not supported for the Terraform setup method. The orphaned
[102]link reference was also cleaned up.Merge readiness
For Datadog employees:
<name>/<description>convention and include the forward slash (/). If you've already created your PR with an incorrect branch name, please rename your branch and open a fresh PR./reviewto run an automated check that catches common issues before a Documentation team member reviews your PR.AI assistance
Changes drafted with Claude Code assistance.
Additional notes