Skip to content

Conversation

dslovinsky
Copy link
Collaborator

@dslovinsky dslovinsky commented Apr 8, 2025

Context

Currently a handful of pages are broken like the ones on authentication. Apparently this was caused by MDX files directly imported into other MDX files. Fern uses a Markdown component for reusable snippets, so this PR migrates all instances of imported mdx files to use that component.

I've only tested a subset of these pages but they've all worked so far. We'll do a more thorough check during QA phase.

Pull Request Checklist


PR-Codex overview

This PR focuses on updating various documentation files to replace the usage of the Snippet component with the Markdown component for better consistency and clarity in the documentation.

Detailed summary

  • Replaced <Snippet /> with <Markdown src="..." /> in multiple .mdx files.
  • Updated files include:
    • index.mdx
    • drop-and-replace.mdx
    • authenticating-with-social-auth.mdx
    • authenticating-with-magic-link.mdx
    • quickstart.mdx
    • authenticating-with-otp.mdx
    • getting-started-rn-bare.mdx
    • getting-started-expo.mdx
    • passkey-signup.mdx
    • setting-up-the-accounts-provider.mdx
    • react-quickstart.mdx
    • setup-guide.mdx
    • overview.mdx
    • sponsor-gas.mdx
    • email-otp.mdx
    • email-magic-link.mdx
    • how-to-set-up-smart-account-client.mdx
    • login-methods/email-login.mdx
    • login-methods/social-login.mdx
    • login-methods/passkey-login.mdx
    • login-methods/social-providers.mdx
    • react-hooks.mdx
    • getting-started.mdx
    • terms.mdx

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

Copy link

vercel bot commented Apr 8, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
aa-sdk-site ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 8, 2025 9:36pm
aa-sdk-ui-demo ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 8, 2025 9:36pm

Copy link
Contributor

graphite-app bot commented Apr 8, 2025

How to use the Graphite Merge Queue

Add the label graphite-merge-queue to this PR to add it to the merge queue.

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

Copy link

github-actions bot commented Apr 8, 2025

🌿 Documentation Preview

Name Preview Updated (UTC)
Alchemy Docs 🔗 Visit Preview Apr 8, 2025, 9:34 PM

@dslovinsky dslovinsky merged commit 2b7df77 into main Apr 8, 2025
8 checks passed
@dslovinsky dslovinsky deleted the ds/fix-broken-docs-pages branch April 8, 2025 21:42
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