Skip to content

Update preview banner link to point to PR - #35789

Merged
Cgg merged 2 commits into
masterfrom
clement.geiger/preview-banner-link-to-pr
Jun 12, 2026
Merged

Update preview banner link to point to PR#35789
Cgg merged 2 commits into
masterfrom
clement.geiger/preview-banner-link-to-pr

Conversation

@Cgg

@Cgg Cgg commented Apr 7, 2026

Copy link
Copy Markdown
Contributor

What does this PR do? What is the motivation?

Changes the preview banner link from /tree/<branch> to /pull/<branch>, so clicking it takes reviewers directly to the PR instead of the branch file tree.

QA Instruction

The PR opened in #35790 generated this preview link: https://docs-staging.datadoghq.com/clement.geiger/preview-banner-demo/cloud_cost_management/planning/commitment_programs/

You can see that clicking the link in the banner at the bottom links to the PR, rather than https://github.com/DataDog/documentation/tree/clement.geiger/preview-banner-demo

Screenshot 2026-04-07 at 13 24 40

Merge instructions

Merge readiness:

  • Ready for merge

Additional notes

Single-line change in layouts/partials/preview_banner/preview_banner.html.

🤖 Generated with Claude Code

@Cgg
Cgg requested review from a team as code owners April 7, 2026 10:40
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@Cgg
Cgg force-pushed the clement.geiger/preview-banner-link-to-pr branch from 4546a12 to f70327c Compare April 7, 2026 10:45
@github-actions github-actions Bot added the Architecture Everything related to the Doc backend label Apr 7, 2026
@Cgg Cgg mentioned this pull request Apr 7, 2026
1 task
@Cgg
Cgg merged commit cf98c31 into master Jun 12, 2026
17 checks passed
@Cgg
Cgg deleted the clement.geiger/preview-banner-link-to-pr branch June 12, 2026 14:35
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.

3 participants