Skip to content

Conversation

@ttypic
Copy link
Contributor

@ttypic ttypic commented Jan 21, 2026

Description

Add Mutable messages ably-java

Checklist

@coderabbitai
Copy link

coderabbitai bot commented Jan 21, 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.

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

Note

Other AI code review bot(s) detected

CodeRabbit has detected other AI code review bot(s) in this pull request and will avoid duplicating their findings in the review comments. This may lead to a less comprehensive 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.

@ttypic ttypic requested review from GregHolmes and sacOO7 January 21, 2026 11:34
@mschristensen mschristensen added the review-app Create a Heroku review app label Jan 21, 2026
@ably-ci ably-ci had a problem deploying to ably-docs-ait-304-java--irgvnr January 21, 2026 12:00 Failure
@ttypic ttypic force-pushed the AIT-304/java-mutable-messages-docs branch from 47bbd5b to bcbe1e9 Compare January 21, 2026 17:48
@ably-ci ably-ci had a problem deploying to ably-docs-ait-304-java--irgvnr January 21, 2026 17:49 Failure
@ttypic ttypic force-pushed the AIT-304/java-mutable-messages-docs branch from bcbe1e9 to 708610a Compare January 21, 2026 18:14
@ably-ci ably-ci had a problem deploying to ably-docs-ait-304-java--irgvnr January 21, 2026 18:14 Failure
@ttypic ttypic force-pushed the AIT-304/java-mutable-messages-docs branch from 708610a to c25a14b Compare January 21, 2026 18:15
@ably-ci ably-ci had a problem deploying to ably-docs-ait-304-java--irgvnr January 21, 2026 18:16 Failure
@ttypic ttypic force-pushed the AIT-304/java-mutable-messages-docs branch from c25a14b to 0940436 Compare January 21, 2026 18:19
@ably-ci ably-ci had a problem deploying to ably-docs-ait-304-java--irgvnr January 21, 2026 18:19 Failure
@ttypic ttypic force-pushed the AIT-304/java-mutable-messages-docs branch from 0940436 to 0a7f5c6 Compare January 21, 2026 18:34
@ably-ci ably-ci had a problem deploying to ably-docs-ait-304-java--irgvnr January 21, 2026 18:35 Failure
@ttypic ttypic force-pushed the AIT-304/java-mutable-messages-docs branch from 0a7f5c6 to 43cb932 Compare January 21, 2026 18:50
@ably-ci ably-ci had a problem deploying to ably-docs-ait-304-java--irgvnr January 21, 2026 18:50 Failure
@sacOO7 sacOO7 requested a review from Copilot January 22, 2026 07:41
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds Ably Java documentation for mutable (updatable/deletable/appendable) messages across the “Updates & deletes” guide and the REST/Realtime Channels API reference.

Changes:

  • Adds Java code examples to the updating/deleting/appending and retrieval sections in updates-deletes.mdx.
  • Extends REST and Realtime Channels API reference docs with Java method signatures for mutable-message APIs.
  • Fixes a corrupted Node.js example line in REST Channels docs.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 14 comments.

File Description
src/pages/docs/messages/updates-deletes.mdx Adds Java usage examples for update/delete/append and message retrieval/version APIs.
src/pages/docs/api/rest-sdk/channels.mdx Adds Java signatures/sections for mutable message APIs and fixes a Node.js snippet typo.
src/pages/docs/api/realtime-sdk/channels.mdx Adds Java signatures/sections for mutable message APIs in Realtime Channel docs.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@GregHolmes GregHolmes force-pushed the AIT-304/java-mutable-messages-docs branch from 43cb932 to 2929b17 Compare January 22, 2026 11:09
@GregHolmes GregHolmes temporarily deployed to ably-docs-ait-304-java--irgvnr January 22, 2026 11:10 Inactive
@ttypic ttypic force-pushed the AIT-304/java-mutable-messages-docs branch from 2929b17 to 345d08a Compare January 22, 2026 15:46
@ably-ci ably-ci had a problem deploying to ably-docs-ait-304-java--irgvnr January 22, 2026 15:46 Failure
@ttypic ttypic force-pushed the AIT-304/java-mutable-messages-docs branch from 345d08a to 0895eb0 Compare January 22, 2026 17:53
@ably-ci ably-ci temporarily deployed to ably-docs-ait-304-java--irgvnr January 22, 2026 17:54 Inactive
@ably-ci ably-ci temporarily deployed to ably-docs-ait-304-java--irgvnr January 22, 2026 17:57 Inactive
@ttypic ttypic force-pushed the AIT-304/java-mutable-messages-docs branch from b33f69e to 75335ee Compare January 22, 2026 18:29
@ably-ci ably-ci temporarily deployed to ably-docs-ait-304-java--irgvnr January 22, 2026 18:29 Inactive
Returns a promise which, upon success, will be fulfilled with a [`Message`](/docs/api/realtime-sdk/messages) object representing the latest version of the message. Upon failure, the promise will be rejected with an [`ErrorInfo`](/docs/api/realtime-sdk/types#error-info) object which explains the error.
</If>

<If lang="java">
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
<If lang="java">
<If lang="java,kotlin">

If kotlin snippet is same as java, then we can share them right?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

let's do in other PR, kotlin syntax a bit different

@ttypic ttypic force-pushed the AIT-304/java-mutable-messages-docs branch from 75335ee to 5d72d36 Compare January 23, 2026 13:21
@ably-ci ably-ci temporarily deployed to ably-docs-ait-304-java--irgvnr January 23, 2026 13:21 Inactive
@ttypic ttypic force-pushed the AIT-304/java-mutable-messages-docs branch from 5d72d36 to 96ccdca Compare January 23, 2026 14:28
@ably-ci ably-ci temporarily deployed to ably-docs-ait-304-java--irgvnr January 23, 2026 14:29 Inactive
@ttypic ttypic merged commit ed81110 into main Jan 23, 2026
7 checks passed
@ttypic ttypic deleted the AIT-304/java-mutable-messages-docs branch January 23, 2026 14:41
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.

6 participants