Skip to content

Include docs redirects in the generated docs artifact #2240

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
Merged
merged 2 commits into from
May 16, 2025
Merged

Include docs redirects in the generated docs artifact #2240

merged 2 commits into from
May 16, 2025

Conversation

NWylynko
Copy link
Contributor

What does this solve?

  • We need the redirects data to properly redirect requests

What changed?

  • This parses in the files (checking they are valid json / jsonc)
  • It optimizes the static redirects so users aren't making multiple hops
  • It writes out the redirects in to the dist folder

Checklist

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

@NWylynko NWylynko requested a review from a team as a code owner May 15, 2025 07:18
@NWylynko NWylynko requested a review from jescalan May 15, 2025 07:18
@github-actions GitHub Actions
Copy link

Hey, here’s your docs preview: https://clerk.com/docs/pr/2240

@jescalan jescalan merged commit 7d38b92 into main May 16, 2025
9 checks passed
@jescalan jescalan deleted the nick/include-docs-redirects-in-dist branch May 16, 2025 21:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jescalan jescalan jescalan approved these changes

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
@NWylynko @jescalan