Skip to content

[AIT-553] Update to ably-js 2.20.0 in examples and use new subscription fields#3275

Open
VeskeR wants to merge 2 commits intomainfrom
AIT-553/lo-examples-use-2.20.0
Open

[AIT-553] Update to ably-js 2.20.0 in examples and use new subscription fields#3275
VeskeR wants to merge 2 commits intomainfrom
AIT-553/lo-examples-use-2.20.0

Conversation

@VeskeR
Copy link
Contributor

@VeskeR VeskeR commented Mar 13, 2026

Resolves AIT-553

Description

Checklist

@coderabbitai
Copy link

coderabbitai bot commented Mar 13, 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: 5f35cc2b-f0e2-43cf-a20b-bf69a6f19c54

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
  • Post copyable unit tests in a comment
  • Commit unit tests in branch AIT-553/lo-examples-use-2.20.0
📝 Coding Plan
  • Generate coding plan for human review comments

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.

Tip

Migrating from UI to YAML configuration.

Use the @coderabbitai configuration command in a PR comment to get a dump of all your UI settings in YAML format. You can then edit this YAML file and upload it to the root of your repository to configure CodeRabbit programmatically.

@VeskeR VeskeR added the review-app Create a Heroku review app label Mar 13, 2026
@ably-ci ably-ci temporarily deployed to ably-docs-ait-553-lo-ex-gxgi6q March 13, 2026 13:24 Inactive
@VeskeR VeskeR force-pushed the AIT-553/lo-examples-use-2.20.0 branch from decaa20 to ad2944d Compare March 13, 2026 13:24
@VeskeR VeskeR changed the title [AIT-553] Update to ably-js 2.20.0 in examples and use subscription fields [AIT-553] Update to ably-js 2.20.0 in examples and use new subscription fields Mar 13, 2026
@ably-ci ably-ci temporarily deployed to ably-docs-ait-553-lo-ex-gxgi6q March 13, 2026 13:24 Inactive
@VeskeR VeskeR requested review from m-hulbert and removed request for GregHolmes March 13, 2026 13:36
@VeskeR
Copy link
Contributor Author

VeskeR commented Mar 13, 2026

@m-hulbert example page in the preview app https://ably-docs-ait-553-lo-ex-gxgi6q.herokuapp.com/examples/liveobjects-live-map?lang=javascript doesn't seem to use the latest ably-js version specified in this PR (2.20.0) - the protocol version used is still 5 in the preview app (should be 6). Did I do something wrong and need to update something else?

Copy link
Contributor

@lawrence-forooghian lawrence-forooghian left a comment

Choose a reason for hiding this comment

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

LGTM, will leave with you to sort out the build issues

@m-hulbert
Copy link
Contributor

@m-hulbert example page in the preview app https://ably-docs-ait-553-lo-ex-gxgi6q.herokuapp.com/examples/liveobjects-live-map?lang=javascript doesn't seem to use the latest ably-js version specified in this PR (2.20.0) - the protocol version used is still 5 in the preview app (should be 6). Did I do something wrong and need to update something else?

Annoyingly you need to update it in the renderer too - just pushed a fixup that should have sorted it when the review app rebuilds @VeskeR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

review-app Create a Heroku review app

Development

Successfully merging this pull request may close these issues.

4 participants