Skip to content

docs(pusher): re-add dropped server library, interoperability, and count details#3393

Merged
SimonWoolf merged 1 commit into
mainfrom
pusher-adapter-readd-dropped-info
Jun 4, 2026
Merged

docs(pusher): re-add dropped server library, interoperability, and count details#3393
SimonWoolf merged 1 commit into
mainfrom
pusher-adapter-readd-dropped-info

Conversation

@SimonWoolf
Copy link
Copy Markdown
Member

@SimonWoolf SimonWoolf commented Jun 3, 2026

Re-adds some information to the Pusher Adapter page that used to be available on the old support article (3000055086 — Using the Ably Pusher protocol adaptor) but was removed in the transition to the current docs page.

What's re-added

  • Server-side (REST) Pusher library configuration — the page advertised the REST features as supported but never showed how to configure a serverside library. Adds a Ruby example with the host, app_id/key/secret split, and TLS option, plus an options table.
  • Interoperability note — Pusher and Ably client libraries can be used side by side against the same channels (e.g. publish/subscribe with Pusher, fetch history via the Ably REST API).
  • User and subscriber count behaviour — both only work on presence channels, with the distinction that user count is uniqued by clientId while subscriber count is the raw presence-set size.

🤖 Generated with Claude Code

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Jun 3, 2026

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: b45f558e-ed54-4d9a-9572-6d7c285d844d

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch pusher-adapter-readd-dropped-info

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.

@SimonWoolf SimonWoolf requested a review from m-hulbert June 3, 2026 10:50
Copy link
Copy Markdown
Contributor

@m-hulbert m-hulbert left a comment

Choose a reason for hiding this comment

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

Just a few small suggested tidy ups.

Comment thread src/pages/docs/protocols/pusher.mdx Outdated
Comment thread src/pages/docs/protocols/pusher.mdx Outdated
Comment thread src/pages/docs/protocols/pusher.mdx Outdated
Comment thread src/pages/docs/protocols/pusher.mdx Outdated
Restore information from the old Pusher adapter FAQ that was dropped in
the transition to the current docs page:

- Server-side (REST) Pusher library configuration, including host,
  app_id/key/secret split, and TLS option
- Note that Pusher and Ably client libraries can be used side by side
  against the same channels
- User and subscriber count behaviour (presence-channel only, and the
  distinction between the two)

Previously documented at:
https://support.ably.io/support/solutions/articles/3000055086-using-the-ably-pusher-protocol-adaptor/

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@SimonWoolf SimonWoolf force-pushed the pusher-adapter-readd-dropped-info branch from d753926 to 5bfd794 Compare June 4, 2026 13:28
@SimonWoolf
Copy link
Copy Markdown
Member Author

Applied all suggested changes

@SimonWoolf SimonWoolf enabled auto-merge June 4, 2026 13:29
@SimonWoolf SimonWoolf merged commit ca1ec69 into main Jun 4, 2026
7 checks passed
@SimonWoolf SimonWoolf deleted the pusher-adapter-readd-dropped-info branch June 4, 2026 13:40
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.

2 participants