-
Notifications
You must be signed in to change notification settings - Fork 324
Closed
Labels
[Type] EnhancementNew feature or requestNew feature or request[Type] ReliabilityPlayground uptime, reliability, not crashingPlayground uptime, reliability, not crashing
Description
This is an attempt to develop a strategy for consuming Playground.
- Using the latest
@wp-playground/client
package from unpkg.com might break the existing sites when the methods change. - Using a fixed version might break the existing sites if the remote API changes.
- Versioning the remote.html API seems like an overly complex maintenance burden.
Let's just have Playground ship its own API client that's compatible with the current version of Playground. It still doesn't address breaking API changes, but that is a separate discussion involving defining stable and unstable APIs.
This will involve an adjustment to the playground.wordpress.net build pipeline:
wordpress-playground/packages/playground/website/project.json
Lines 12 to 25 in 6f4f269
"build:wasm-wordpress-net": { | |
"executor": "nx:run-commands", | |
"options": { | |
"commands": [ | |
"rimraf --no-preserve-root ./wasm-wordpress-net", | |
"mkdir ./wasm-wordpress-net", | |
"cp -r ./blueprints/blueprint-schema.json ./wasm-wordpress-net/", | |
"cp -r ./remote/* ./wasm-wordpress-net/", | |
"cp -r ./website/* ./wasm-wordpress-net/", | |
"cat ./remote/.htaccess > ./wasm-wordpress-net/.htaccess" | |
], | |
"cwd": "dist/packages/playground", | |
"parallel": false | |
}, |
Done is:
- Running
nx build:wasm-wordpress-net playground-website
yields a build where theclient.js
library is exposed - Any references to
client.js
coming from unpkg in this repo are updated to talk abouthttps://playground.wordpress.net/client.js
instead.
Metadata
Metadata
Assignees
Labels
[Type] EnhancementNew feature or requestNew feature or request[Type] ReliabilityPlayground uptime, reliability, not crashingPlayground uptime, reliability, not crashing