-
Notifications
You must be signed in to change notification settings - Fork 42
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add LiveObjects product to site navigation #2464
base: PUB-773/liveobjects-product-docs
Are you sure you want to change the base?
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:
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 (
|
9ac84e8
to
e5e6ece
Compare
3bfe8d2
to
145457e
Compare
e5e6ece
to
48b36e8
Compare
145457e
to
5d95e7f
Compare
48b36e8
to
c284b79
Compare
5d95e7f
to
1f7289e
Compare
c284b79
to
4c70358
Compare
1f7289e
to
2f870cd
Compare
4c70358
to
8b99a80
Compare
title: 'LiveObjects', | ||
type: 'feature', | ||
content: 'Seamlessly sync application state between clients.', | ||
image: 'liveobjects.png', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this needs to be added
src/data/languages/languageData.ts
Outdated
@@ -31,6 +31,9 @@ export default { | |||
javascript: 0.4, | |||
react: 0.4, | |||
}, | |||
liveObjects: { | |||
javascript: 2.6, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
not sure about this one. LiveObjects is not a separate product, so it doesn't have its own versioning.
will it inherit the pub/sub SDKs version then? e.g. 2.6 for ably-js
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes I think we should just specify the Ably JS version
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok, works for me.
Also, technically, it wouldn't be 2.6
, as with the liveobjects release we will bump the minor version in ably-js to at least 2.7
(or more if other versions released before that).
@ably/team-deved would you guys prefer:
- this PR to have
2.6
version in this PR, and then open a second PR to update the ably-js versions for all products (not just liveobjects) to higher version once it's published - keep this PR open until ably-js is published and change the version here to the actual one, as well as for other products?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Assuming that LiveObjects is the only change going into 2.7
of ably-js then I think we can just bump both of them to 2.7
in this PR.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sgtm, I've preemptively updated it to 2.7
for everything, but will change it again if we end up releasing liveobjects under another minor ably-js version
Looks good, we just need some assets from the design team - I'll follow up with Jamie |
7c9bad3
to
25b2298
Compare
f674eaa
to
28b40a7
Compare
I've removed the Setup page as we decided to drop it in favor of Quickstart: #2463 (comment) |
src/data/languages/languageData.ts
Outdated
@@ -31,6 +31,9 @@ export default { | |||
javascript: 0.4, | |||
react: 0.4, | |||
}, | |||
liveObjects: { | |||
javascript: 2.6, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Assuming that LiveObjects is the only change going into 2.7
of ably-js then I think we can just bump both of them to 2.7
in this PR.
name: 'Batch Operations', | ||
link: '/docs/liveobjects/batch', | ||
}, | ||
{ | ||
name: 'Lifecycle Events', | ||
link: '/docs/liveobjects/lifecycle', | ||
}, | ||
{ | ||
name: 'Typing', | ||
link: '/docs/liveobjects/typing', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As per my review on the main PR; I wonder if the features should remain only LiveCounter and LiveMap, and then these 3 items fall under a separate heading. Any thoughts?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
then these 3 items fall under a separate heading. Any thoughts?
Not against putting them under a separate heading. Honestly, they're under LiveObjects features heading right now because I couldn't think of another collective name for these three.
Any ideas?
25b2298
to
453be49
Compare
28b40a7
to
266f785
Compare
70e3d74
to
0d1c04b
Compare
266f785
to
1fc1b12
Compare
This was added by searching for "livesync" in "src" folder and adding corresponding "liveObjects" sections where necessary.
1fc1b12
to
20ad800
Compare
Description
Adds navigation for LiveObjects product in
src
folder.This was added by searching for "livesync" in "src" folder and adding corresponding "liveObjects" sections where necessary.
Currently missing:
Checklist