Skip to content

Migrate _most_ of the Client Library Development Guide to ably/specification - #1608

Merged
QuintinWillison merged 5 commits into
mainfrom
migrate-client-lib-development-guide
Oct 4, 2022
Merged

Migrate _most_ of the Client Library Development Guide to ably/specification#1608
QuintinWillison merged 5 commits into
mainfrom
migrate-client-lib-development-guide

Conversation

@QuintinWillison

Copy link
Copy Markdown
Contributor

Apologies for the size of this pull request, but it's logical and necessary to perform this content migration atomically.

All 'moved to' (new home) URLs I've added begin with https://sdk.ably.com/builds/ably/specification/main/ however those links will give you a 403 Access Denied until ably/specification#4 has landed to the main branch of the ably/specification repository, which is the source code base that renders the content to HTML and pushes out to sdk.ably.com.

…ks to new homes on sdk.ably.com.

This commit does not move:
- features.textile, which will be done under a dedicated commit
- documentation-formatting-guide.textile, as it is likely to continue living in this repository, perhaps under a different folder
… guide with links to new homes on sdk.ably.com.

Done in VS Code with a regex search and replace:
- Search: `^(\*+) @\((\w+)\)@.+`
- Replace: `$1 @($2)@ Moved to "here on *sdk.ably.com*":https://sdk.ably.com/builds/ably/specification/main/features/#$2`
… library development guide with links to new homes on sdk.ably.com.
Validation of the integrity of the features spec will now take place in its new home.
see: ably/specification#1 (comment)
@QuintinWillison QuintinWillison self-assigned this Oct 1, 2022
@kennethkalmer
kennethkalmer temporarily deployed to ably-docs-migrate-clien-woykp3 October 1, 2022 15:26 Inactive
@QuintinWillison
QuintinWillison marked this pull request as ready for review October 1, 2022 15:30
QuintinWillison added a commit to ably/specification that referenced this pull request Oct 2, 2022
While this file was in the `content/client-lib-development-guide` folder in its original `ably/docs` repository home, its contents falls outside of scope for this `ably/specification` repository.

see: ably/docs#1608

@tbedford tbedford 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.

Epic!

@stmoreau stmoreau 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.

LGTM - nice one!

@QuintinWillison

Copy link
Copy Markdown
Contributor Author

Thanks for the review input so far. ❤️ ... Just a quick reminder that we'll not merge this pull request until after ably/specification#4 has merged.

@m-hulbert m-hulbert 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.

Approving as this all looks good and I have no objections.

Two comments I'd like your opinion on though @QuintinWillison:

  • I think the previous versions and diffs should be preserved, and in my opinion this makes sense to be stored with the latest spec (even if just in raw textile).
  • Any objections if we get redirects set up with the web team in the future to remove these pages from docs entirely? Do we need to preserve the deep links of the features spec for anything?

@QuintinWillison

Copy link
Copy Markdown
Contributor Author

Thanks, @m-hulbert. I have thoughts on both:

I think the previous versions and diffs should be preserved, and in my opinion this makes sense to be stored with the latest spec (even if just in raw textile).

I disagree. The change history has to, logically, be discernible using git tools based on commit history - which I've carried over to the new home. These static snapshots of diffs were always a bad idea, in my opinion, as they duplicate that information and could conceivably drift out of date or be wrong in ways that tools cannot spot. I have, however, created ably/specification#94 to work on at my end to bridge this discoverability gap. Feel free to carry the conversation on in that issue, perhaps expanding on what you would like to see visibility of.

Any objections if we get redirects set up with the web team in the future to remove these pages from docs entirely? Do we need to preserve the deep links of the features spec for anything?

I would love for that to happen and I had presumed that this would be the next logical step, after this pull request has landed. My intention here was to get the 'soft redirects' in place. What I would want to see from future redirects, of course, is that they also carry over the HTML element id anchors too. Some of these documents are quite long and that was one of my goals, to keep that rich, deep linking carrying forwards.

@QuintinWillison
QuintinWillison merged commit 7cbcd89 into main Oct 4, 2022
@QuintinWillison
QuintinWillison deleted the migrate-client-lib-development-guide branch October 4, 2022 11:57
kennethkalmer added a commit that referenced this pull request Dec 6, 2022
* Tutorials should be accessed from ably.com/tutorials
* Client library development guides have moved to sdk.ably.com in #1608
ttypic pushed a commit to ably/specification that referenced this pull request Mar 6, 2026
While this file was in the `content/client-lib-development-guide` folder in its original `ably/docs` repository home, its contents falls outside of scope for this `ably/specification` repository.

see: ably/docs#1608
ttypic pushed a commit to ably/specification that referenced this pull request Mar 6, 2026
While this file was in the `content/client-lib-development-guide` folder in its original `ably/docs` repository home, its contents falls outside of scope for this `ably/specification` repository.

see: ably/docs#1608
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

6 participants