404 errors for missing assets/scripts when running yarn run dev

[swissspidy]

First of all, kudos to this new setup. I just tried yarn build followed by yarn run dev and it's very fast! Very nice experience.

What caught my eye though when opening the playground site is errors about some seemingly missing files:

[adamziel]

Thank you for the report! It looks like it can’t find some static files from wp-6.1 directory. The wp-includes dir is here:

https://github.com/WordPress/wordpress-playground/tree/trunk/packages/playground/remote/public/wp-6.1/wp-includes

But the dist directory is nowhere to be found. I bet that’s because it got excluded because of the dist rule in .gitignore.

You can restore these files by rebuilding WordPress:

nx recompile-wordpress:all playground-remote

I’m traveling to Rust now and can’t push a fix, but if you’d start a PR then I would merge it!

[swissspidy]

Makes sense. That doesn't explain the first error with the missing SVG file though, as that one exists: https://github.com/WordPress/wordpress-playground/blob/b60ea1625ecec159cac0e9585e1edebdd786751b/packages/playground/remote/public/wp-6.1/wp-admin/images/about-header-about.svg

---

Aside: seems weird to have full copies of all WP versions in the repo. Maybe those should be git submodules or separate packages or something?

[adamziel]

The SVG gets removed somewhere in the WP build process, unfortunately, it would be good to retain it.

The dist/ issue is fixed in 70e5aa4.

As for the full copies – I agree it's weird. They're here for now to reduce the initial setup hurdle but I'd like to offer that benefit without those files in the repo.

[adamziel]

This is now resolved, and the issue related to keeping the full copies of source files is tracked separately in #58