Skip to content

docs: update SSE fetchStream subscription example - #15166

Closed
v0eak wants to merge 1 commit into
medusajs:developfrom
v0eak:docs
Closed

docs: update SSE fetchStream subscription example#15166
v0eak wants to merge 1 commit into
medusajs:developfrom
v0eak:docs

Conversation

@v0eak

@v0eak v0eak commented Apr 22, 2026

Copy link
Copy Markdown
Contributor

Summary

What — What changes are introduced in this PR?
Example on how to receive SSE events using JS fetchStream updated to reflect refactor

Why — Why are these changes relevant or necessary?
Reflect Refactor

Additional Context

Intended to be merged after: #15134

@v0eak
v0eak requested a review from a team as a code owner April 22, 2026 07:34
@changeset-bot

changeset-bot Bot commented Apr 22, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 07a06d4

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@vercel

vercel Bot commented Apr 22, 2026

Copy link
Copy Markdown

@v0eak is attempting to deploy a commit to the medusajs Team on Vercel.

A member of the Team first needs to authorize it.

@medusa-os-bot

medusa-os-bot Bot commented Apr 22, 2026

Copy link
Copy Markdown

Thank you for your contribution, @v0eak!

Could you please fill in the PR description following our pull request template? We need the What, Why, How, and Testing sections completed so we can review your PR efficiently.

Once you've updated the description, another review will be triggered automatically.

Note: This PR removes the if (\!stream) null check based on the refactor in PR #15134, which is still open. Please clarify in the description whether this PR is intended to be merged after #15134, so the team can coordinate the merge order and avoid shipping documentation that describes behavior not yet in the codebase.

Thanks!

Triggered by: new PR opened

@medusa-os-bot

medusa-os-bot Bot commented Apr 22, 2026

Copy link
Copy Markdown

Thank you for your contribution, @v0eak!

After reviewing the updated description, we're still missing a couple of sections before we can move forward:

Required changes:

  • Fill in the How and Testing sections of the PR description — the previous review requested all four sections of our pull request template, and these two sections are still absent.

Once you've updated the description, another review will be triggered automatically.

Thanks!

Triggered by: PR description updated

@github-actions

Copy link
Copy Markdown
Contributor

This PR is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 5 days.

@github-actions github-actions Bot added the Stale label May 14, 2026
@github-actions

Copy link
Copy Markdown
Contributor

This PR was closed because it has been stalled for 5 days with no activity.

@github-actions github-actions Bot closed this May 24, 2026
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.

1 participant