Skip to content

Retain emitter query param when a playground sample is selected#10455

Open
Copilot wants to merge 3 commits intomainfrom
copilot/fix-emitter-query-param
Open

Retain emitter query param when a playground sample is selected#10455
Copilot wants to merge 3 commits intomainfrom
copilot/fix-emitter-query-param

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 22, 2026

  • Identify root cause in packages/playground/src/react/standalone.tsx where emitter was being explicitly cleared when a sampleName was set
  • Remove emitter: undefined from the save logic so the emitter query param is retained alongside ?sample=...
  • Add changelog entry under .chronus/changes
  • Apply prettier formatting

Copilot AI changed the title [WIP] Fix emitter query parameter not saved with pre-populated samples Retain emitter query param when a playground sample is selected Apr 22, 2026
Copilot AI requested a review from JoshLove-msft April 22, 2026 17:53
@JoshLove-msft JoshLove-msft marked this pull request as ready for review April 22, 2026 23:02
@JoshLove-msft JoshLove-msft enabled auto-merge April 22, 2026 23:02
@azure-sdk
Copy link
Copy Markdown
Collaborator

You can try these changes here

🛝 Playground 🌐 Website 🛝 VSCode Extension

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented Apr 22, 2026

Open in StackBlitz

npm i https://pkg.pr.new/@typespec/playground@10455

commit: 74fda8f

@JoshLove-msft
Copy link
Copy Markdown
Contributor

@copilot run prettier

Agent-Logs-Url: https://github.com/microsoft/typespec/sessions/19ad674f-5550-4aa3-babd-064ffe985b7d

Co-authored-by: JoshLove-msft <54595583+JoshLove-msft@users.noreply.github.com>
Copy link
Copy Markdown
Contributor Author

Copilot AI commented Apr 23, 2026

@copilot run prettier

Done in 74fda8f.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Emitter query parameter is not saved when using pre-populated samples

4 participants