Skip to content

LiveObjects: merge path-based API into main - #485

Merged
lawrence-forooghian merged 4 commits into
mainfrom
integration/liveobjects-path-based-api-ditch-merge-commits
May 27, 2026
Merged

LiveObjects: merge path-based API into main#485
lawrence-forooghian merged 4 commits into
mainfrom
integration/liveobjects-path-based-api-ditch-merge-commits

Conversation

@lawrence-forooghian

@lawrence-forooghianlawrence-forooghian commented May 27, 2026

Copy link
Copy Markdown
Contributor

This is integration/liveobjects-path-based-api at 8528cdb, rewritten to drop merge commit 2558d8f (of main into the integration branch).

Contents:

VeskeRand others added 4 commits May 27, 2026 17:23
…t()`, rename `RealtimeObjects` -> `RealtimeObject`
Also update the plugins key name. All these to match the changes made in
JS.
Co-Authored-By: Lawrence Forooghian <lawrence.forooghian@ably.com>
This contains the contents of PR #427 (at fdd80da; I'm keeping the full
commit history around in that PR for future reference), which Andrii
originally wrote a few months ago based on the ably-js path-based API
changes (ably/ably-js#2134), and which more recently went through review
and subsequent alterations by Sachin and I.
Co-Authored-By: Andrew Bulat <andrii.bulat@gmail.com>
Co-Authored-By: sacOO7 <sachinshinde7676@gmail.com>
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
[AIT-30] Rename LiveObjects entrypoint and class name from "objects" to "object"
…pec-squashed
[AIT-30] LiveObjects path-based API spec

@paddybyerspaddybyers left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, thanks

@lawrence-forooghian
lawrence-forooghian merged commit b30a71a into mainMay 27, 2026
2 checks passed
@lawrence-forooghian
lawrence-forooghian deleted the integration/liveobjects-path-based-api-ditch-merge-commits branch May 27, 2026 20:35
Sign up for freeto 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.

3 participants

@lawrence-forooghian@paddybyers@VeskeR