Skip to content

Conversation

@GregHolmes
Copy link
Contributor

Description

This PR converts the REST SDK Constructor API reference page from Textile format to MDX, as part of a full textile to MDX migration.

@GregHolmes GregHolmes self-assigned this Nov 3, 2025
@coderabbitai
Copy link

coderabbitai bot commented Nov 3, 2025

Important

Review skipped

Auto reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch EDU-2112-Convert-Pub-Sub-REST-SDK-API-refs-Constructor-to-MDX

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@GregHolmes GregHolmes force-pushed the EDU-2112-Convert-Pub-Sub-REST-SDK-API-refs-Constructor-to-MDX branch from c909e08 to 8bb51d0 Compare November 3, 2025 15:38
@GregHolmes GregHolmes added the review-app Create a Heroku review app label Nov 3, 2025
@ably-ci ably-ci temporarily deployed to ably-docs-edu-2112-conv-cy7rc0 November 3, 2025 15:40 Inactive
@GregHolmes GregHolmes changed the title Migrate REST SDK API spec Constructor page to MDX [EDU-2112] - Migrate REST SDK API spec Constructor page to MDX Nov 3, 2025
@GregHolmes GregHolmes temporarily deployed to ably-docs-edu-2112-conv-fpog2w November 4, 2025 09:29 Inactive
@franrob-projects franrob-projects temporarily deployed to ably-docs-edu-2112-conv-zjhmad November 4, 2025 13:23 Inactive
Copy link
Contributor

@franrob-projects franrob-projects left a comment

Choose a reason for hiding this comment

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

Just needs PHP and Ruby adding to the table's commented :)

Links are all good


<If lang="csharp,flutter,java,objc,swift,python,go">

| Parameter | Description | Type |
Copy link
Contributor

Choose a reason for hiding this comment

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

php n ruby seems to have been dropped. So the tables vanished. (the orgignal has one for PHP)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks Fran, good spot! I've fixed this.

@GregHolmes GregHolmes force-pushed the EDU-2112-Convert-Pub-Sub-REST-SDK-API-refs-Constructor-to-MDX branch from 8bb51d0 to 877680a Compare November 5, 2025 12:41
@GregHolmes GregHolmes temporarily deployed to ably-docs-edu-2112-conv-zjhmad November 5, 2025 12:42 Inactive
@GregHolmes GregHolmes force-pushed the EDU-2112-Convert-Pub-Sub-REST-SDK-API-refs-Constructor-to-MDX branch from 4051fb7 to feda7ee Compare November 5, 2025 12:45
@GregHolmes GregHolmes temporarily deployed to ably-docs-edu-2112-conv-zjhmad November 5, 2025 12:46 Inactive
Copy link
Contributor

@franrob-projects franrob-projects left a comment

Choose a reason for hiding this comment

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

Perfecto

@GregHolmes GregHolmes temporarily deployed to ably-docs-edu-2112-conv-jb3thr November 5, 2025 14:03 Inactive
@GregHolmes GregHolmes merged commit 33cde63 into main Nov 5, 2025
6 checks passed
@GregHolmes GregHolmes deleted the EDU-2112-Convert-Pub-Sub-REST-SDK-API-refs-Constructor-to-MDX branch November 5, 2025 14:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

review-app Create a Heroku review app

Development

Successfully merging this pull request may close these issues.

4 participants