Skip to content

Update moq api#85

Merged
Karolk99 merged 3 commits into
mainfrom
fce-3479/update-moq-api
Jun 23, 2026
Merged

Update moq api#85
Karolk99 merged 3 commits into
mainfrom
fce-3479/update-moq-api

Conversation

@Karolk99

Copy link
Copy Markdown
Collaborator

@Karolk99 Karolk99 requested a review from Copilot June 22, 2026 22:42
@linear

linear Bot commented Jun 22, 2026

Copy link
Copy Markdown

FCE-3479

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

Updates the SDK’s MoQ token endpoint integration to match a server-side API change where the response now provides a relay connection URL (with the JWT embedded) instead of a raw JWT token.

Changes:

  • Bump package version from 0.28.0 to 0.28.1 (and update uv.lock accordingly).
  • Update FishjamClient.create_moq_token() to return response.url and adjust its docstring to describe the URL return value.
  • Update the OpenAPI model MoqToken to use a url field instead of token.

Reviewed changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 2 comments.

File Description
uv.lock Updates the locked editable package version to 0.28.1.
pyproject.toml Bumps the SDK package version to 0.28.1.
fishjam/api/_fishjam_client.py Changes MoQ helper to return a relay connection URL instead of a JWT token.
fishjam/_openapi_client/models/moq_token.py Renames the response field from token to url and updates serialization.

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

Comment thread fishjam/api/_fishjam_client.py Outdated
Comment thread pyproject.toml
@Karolk99 Karolk99 requested review from czerwiukk and roznawsk June 23, 2026 10:16
@Karolk99 Karolk99 merged commit fdd5570 into main Jun 23, 2026
11 checks passed
@Karolk99 Karolk99 deleted the fce-3479/update-moq-api branch June 23, 2026 11:25
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.

4 participants