Conversation
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
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. Comment Tip Migrating from UI to YAML configuration.Use the |
splindsay-92
left a comment
There was a problem hiding this comment.
Just a few things so far, but mostly looks good!
| | Property | Required | Description | Type | | ||
| | --- | --- | --- | --- | | ||
| | name | Required | The name of the room. | String | | ||
| | options | Optional | Options to use when creating the room. Must be memoized to prevent unnecessary room recreations. Room options are immutable after creation; differing options for the same room name cause errors. | <Table id='RoomOptions'/> | |
There was a problem hiding this comment.
| | options | Optional | Options to use when creating the room. Must be memoized to prevent unnecessary room recreations. Room options are immutable after creation; differing options for the same room name cause errors. | <Table id='RoomOptions'/> | | |
| | options | Optional | Options to use when creating the room. Must be memoized to prevent unnecessary room recreations. Room options are immutable after creation; differing options for the same room name will reject with a `RoomExistsWithDifferentOptions` error. | <Table id='RoomOptions'/> | |
| | --- | --- | --- | --- | | ||
| | name | Required | The name of the room. | String | | ||
| | options | Optional | Options to use when creating the room. Must be memoized to prevent unnecessary room recreations. Room options are immutable after creation; differing options for the same room name cause errors. | <Table id='RoomOptions'/> | | ||
| | children | Optional | Child components that will have access to the room context. | ReactNode or ReactNode[] | |
There was a problem hiding this comment.
| | children | Optional | Child components that will have access to the room context. | ReactNode or ReactNode[] | | |
| | children | Optional | Child components that will have access to the room context. | ReactNode, ReactNode[] or null | |
|
|
||
| | Property | Required | Description | Type | | ||
| | --- | --- | --- | --- | | ||
| | enableEvents | Optional | Whether to receive occupancy events. Enabling this increases message volume as the server sends additional updates for occupancy changes. Default false. | Boolean | |
There was a problem hiding this comment.
Can we highlight the default values so they are clear please?
| | code | Ably-specific error code. | Number | | ||
| | statusCode | HTTP status code corresponding to this error, where applicable. | Number | | ||
| | message | Additional information about the error. | String | | ||
| | cause | The underlying cause of the error, where applicable. | String, ErrorInfo, or Error | |
There was a problem hiding this comment.
This is now incorrect, ably-js lists this as an errorInfo only I believe, chat is likely using the same type now. I'll double check and update the codebase if I've not already done so :)
|
|
||
| </Table> | ||
|
|
||
| <Table id='MessageReactionSummaryEvent' hidden> |
There was a problem hiding this comment.
I think the 'clipped' flag is missing from these summary event types.
| | updateMessage | Updates a message's content. | [updateMessage()](#update) | | ||
| | deleteMessage | Soft-deletes a message. | [deleteMessage()](#delete) | | ||
| | history | Retrieves historical messages with pagination. | [history()](#history) | | ||
| | historyBeforeSubscribe | Retrieves messages sent before the subscription was established. Only available when a `listener` is provided. | [historyBeforeSubscribe()](#historyBeforeSubscribe) | |
There was a problem hiding this comment.
| | historyBeforeSubscribe | Retrieves messages sent before the subscription was established. Only available when a `listener` is provided. | [historyBeforeSubscribe()](#historyBeforeSubscribe) | | |
| | historyBeforeSubscribe | Retrieves messages sent before the subscription was established, up to the point of attachment. Only available when a `listener` is provided. | [historyBeforeSubscribe()](#historyBeforeSubscribe) | |
Description
This PR adds an API reference for Chat in React using the new NestedTable and MethodSignature components. It should have parity with the TypeDoc references.
There is 1 commit per file, however I haven't yet updated the references in the conceptual content to point to these references yet to keep this PR smaller for review feedback. I will add that as a final commit once we are happy with this content.
Checklist