Skip to content

Conversation

rpmccarter
Copy link
Contributor

@rpmccarter rpmccarter commented Dec 17, 2024

We're attempting to migrate from the pages router to the app router in our main project. Unfortunately, next-mdx-remote does not play nicely with newer versions of the app router (see this issue). To move forward, we're migrating from next-mdx-remote to next-mdx-remote-client, a fork of next-mdx-remote that is actively maintained and has a better API

When testing this, I ran into a huge host of issues attempting to test the example projects. I reckoned with God. As a result, I also decided to migrate the repo to a Yarn v4 workspace in this PR. Excitingly, the example projects appear to be working with the version of @mintlify/mdx in the codebase!

Copy link

vercel bot commented Dec 17, 2024

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

Name Status Preview Comments Updated (UTC)
mdx-app-router ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 18, 2024 1:02am
mdx-pages-router ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 18, 2024 1:02am

@hahnbeelee
Copy link
Member

can u update the readme outlining the new api?

@rpmccarter
Copy link
Contributor Author

@hahnbeelee updated readmes!

Copy link
Member

@hahnbeelee hahnbeelee left a comment

Choose a reason for hiding this comment

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

LGTM

@rpmccarter rpmccarter merged commit f7d4680 into main Dec 18, 2024
3 checks passed
@rpmccarter rpmccarter deleted the ronan/migrate-to-new-package branch December 18, 2024 02:37
@talatkuyuk
Copy link

talatkuyuk commented Jan 31, 2025

Hi @rpmccarter, as the author, I'm glad to see that Mintlify is using next-mdx-remote-client under the hood. 👏👏👏

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.

3 participants