Skip to content
This repository was archived by the owner on Aug 10, 2026. It is now read-only.

DOC-367: Update quickstart JSBins - #1166

Merged
m-hulbert merged 4 commits into
fea-1041-update-quickstartfrom
doc-367-update-quickstart-jsbins
Aug 6, 2021
Merged

m-hulbert merged 4 commits into
fea-1041-update-quickstartfrom
doc-367-update-quickstart-jsbins

Conversation

@m-hulbert

@m-hulbert m-hulbert commented Aug 6, 2021 •

Copy link
Copy Markdown
Contributor

NOTE TO REVIEWERS - please do not review PRs in the DRAFT state, as the PR may change substantially before it is ready to review. Thanks.

Description

This PR updates the existing JSBins for the quickstart to modern JavaScript. It also adds a new JSBin for closing a connection to Ably. See JIRA for further information.

Note: I have raised this JIRA to look at removing JQuery from JSBin as a future improvement.

Review

The quickstart should be checked to ensure the new JSBins are correctly linked.

The new JSBins can be checked from that page or individually using:

@m-hulbert m-hulbert added the DRAFT Draft PR - do not review label Aug 6, 2021
@m-hulbert m-hulbert self-assigned this Aug 6, 2021
@mattheworiordan
mattheworiordan temporarily deployed to ably-docs-pr-1166 August 6, 2021 12:06 Inactive
@m-hulbert
m-hulbert requested a review from marklewin August 6, 2021 12:11

@marklewin marklewin left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@m-hulbert and I discussed the desirability of moving away from JQuery in all JS Bin code (DOC-367), but in the meantime this LGTM!

@m-hulbert m-hulbert removed the DRAFT Draft PR - do not review label Aug 6, 2021
@m-hulbert
m-hulbert marked this pull request as ready for review August 6, 2021 16:30
@m-hulbert
m-hulbert merged commit 033c25b into fea-1041-update-quickstart Aug 6, 2021
@m-hulbert
m-hulbert deleted the doc-367-update-quickstart-jsbins branch August 6, 2021 16:30

This branch was previously deployed

1 inactive deployment
ably-docs-pr-1166 — aa5cfa0c Deployed Aug 6, 2021 by mattheworiordan
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants