-
Notifications
You must be signed in to change notification settings - Fork 26
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
Check 1.0 docs for accuracy against 0.9 changes #581
Comments
Use (only the Blame view provides the ability to link to a specific line within the source)
|
Done ✅ |
Thanks, please leave this open as we need to update this before we release 1.0. |
@ricardopereira I believe this is all fixed in 155fa73. I'm merging into www.ably.io/documentation now so it would be nice if you could double check the issues you mentioned above and confirm everything is OK. Thanks |
@mattheworiordan Reviewed. I only found this issue ably/docs#285. |
Thanks @ricardopereira |
See ably/docs#264 and 1.0 PR at ably/docs#260
@ricardopereira please can you do a similar task to what Peter did and review all Realtime & REST API docs and provide a list of what needs changing please.
Please note that Peter did not provide a list of links to Github as a reference for method definitions / interfaces etc. Please can you do that so that it's easier for me to jump straight to code when working out what to change.
The text was updated successfully, but these errors were encountered: