Skip to content

Conversation

@bridgeyuwa
Copy link

What is the reason for this PR?

Updated the default image placeholder URL in the Image provider from https://via.placeholder.com (which is no longer available) to https://placehold.co. This change ensures that Faker's imageUrl() method returns valid, working placeholder image URLs. It fixes broken image links and maintains compatibility for users relying on Faker-generated images.

Updated the default image placeholder URL in the Image provider from https://via.placeholder.com (which is no longer available) to https://placehold.co. This change ensures that Faker's imageUrl() method returns valid, working placeholder image URLs. It fixes broken image links and maintains compatibility for users relying on Faker-generated images.
@pimjansen
Copy link

Failed tests

@bridgeyuwa
Copy link
Author

Failed tests

Oh no, I replaced https://via.placeholder.com/ with https://placehold.co/ and didn’t write a full test suite, integration plan, and ISO certification report. Truly reckless. I’ll get started on building an entire QA department just to verify this image URL switch.

@pimjansen pimjansen closed this Aug 9, 2025
@bridgeyuwa bridgeyuwa deleted the 1.24 branch September 11, 2025 03:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants