Skip to content
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.

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

There is an issue with the urltransform prop, this prop is unstable and throwing an error named "Assertion: Unexpected transformLinkUri prop, use urlTransform instead" #859

Closed
4 tasks done
Ishuboi07 opened this issue Sep 18, 2024 · 8 comments
Labels
🤷 no/invalid This cannot be acted upon 👎 phase/no Post cannot or will not be acted on

Comments

@Ishuboi07
Copy link

Initial checklist

Affected packages and versions

^9.0.1

Link to runnable example

No response

Steps to reproduce

Installed the library into my Next.js application to transform the LLM's responses as the LLMs respond in markdown language, but after I landed at my home page, my client-side threw an error saying "Assertion: Unexpected transformLinkUri prop, use urlTransform instead"

Expected behavior

There should have been no error ideally as I didn't use the transformLinkUri prop and it was coming from the library itself.

Actual behavior

throwing this error "Assertion: Unexpected transformLinkUri prop, use urlTransform instead"

image

Runtime

Node v17

Package manager

npm 8

OS

Windows

Build and bundle tools

Next.js

@github-actions github-actions bot added 👋 phase/new Post is being triaged automatically 🤞 phase/open Post is being triaged manually and removed 👋 phase/new Post is being triaged automatically labels Sep 18, 2024
@ChristianMurphy
Copy link
Member

Welcome @Ishuboi07! 👋
What library are you using that wraps react-markdown? If there is one, open an issue there so they don't set the prop.
If you are directly using react markdown, please share a minimal reproducible error of how you got to that error.

@ChristianMurphy ChristianMurphy added the 🙊 open/needs-repro This needs a reproduction label Sep 18, 2024

This comment has been minimized.

@Ishuboi07
Copy link
Author

Welcome @Ishuboi07! 👋
What library are you using that wraps react-markdown? If there is one, open an issue there so they don't set the prop.
If you are directly using react markdown, please share a minimal reproducible error of how you got to that error.

There is no library wrapping react-markdown the error is being produced internally but working fine with react-showdown library

@Ishuboi07
Copy link
Author

Welcome @Ishuboi07! 👋
What library are you using that wraps react-markdown? If there is one, open an issue there so they don't set the prop.
If you are directly using react markdown, please share a minimal reproducible error of how you got to that error.

There is no library wrapping react-markdown the error is being produced internally but working fine with react-showdown library

You can reproduce the error by wrapping the ReactMarkdown into any text in the nextjs app in a different route. The error pops up into the home route, I converted the whole route into a client side rendered page and imported the component and wrapped the component upon the response received from LLMs since they use markdown language to give response

@ChristianMurphy
Copy link
Member

ChristianMurphy commented Sep 18, 2024

You can reproduce the error by wrapping the ReactMarkdown into any text in the nextjs app in a different route. The error pops up into the home route, I converted the whole route into a client side rendered page and imported the component and wrapped the component upon the response received from LLMs since they use markdown language to give response

@Ishuboi07 that isn't an example, much less a minimal reproducible example (MRE).
Here is some context on what an MRE is https://stackoverflow.com/help/minimal-reproducible-example
Create a small next app that shows what you are talking about, consider using https://next.new to create and share it.

@wooorm
Copy link
Member

wooorm commented Sep 18, 2024

Ishaan, millions of people use this project. They have no problems. You have unique problems. You have to be very clear.

@Ishuboi07
Copy link
Author

I don't know how, but it works now. I am sorry for creating this issue, you can close it now

This comment was marked as resolved.

@wooorm wooorm added the 🤷 no/invalid This cannot be acted upon label Sep 18, 2024
@github-actions github-actions bot added 👎 phase/no Post cannot or will not be acted on and removed 🤞 phase/open Post is being triaged manually 🙊 open/needs-repro This needs a reproduction labels Sep 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🤷 no/invalid This cannot be acted upon 👎 phase/no Post cannot or will not be acted on
Development

No branches or pull requests

3 participants