Skip to content

docs: improve clarity for typescript + sdk#664

Merged
ComfortablyCoding merged 5 commits into
directus:mainfrom
kheiner:docs/improve-language-for-typescript-types
Jun 5, 2026
Merged

docs: improve clarity for typescript + sdk#664
ComfortablyCoding merged 5 commits into
directus:mainfrom
kheiner:docs/improve-language-for-typescript-types

Conversation

@kheiner
Copy link
Copy Markdown
Contributor

@kheiner kheiner commented Apr 25, 2026

Update the language in callout to better represent linked content.

I'm open to suggestions on how to best do this, but I'm of the belief that the SDK Documentation favors using JS and should probably be written to assume that the developers will be using TypeScript. This is a small change that I'm fine with the scope increasing if someone on the directus team wants help with re-orienting the preference for TypeScript as a preferred implementation.

@kheiner kheiner requested a review from a team as a code owner April 25, 2026 18:39
@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 25, 2026

@kheiner is attempting to deploy a commit to the Directus Team on Vercel.

A member of the Team first needs to authorize it.

Update the language in callout to better represent linked content.
@kheiner kheiner force-pushed the docs/improve-language-for-typescript-types branch from 1bab86c to 7a46300 Compare May 13, 2026 16:51
Comment thread content/guides/04.connect/6.sdk.md Outdated
Copy link
Copy Markdown
Member

@ComfortablyCoding ComfortablyCoding left a comment

Choose a reason for hiding this comment

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

LGTM agreed and thanks!

@ComfortablyCoding ComfortablyCoding merged commit 33c6446 into directus:main Jun 5, 2026
3 of 4 checks passed
@kheiner kheiner deleted the docs/improve-language-for-typescript-types branch June 5, 2026 21:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants