-
Notifications
You must be signed in to change notification settings - Fork 7.8k
fix: replace imgur domains with official proxy to fix UK access issue… #8213
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
|
Hi @zebra0307! Thank you for your pull request and welcome to our community. Action RequiredIn order to merge any pull request (code, docs, etc.), we require contributors to sign our Contributor License Agreement, and we don't seem to have one on file for you. ProcessIn order for us to review and merge your suggested changes, please sign at https://code.facebook.com/cla. If you are contributing on behalf of someone else (eg your employer), the individual CLA may not be sufficient and your employer may need to sign the corporate CLA. Once the CLA is signed, our tooling will perform checks and validations. Afterwards, the pull request will be tagged with If you have received this in error or have any questions, please contact us at [email protected]. Thanks! |
Size changesDetails📦 Next.js Bundle Analysis for react-devThis analysis was generated by the Next.js Bundle Analysis action. 🤖 This PR introduced no changes to the JavaScript bundle! 🙌 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR updates all Imgur image URLs across the React documentation to use the official proxy domain i.imgur.com.prx.react.dev to resolve UK regional access issues. The changes ensure reliable image loading for all users by routing through a documentation-managed proxy while preserving all original image IDs and size parameters.
- Replaces
i.imgur.comwithi.imgur.com.prx.react.devacross all documentation files - Updates both hardcoded URLs and helper utility functions
- Maintains thumbnail size logic and file extensions
Reviewed changes
Copilot reviewed 18 out of 18 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| src/content/reference/react/useState.md | Updated image URLs in state management examples to use proxy domain |
| src/content/reference/react-dom/components/common.md | Updated avatar image URL in common component examples to use proxy domain |
| src/content/learn/your-first-component.md | Updated profile image URLs throughout component introduction tutorial to use proxy domain |
| src/content/learn/writing-markup-with-jsx.md | Updated Hedy Lamarr image URLs in JSX markup examples to use proxy domain |
| src/content/learn/updating-objects-in-state.md | Updated Blue Nana sculpture image URLs in object state examples to use proxy domain |
| src/content/learn/state-a-components-memory.md | Updated all sculpture list image URLs (12 artworks, 6 instances each) to use proxy domain |
| src/content/learn/rendering-lists.md | Updated getImageUrl helper function (6 instances) to construct URLs with proxy domain |
| src/content/learn/render-and-commit.md | Updated Floralis Genérica flower sculpture image URLs to use proxy domain |
| src/content/learn/reacting-to-input-with-state.md | Updated Rainbow houses image URLs in state interaction examples to use proxy domain |
| src/content/learn/preserving-and-resetting-state.md | Updated colorful building image URLs (7 locations, 2 instances each) to use proxy domain |
| src/content/learn/passing-props-to-a-component.md | Updated getImageUrl helper function (8 instances) and avatar image URLs to use proxy domain |
| src/content/learn/passing-data-deeply-with-context.md | Updated getImageUrl helper function (2 instances) for place images to use proxy domain |
| src/content/learn/keeping-components-pure.md | Updated getImageUrl helper function (2 instances) to use proxy domain |
| src/content/learn/javascript-in-jsx-with-curly-braces.md | Updated avatar URLs, getImageUrl helper, and explanatory text mentioning URLs to use proxy domain |
| src/content/learn/index.md | Updated user avatar image URL in landing page example to use proxy domain |
| src/content/learn/importing-and-exporting-components.md | Updated profile image URLs in import/export tutorial to use proxy domain |
| src/content/learn/describing-the-ui.md | Updated profile, avatar image URLs and getImageUrl helper functions to use proxy domain |
| src/content/learn/adding-interactivity.md | Updated sculpture list URLs (12 artworks) and state form examples to use proxy domain |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Meta Open Source project. Thanks! |
This PR redirects all Imgur-hosted assets to the official React documentation proxy: i.imgur.com.prx.react.dev.
Using this proxy ensures:
tags.
1.Reliability: Images are served through a domain managed for the documentation, bypassing regional blocks on the primary Imgur domain.
2.Zero-Change to Content: We do not need to manually re-host thousands of assets. The proxy handles the routing while keeping the original image IDs intact.
3.Consistency: This fix covers all 18 major tutorial and reference files, updating both helper functions like getImageUrl() and any hardcoded
Changes:
Updated i.imgur.com to i.imgur.com.prx.react.dev in all MDX files under src/content/learn/ and src/content/reference/.
Fixed helper utility functions in Sandpack interactive examples to use the proxy URL.
Verified that thumbnail size logic (e.g., s.jpg, b.jpg) remains functional through the proxy.