Skip to content

core: decode roomName doAppStart - #1038

Merged
sarahrose-ferris merged 1 commit into
mainfrom
sarahrose/PAN-2678-roomurl-decoding
Jul 8, 2026
Merged

core: decode roomName doAppStart#1038
sarahrose-ferris merged 1 commit into
mainfrom
sarahrose/PAN-2678-roomurl-decoding

Conversation

@sarahrose-ferris

@sarahrose-ferris sarahrose-ferris commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

Description

Summary:

In order to support rooms with special characters in the name, we need to do some decoding on roomUrl. This prevents clients trying to join rooms with the encoded URL, which technically is a non-existing room.

Related Issue:

Testing

Test instructions here: https://github.com/whereby/recording-service/pull/1096

Screenshots/GIFs (if applicable)

Checklist

  • My code follows the project's coding standards.
  • Prefixed the PR title and commit messages with the service or package name
  • I have written unit tests (if applicable).
  • I have updated the documentation (if applicable).
  • By submitting this pull request, I confirm that my contribution is made
    under the terms of the MIT license.

Additional Information

@changeset-bot

changeset-bot Bot commented Jun 16, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 6be9bc2

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 4 packages
Name Type
@whereby.com/core Minor
@whereby.com/assistant-sdk Patch
@whereby.com/browser-sdk Patch
@whereby.com/react-native-sdk Patch

Not sure what this means? Click here to learn what changesets are.

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

@sarahrose-ferris
sarahrose-ferris force-pushed the sarahrose/PAN-2678-roomurl-decoding branch from e7d8dc3 to 5aa954e Compare June 16, 2026 11:29
@sarahrose-ferris

Copy link
Copy Markdown
Contributor Author

/canary

@github-actions

Copy link
Copy Markdown
Contributor

🚀 The canary releases have been published to npm.

You can test the releases by installing the newly published versions:

pnpm add @whereby.com/assistant-sdk@0.0.0-canary-20260616113857
pnpm add @whereby.com/browser-sdk@0.0.0-canary-20260616113857
pnpm add @whereby.com/core@0.0.0-canary-20260616113857
pnpm add @whereby.com/react-native-sdk@0.0.0-canary-20260616113857

@sarahrose-ferris
sarahrose-ferris force-pushed the sarahrose/PAN-2678-roomurl-decoding branch from 5aa954e to f7a8b71 Compare June 16, 2026 11:56
@sarahrose-ferris

Copy link
Copy Markdown
Contributor Author

/canary

@github-actions

Copy link
Copy Markdown
Contributor

🚀 The canary releases have been published to npm.

You can test the releases by installing the newly published versions:

pnpm add @whereby.com/assistant-sdk@0.0.0-canary-20260616120149
pnpm add @whereby.com/browser-sdk@0.0.0-canary-20260616120149
pnpm add @whereby.com/core@0.0.0-canary-20260616120149
pnpm add @whereby.com/react-native-sdk@0.0.0-canary-20260616120149

@sarahrose-ferris sarahrose-ferris changed the title core: decode roomUrl in joinRoom core: decode roomUrl in joinRoom and doAppStart Jun 16, 2026
@sarahrose-ferris
sarahrose-ferris force-pushed the sarahrose/PAN-2678-roomurl-decoding branch from f7a8b71 to 666c68b Compare June 17, 2026 09:36
@sarahrose-ferris sarahrose-ferris changed the title core: decode roomUrl in joinRoom and doAppStart core: decode roomUrl in doAppStart Jun 17, 2026
@sarahrose-ferris

Copy link
Copy Markdown
Contributor Author

/canary

@github-actions

Copy link
Copy Markdown
Contributor

🚀 The canary releases have been published to npm.

You can test the releases by installing the newly published versions:

pnpm add @whereby.com/assistant-sdk@0.0.0-canary-20260617094509
pnpm add @whereby.com/browser-sdk@0.0.0-canary-20260617094509
pnpm add @whereby.com/core@0.0.0-canary-20260617094509
pnpm add @whereby.com/react-native-sdk@0.0.0-canary-20260617094509

@sarahrose-ferris
sarahrose-ferris requested a review from a team June 17, 2026 14:36

@richtrwhereby richtrwhereby 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.

LGTM 🚀

@sarahrose-ferris sarahrose-ferris changed the title core: decode roomUrl in doAppStart core: decode roomName on join_room Jun 18, 2026
@sarahrose-ferris
sarahrose-ferris force-pushed the sarahrose/PAN-2678-roomurl-decoding branch from 666c68b to 2d0c47e Compare June 18, 2026 12:19
@sarahrose-ferris

Copy link
Copy Markdown
Contributor Author

/canary

@github-actions

Copy link
Copy Markdown
Contributor

🚀 The canary releases have been published to npm.

You can test the releases by installing the newly published versions:

pnpm add @whereby.com/assistant-sdk@0.0.0-canary-20260618122331
pnpm add @whereby.com/browser-sdk@0.0.0-canary-20260618122331
pnpm add @whereby.com/core@0.0.0-canary-20260618122331
pnpm add @whereby.com/react-native-sdk@0.0.0-canary-20260618122331

@sarahrose-ferris
sarahrose-ferris force-pushed the sarahrose/PAN-2678-roomurl-decoding branch from 2d0c47e to 6fd4160 Compare June 18, 2026 12:56

@richtrwhereby richtrwhereby 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.

LGTM 💯

Comment thread packages/core/src/redux/slices/roomConnection/index.ts
@sarahrose-ferris
sarahrose-ferris force-pushed the sarahrose/PAN-2678-roomurl-decoding branch from 6fd4160 to 6be9bc2 Compare June 23, 2026 14:53
@sarahrose-ferris sarahrose-ferris changed the title core: decode roomName on join_room core: decode roomName doAppStart Jun 23, 2026
@sarahrose-ferris

Copy link
Copy Markdown
Contributor Author

/canary

@github-actions

Copy link
Copy Markdown
Contributor

🚀 The canary releases have been published to npm.

You can test the releases by installing the newly published versions:

pnpm add @whereby.com/assistant-sdk@0.0.0-canary-20260623145732
pnpm add @whereby.com/browser-sdk@0.0.0-canary-20260623145732
pnpm add @whereby.com/core@0.0.0-canary-20260623145732
pnpm add @whereby.com/react-native-sdk@0.0.0-canary-20260623145732

@sarahrose-ferris
sarahrose-ferris merged commit 42407a0 into main Jul 8, 2026
3 checks passed
@sarahrose-ferris
sarahrose-ferris deleted the sarahrose/PAN-2678-roomurl-decoding branch July 8, 2026 12:35
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