Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: don't compare agent channel param when determining whether channel options require reattach #1478

Merged
merged 2 commits into from
Nov 1, 2023

Conversation

owenpearson
Copy link
Member

Resolves #1477

Excludes the agent param is excluded when checking whether channel options provided to Channels.get will require re-attachment

@github-actions github-actions bot temporarily deployed to staging/pull/1478/features November 1, 2023 13:31 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/1478/bundle-report November 1, 2023 13:32 Inactive
@owenpearson owenpearson force-pushed the react-channel-options-fix branch from 1651f8d to ad264d9 Compare November 1, 2023 13:33
@github-actions github-actions bot temporarily deployed to staging/pull/1478/features November 1, 2023 13:33 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/1478/bundle-report November 1, 2023 13:35 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/1478/typedoc November 1, 2023 13:35 Inactive
@owenpearson owenpearson force-pushed the react-channel-options-fix branch from ad264d9 to 49b4b6d Compare November 1, 2023 14:51
@github-actions github-actions bot temporarily deployed to staging/pull/1478/features November 1, 2023 14:51 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/1478/bundle-report November 1, 2023 14:53 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/1478/typedoc November 1, 2023 14:53 Inactive
Copy link
Collaborator

@ttypic ttypic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@owenpearson owenpearson merged commit 5857e59 into main Nov 1, 2023
11 checks passed
@owenpearson owenpearson deleted the react-channel-options-fix branch November 1, 2023 17:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1.2.46 breaks useChannel hook
3 participants