Skip to content

docs: update mcp_calculator README to reflect Pong game sample transi… - #2156

Merged
sugoi-yuzuru merged 2 commits into
mainfrom
readme_webframe
Aug 3, 2026
Merged

docs: update mcp_calculator README to reflect Pong game sample transi…#2156
sugoi-yuzuru merged 2 commits into
mainfrom
readme_webframe

Conversation

@sugoi-yuzuru

Copy link
Copy Markdown
Collaborator

…tion and summoning methods

Description

Update the readme to focus on helping developers test the Pong game example in its 3 different forms: MCP App, Iframe via URL, Iframe via srcdoc

Pre-launch Checklist

One time:

For this PR:

  • I have updated the relevant CHANGELOG.md file.
  • I updated/added relevant documentation.
  • My code changes (if any) have tests.
  • If my branch is on a fork, I have verified that scripts/e2e_test.sh passes.

If you need help, consider asking for advice on the discussion board.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request updates the README.md file for the Angular client sample, transitioning it from describing the MCP Calculator to the Pong game sample. The updated documentation outlines the prerequisites, setup, and various summoning methods for the Pong game. The review feedback suggests improving the clarity of the setup instructions by explicitly adding directory navigation commands (cd) inside the code blocks for installing dependencies and starting the Angular application.

Comment thread samples/community/client/angular/projects/mcp_calculator/README.md
Comment thread samples/community/client/angular/projects/mcp_calculator/README.md
@sugoi-yuzuru
sugoi-yuzuru merged commit 2c3d0b9 into main Aug 3, 2026
24 checks passed
@sugoi-yuzuru
sugoi-yuzuru deleted the readme_webframe branch August 3, 2026 16:06
@github-project-automation github-project-automation Bot moved this from Todo to Done in A2UI Aug 3, 2026
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