Skip to content

ia/kyle feedback #2600

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 7 commits into
base: ia/docs-team-proposal
Choose a base branch
from
Open

ia/kyle feedback #2600

wants to merge 7 commits into from

Conversation

kylemac
Copy link
Contributor

@kylemac kylemac commented Sep 15, 2025 •
edited
Loading

  • make quickstart + next steps not collapsible
  • move sdk reference above ui components in top-level nav
  • refer to it as "reference" (singular, not plural)
  • move demo repos to bottom of sdk reference in nextjs
  • change name of "app/pages router references" to just app/pages router (still need to update URL paths)
  • un-nest iOS SDK helper functions
  • make "composables" list be monospace like with "hooks"
  • update API reference icon
  • remove an erroneous page from Android SDK reference

@kylemac
a few misc changes
ae80113 
- make quickstart + next steps not collapsible 
- move sdk reference above ui components in top-level nav
- refer to it as "reference" (singular, not plural)
- move demo repos to bottom of sdk reference in nextjs
@kylemac
change name of "app/pages router references" to just app/pages router
d1c3cb2
@kylemac
expand client-side helpers in astro sdk ref
1f61524
@kylemac
un-nest ios sdk helpers
bcf1480
@kylemac
make composables list be monospaced, like we do with hooks
539a9a6
@kylemac
update api reference icon
6458aa3
@kylemac kylemac requested a review from a team as a code owner September 15, 2025 17:19
@vercel Vercel
Copy link

vercel bot commented Sep 15, 2025 •
edited
Loading

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

Project Deployment Preview Updated (UTC)
clerk-docs Ready Ready Preview Sep 15, 2025 6:05pm

@kylemac kylemac changed the title Ia/kyle feedback ia/kyle feedback Sep 15, 2025
alexisintech
alexisintech approved these changes Sep 15, 2025
View reviewed changes
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.

makes sense to me!

@alexisintech
Copy link
Member

lets see what nick and sarah think!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@alexisintech alexisintech alexisintech approved these changes

@SarahSoutoul SarahSoutoul Awaiting requested review from SarahSoutoul

@NWylynko NWylynko Awaiting requested review from NWylynko

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@kylemac @alexisintech