Skip to content

FCE-3471 add sandbox moq methods#549

Merged
czerwiukk merged 2 commits into
mainfrom
FCE-3471/sandbox-api-moq-methods
Jun 19, 2026
Merged

FCE-3471 add sandbox moq methods#549
czerwiukk merged 2 commits into
mainfrom
FCE-3471/sandbox-api-moq-methods

Conversation

@czerwiukk

@czerwiukk czerwiukk commented Jun 18, 2026

Copy link
Copy Markdown
Member

Description

Adds sandbox api MoQ methods to the useSandbox hook.

Documentation impact

  • Documentation update required
  • Documentation updated in another PR
  • No documentation update required

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to
    not work as expected)

@czerwiukk czerwiukk requested review from Karolk99 and Copilot June 18, 2026 14:31
@czerwiukk czerwiukk self-assigned this Jun 18, 2026
@linear

linear Bot commented Jun 18, 2026

Copy link
Copy Markdown

FCE-3471

Copilot AI 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.

Pull request overview

Adds MoQ token retrieval helpers to the useSandbox hook so consumers can request publisher/subscriber tokens from the sandbox API.

Changes:

  • Introduces a shared fetchMoqToken(streamName, type) helper for MoQ token fetching.
  • Exposes two new hook methods: getSandboxMoqPublisherToken and getSandboxMoqSubscriberToken.
  • Updates the hook return object to include the new methods.

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

Comment thread packages/react-client/src/hooks/useSandbox.ts Outdated
@czerwiukk czerwiukk merged commit 75d8f9c into main Jun 19, 2026
2 checks passed
@czerwiukk czerwiukk deleted the FCE-3471/sandbox-api-moq-methods branch June 19, 2026 09:26
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.

3 participants