Skip to content

Simplify personal WP app installer#3623

Merged
akirk merged 7 commits into
WordPress:trunkfrom
akirk:personal-wp-app-launcher-sidebar
May 14, 2026
Merged

Simplify personal WP app installer#3623
akirk merged 7 commits into
WordPress:trunkfrom
akirk:personal-wp-app-launcher-sidebar

Conversation

@akirk
Copy link
Copy Markdown
Member

@akirk akirk commented May 13, 2026

Screenshot 2026-05-14 at 05 00 23

Summary

  • Replace the personal-wp app catalog fetch with a single App Launcher install link → this allows previous My WordPress users to migrate to the App launcher as well as install the newest version.
  • Encode the App Launcher install blueprint inline as a data URL
  • Add a UI assertion for the App Launcher blueprint link

Testing

  • npm exec -- tsc -p packages/playground/personal-wp/tsconfig.spec.json --noEmit
  • npm exec -- tsc -p packages/playground/personal-wp/tsconfig.app.json --noEmit
  • ESLINT_USE_FLAT_CONFIG=false npm exec -- eslint "packages/playground/personal-wp/**/*.{ts,tsx,js,jsx}" --max-warnings=0
  • npm exec -- prettier --check packages/playground/personal-wp/src/components/site-manager/site-info-panel/index.tsx packages/playground/personal-wp/playwright/e2e/ui.spec.ts

Note: nx target wrappers failed in the local sandbox before running tasks because Nx plugin workers could not start; direct equivalent commands passed.

@akirk akirk requested review from a team, ashfame and Copilot May 13, 2026 08:05
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

This PR simplifies the Personal WP “Install Apps” flow by removing the remote app catalog/custom app paste UI and replacing it with a single “App Launcher” install link backed by an inline data-URL blueprint.

Changes:

  • Removed remote app catalog fetching + custom app paste/remove UI in favor of a single App Launcher link
  • Added an inline App Launcher blueprint encoded as a data URL
  • Added a Playwright assertion that validates the App Launcher link includes an encoded blueprint URL

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
packages/playground/personal-wp/src/components/site-manager/site-info-panel/index.tsx Replaces apps list/custom apps with a single App Launcher install link backed by an inline blueprint data URL
packages/playground/personal-wp/playwright/e2e/ui.spec.ts Adds an E2E assertion verifying the App Launcher link contains an encoded blueprint URL

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@akirk akirk merged commit 6ed3bc7 into WordPress:trunk May 14, 2026
53 checks passed
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