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

Debugging react code in production from editor #1516

Closed
bytasv opened this issue Jan 5, 2023 · 1 comment · Fixed by #1881
Closed

Debugging react code in production from editor #1516

bytasv opened this issue Jan 5, 2023 · 1 comment · Fixed by #1881
Labels
enhancement This is not a bug, nor a new feature feature: App Editor scope: toolpad-studio Abbreviated to "studio"

Comments

@bytasv
Copy link
Contributor

bytasv commented Jan 5, 2023

Summary 💡

When deployed, Toolpad editor runs React in production mode. react is shared across all aspects of Toolpad. This means that React in the runtime also runs in production mode. This has the side effect that errors become illegible and very hard to debug.

Examples 🌈

https://master--toolpad.mui.com/_toolpad/app/clcixtnzp0001p69solvtz8w7/codeComponents/uc03m0j
https://master--toolpad.mui.com/_toolpad/app/clcixtnzp0001p69solvtz8w7/pages/00196rt

@bytasv bytasv added status: waiting for maintainer These issues haven't been looked at yet by a maintainer feature: App Editor enhancement This is not a bug, nor a new feature labels Jan 5, 2023
@Janpot Janpot removed the status: waiting for maintainer These issues haven't been looked at yet by a maintainer label Jan 5, 2023
@Janpot
Copy link
Member

Janpot commented Jun 12, 2023

This has been fixed by #1881

@Janpot Janpot closed this as completed Jun 12, 2023
@oliviertassinari oliviertassinari added the scope: toolpad-studio Abbreviated to "studio" label Nov 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement This is not a bug, nor a new feature feature: App Editor scope: toolpad-studio Abbreviated to "studio"
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants