Skip to content

Unifying assets folder - #2521

Merged
brandonpayton merged 14 commits into
trunkfrom
unifying-assets-folder
Aug 20, 2025
Merged

Unifying assets folder#2521
brandonpayton merged 14 commits into
trunkfrom
unifying-assets-folder

Conversation

@fellyph

@fellyph fellyph commented Aug 16, 2025 •
edited
Loading

Copy link
Copy Markdown
Collaborator

Motivation for the change, related issues

This pull request moves the image assets to the static folder to avoid duplication of assets and reduce the errors with missing assets.

Based on issue #2516

Steps to test

  • Check out this pull request branch locally.
  • Run npm install
  • Run npm run build:docs and check if the build has no errors
  • Run npm run dev:docs
  • Open the local site in your browser (http://localhost:3000/wordpress-playground/).
  • Navigate through the documentation, specifically checking pages within the Blueprints and Main sections:
    • "Troubleshoot and Debug Blueprints"
    • "About Playground > Build"
    • "Build your first Blueprints"
    • "Web Instance"
    • "Build your first blueprint"
  • Verify that all images load correctly and are not broken.

@fellyph
fellyph requested review from akirk and brandonpayton August 16, 2025 12:34
brandonpayton
brandonpayton approved these changes Aug 20, 2025
View reviewed changes

@brandonpayton brandonpayton left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I reviewed the content changes and built and looked at a page or two. This looks good to me. Thank you!

@brandonpayton
brandonpayton merged commit 8037056 into trunk Aug 20, 2025
@brandonpayton
brandonpayton deleted the unifying-assets-folder branch August 20, 2025 17:26
@brandonpayton brandonpayton mentioned this pull request Aug 20, 2025
Closed
5 tasks
@shimotmk shimotmk mentioned this pull request Aug 21, 2025
Merged
fellyph pushed a commit that referenced this pull request Aug 22, 2025
@shimotmk
Japanese translations: blueprints Unifying assets folder (#2536)
50911ea 
## Motivation for the change, related issues

This corresponds to the image paths integrated in the PR #2521 . 
The assets folder is no longer used, so it will be deleted.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@brandonpayton brandonpayton brandonpayton approved these changes
@akirk akirk Awaiting requested review from akirk

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@fellyph @brandonpayton