Skip to content

fix(extension): update README URL to correct repository path#40574

Open
AndreVale69 wants to merge 2 commits intomicrosoft:mainfrom
AndreVale69:fix/mcp-extension-readme-link
Open

fix(extension): update README URL to correct repository path#40574
AndreVale69 wants to merge 2 commits intomicrosoft:mainfrom
AndreVale69:fix/mcp-extension-readme-link

Conversation

@AndreVale69
Copy link
Copy Markdown
Contributor

The previous URL pointed to a non-existent location. This change ensures users are directed to the correct documentation for version mismatch errors.

Closes #40573

The previous URL pointed to a non-existent location.
This change ensures users are directed to the correct documentation for
version mismatch errors.

Closes microsoft#40573
Comment thread packages/extension/src/ui/connect.tsx Outdated
const VersionMismatchError: React.FC<{ extensionVersion: string }> = ({ extensionVersion }) => {
const readmeUrl = 'https://github.com/microsoft/playwright-mcp/blob/main/packages/extension/README.md';
const readmeUrl = 'https://github.com/microsoft/playwright/blob/main/packages/extension/README.md';
const chromeWebStoreUrl = 'https://chromewebstore.google.com/detail/playwright-mcp-bridge/mmlmfjhmonkocbjadbfplnigmagldckm';
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.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

While you are here, can you also update this link to 'https://chromewebstore.google.com/detail/playwright-extension/mmlmfjhmonkocbjadbfplnigmagldckm' ?

Done. However, I noticed that these links might need replacing too:

export const playwrightExtensionInstallUrl = `https://chromewebstore.google.com/detail/playwright-mcp-bridge/${playwrightExtensionId}`;

Install [Playwright Extension](https://chromewebstore.google.com/detail/playwright-mcp-bridge/mmlmfjhmonkocbjadbfplnigmagldckm) from the Chrome Web Store.

Should I replace them too?

The previous URL was pointing to an incorrect or outdated listing. This
change ensures users are directed to the correct extension page where
they can install or update the Playwright extension compatible with
their client version.

Fixes mismatch error messaging in connect flow.

Refs: microsoft#40573
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.

[Bug]: Broken MCP extension README link in extension UI

2 participants