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

fix: disable source editor to avoid SWC bug #1575

Closed
wants to merge 1 commit into from

Conversation

petyosi
Copy link
Contributor

@petyosi petyosi commented Aug 30, 2023

Due to swc-project/swc#7640, the editor fails in production. The change here removes the plugin that causes the error.

There are two other alternatives:

  • disable the swc minification
  • upgrade to the latest Next.js version

Given the non-technical audience, I think that disabling the diff/source feature is the best option. Let me know if you think otherwise, though.

@github-actions
Copy link

github-actions bot commented Aug 30, 2023

✅ Tests will run for this PR. Once they succeed it can be merged.

@igoychev igoychev added the run tests Allows running the tests workflows for forked repos label Aug 31, 2023
@github-actions github-actions bot removed the run tests Allows running the tests workflows for forked repos label Aug 31, 2023
@igoychev
Copy link
Contributor

igoychev commented Sep 1, 2023

closing as it was fixed by updating next as per your recomendation in this PR: #1576

@igoychev igoychev closed this Sep 1, 2023
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