[EDU-1590] Update JS API references to match ably-js v2.9 API#2648
[EDU-1590] Update JS API references to match ably-js v2.9 API#2648
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the 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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
de8f17d to
b90fcdf
Compare
|
@m-hulbert There are couple of rendering issues I wasn't able to resolve, would like to get your help with them:
Sreenshots at the bottom. I think all of those issues are caused by the use of language tags, |
b90fcdf to
f289501
Compare
m-hulbert
left a comment
There was a problem hiding this comment.
Thanks for finishing this off @VeskeR
I pushed a fixup which sorts out the rendering issues you found. I've also been through everything else and it all seems to be showing the correct info.
Anything that is slightly awry is down to the current toolchain and format, which will be resolved in the not-too-distant-future. The more important thing is to get the updated info out.
| jsall: enter(Object data?): Promise<void> | ||
|
|
||
| Enter a presence channel and provide data that is associated with the current present member. If the channel is @initialized@ (i.e. no attempt to attach has yet been made for this channel), then calling @enter@ will implicitly attach the channel. | ||
| Enter a presence channel with an optional data that is associated with the current present member. If the channel is @initialized@ (i.e. no attempt to attach has yet been made for this channel), then calling @enter@ will implicitly attach the channel. |
There was a problem hiding this comment.
| Enter a presence channel with an optional data that is associated with the current present member. If the channel is @initialized@ (i.e. no attempt to attach has yet been made for this channel), then calling @enter@ will implicitly attach the channel. | |
| Enter a presence channel with optional data that is associated with the current presence member. If the channel is @initialized@ (i.e. no attempt to attach has yet been made for this channel), then calling @enter@ will implicitly attach the channel. |
There was a problem hiding this comment.
I've removed "an", but I think we should leave the "present member" phrasing, as it is what currently used throughout the whole realtime-sdk/presence.textile file, 21 occurrences
|
|
||
| blang[jsall,ruby,flutter]. | ||
|
|
||
| h3(#device-details). |
There was a problem hiding this comment.
Is this correct for device details and local device to be underneath this blang? I think they exist in more SDKs than just Ruby, Flutter and JS.
There was a problem hiding this comment.
I believe they exists in dotnet, java, obj-c, swift as well, but if they haven't been added in API references before, I would suggest adding them in the separate PR (it's a bit out of scope here)
There was a problem hiding this comment.
I agree with @ttypic here. I've used the existing blang and I don't think we should add changes related to other languages to this PR, as it's quite huge as it is.
It would also require a minimal research to figure out if DeviceDetails and LocalDevice are publicly exposed in other languages, and if so, what fields they have.
I'd prefer we create a separate ticket for this
4f5c96d to
ad4415b
Compare
|
@m-hulbert Hey, thanks for going through the updates and helping with rendering issues. I've noticed that at least locally I can still see Is this one of the things that is hard to fix due to the toolchain/format and you're okay to merge it as is? I don't think I've seen it happen for other partials, so there must be something wrong with my changes I assume. |
c40fb8d to
cca966c
Compare
5d03533 to
dc7ff18
Compare
This includes next changes for the JS API: - update Messages docs - update ClientOptions docs - update Encryption docs - update Stats docs - update Request docs - update Connection docs - update Presence docs - update Authentication docs - update History docs - update Push Device docs - update PushAdmin docs - update PaginatedResult and HttpPaginatedResponse docs - add docs for batch publish and batch presence - various formatting, styling and bug fixes across the board
dc7ff18 to
e6fa9f1
Compare




Description
Based on the work done by @m-hulbert in 521c24f, this PR adds final changes in de8f17d to update all JS API docs to match latest ably-js v2.9 release.
Note:
Message.summary,Annotationmessage,RestAnnotationsandRealtimeAnnotationsare not included in this PR. Should be added once Feature/pub 1676 message annotations #2642 is completed and merged.Resolves EDU-1590
Checklist