Skip to content

Conversation

SarahSoutoul
Copy link
Contributor

@SarahSoutoul SarahSoutoul commented Oct 7, 2025

🔎 Previews:

What does this solve?

It was reported that the Expo quickstart doesn't mention the need to enable Native API. However, we have a callout at the top of the file stating that need, which is the case for the other Native SDK quickstarts.

We thought moving it as an actual step might help in not missing the callout.

What changed?

  • Moved Native API from a callout to an actual step (did it for all native SDKs to keep things consistent)

Checklist

  • I have clicked on "Files changed" and performed a thorough self-review
  • All existing checks pass

@SarahSoutoul SarahSoutoul self-assigned this Oct 7, 2025
@SarahSoutoul SarahSoutoul requested a review from a team as a code owner October 7, 2025 19:06
Copy link

vercel bot commented Oct 7, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Updated (UTC)
clerk-docs Ready Ready Preview Oct 8, 2025 6:00pm

@@ -1,2 +1,3 @@
> [!IMPORTANT]
> Ensure that the Native API is enabled to integrate Clerk in your native application. In the Clerk Dashboard, navigate to the [**Native Applications**](https://dashboard.clerk.com/last-active?path=/native-applications) page.
## Enable Native API
Copy link
Member

Choose a reason for hiding this comment

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

what do you think about keeping the heading out incase we want to reuse the text in places that don't have a heading or require a different heading level

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah that sound good! Will make the change.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Change made here: Changes post feedback

Copy link
Member

@alexisintech alexisintech left a comment

Choose a reason for hiding this comment

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

awesome, thanks sarah 😸💖

@SarahSoutoul SarahSoutoul merged commit 4631e57 into main Oct 8, 2025
7 checks passed
@SarahSoutoul SarahSoutoul deleted the ss/DOCS-11023 branch October 8, 2025 18:05
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