nksaraf / vinxi

The Full Stack JavaScript SDK
https://vinxi.vercel.app
MIT License
1.33k stars 57 forks source link

Fix Vinxi MDX ignoring query params #313

Open XiNiHa opened 2 weeks ago

XiNiHa commented 2 weeks ago

Currently, Vinxi MDX transforms MDX files whether they are imported with query params or not. Since Vinxi MDX has enforce: "pre", it blocks features like ?raw query param from working correctly. This PR fixes the issue by ensuring MDX files are only transformed when there are no query params.

codesandbox[bot] commented 2 weeks ago

Review or Edit in CodeSandbox

Open the branch in Web EditorVS CodeInsiders
Open Preview

vercel[bot] commented 2 weeks ago

@XiNiHa is attempting to deploy a commit to the Nikhil Saraf's projects Team on Vercel.

A member of the Team first needs to authorize it.

changeset-bot[bot] commented 2 weeks ago

⚠️ No Changeset found

Latest commit: 0ff99efc05c0829b2b58fea5bb8fd3b6e25d80ee

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR