Repository navigation
DOC-7128 integrate: convert integrate/ images - #4190
Merged
Merged
Conversation
Runs build/migrate_image_shortcodes.py over 27 files in content/integrate/, including redis-data-integration/, converting all 68 image shortcodes to Markdown images (45 bare paths rewritten to site-root /images/...). None needed skipping, and there are no raw <img> tags in this tree. Two older open PRs touch files here: #3886 (Datadog) and #3489 (RDI restructure, labelled do not merge yet). Test-merging each onto main and onto this branch gives the same conflicts, which they already have with main, so this adds none. Ticket: DOC-7128 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Contributor
Contributor
Contributor
🧠 Redis MemoryFound 5 related items from repository history (5 new this commit):
Memory updated at 8430cd9 |
EliShteinman
added a commit
to EliShteinman/docs
that referenced
this pull request
Oct 3, 2026
Converts the integrate image shortcodes to Markdown images. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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
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.
Summary
Converts the
imageshortcodes incontent/integrate/, includingredis-data-integration/, to Markdown images, using the codemod from #4151. Part of DOC-7128.Counts: all 68 images converted in 27 files. None needed skipping, so
integrate/has no image shortcodes left in its own pages. 45 bare paths (images/...) were rewritten to site-root/images/.... There are no raw<img>tags in this tree.Rendered changes: none, apart from the dropped
<a sdata-lightbox>wrapper.Overlap with open PRs: two older PRs touch files here.
datadog-with-redis-cloud/_index.md.redis-data-integration/architecture/_index.mdanddata-pipelines/_index.md.I test-merged each one onto
mainand onto this branch. Both give the same conflict list: #3886 already conflicts withmainoverrc/accounts/account-settings.md, and #3489 already conflicts in six RDI files. So this PR adds no conflicts for either.Verification
mainand this branch under the production baseURL (/docs/latest/): both succeeded (19,664 pages), and the hook reported no missing images.src/width/alt/classfor every image on every page: no differences.sdata-lightboxwrappers inintegrate/drop by exactly 68 (106 to 38), so every converted image was on a published page and the comparison saw it. This branch leaves no image shortcodes inintegrate/itself. The 38 still rendering come from embeds these pages include, which DOC-7128 embeds-oss: convert embeds/ and oss_and_stack/ images #4189 converts.integrate/: the same 106 absolute image URLs before and after, with no attribute lines and no relative/images/links.🤖 Generated with Claude Code
Note
Low Risk
Documentation-only markup migration with verified equivalent rendering; no application or security logic changes.
Overview
Replaces Hugo
imageshortcodes with standard Markdown images acrosscontent/integrate/(27 files, 68 figures), including Amazon Bedrock guides, observability integrations (Datadog, Dynatrace, New Relic, Prometheus), Railway, RDI/write-behind docs, and versioned 1.19.1 RDI paths.Each conversion uses
with{width="..."}on the following line where widths were set before; relativeimages/...paths become site-root/images/.... Alt text is kept on screenshots that had it; a few dashboard hero images become bare.DOC-7128 / codemod from #4151. Per author verification, rendered output should match aside from removing the
sdata-lightboxwrapper around these figures.Reviewed by Cursor Bugbot for commit 8430cd9. Bugbot is set up for automated code reviews on this repo. Configure here.