Skip to content

feat: Web SDK update for version 22.1.0#154

Merged
loks0n merged 1 commit intomainfrom
dev
Feb 13, 2026
Merged

feat: Web SDK update for version 22.1.0#154
loks0n merged 1 commit intomainfrom
dev

Conversation

@ArnabChatterjee20k
Copy link
Contributor

@ArnabChatterjee20k ArnabChatterjee20k commented Feb 13, 2026

This PR contains updates to the Web SDK for version 22.1.0.

Summary by CodeRabbit

  • Chores
    • Added a new static accessor method to the Channel class for memberships.

@coderabbitai
Copy link

coderabbitai bot commented Feb 13, 2026

Walkthrough

The pull request adds a new static method memberships() to the Channel class in src/channel.ts that returns a string constant. This method follows the existing pattern of the teams() static accessor. However, the change introduces duplicate declarations of the same static memberships(): string method within the same class, resulting in conflicting definitions.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 2
❌ Failed checks (1 warning, 1 inconclusive)
Check name Status Explanation Resolution
Merge Conflict Detection ⚠️ Warning ❌ Merge conflicts detected (1 file):

⚔️ src/channel.ts (content)

These conflicts must be resolved before merging into main.
Resolve conflicts locally and push changes to this branch.
Title check ❓ Inconclusive The title is vague and generic, using 'update for version' without specifying what substantive changes the PR introduces. The summary reveals duplicate method declarations in Channel class, but the title doesn't reflect this specific technical issue. Consider a more descriptive title that reflects the actual changes, such as 'fix: Remove duplicate memberships() method in Channel class' or 'chore: Web SDK 22.1.0 update with duplicate method fix'.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch dev
⚔️ Resolve merge conflicts (beta)
  • Auto-commit resolved conflicts to branch dev
  • Create stacked PR with resolved conflicts
  • Post resolved changes as copyable diffs in a comment

No actionable comments were generated in the recent review. 🎉


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@loks0n loks0n merged commit 13a9dc0 into main Feb 13, 2026
2 checks passed
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