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

ability to specify version in template + no more exports_default #765

Merged
merged 4 commits into from
Jun 19, 2022

Conversation

rmorshea
Copy link
Collaborator

@rmorshea rmorshea commented Jun 18, 2022

Description

Fixes broken Material UI example in docs. The React version being loaded did not work with the React version required by material UI. Now you specify which version of react you want in the template using standard NPM versioning (e.g. react@^17.0.0). This works for now, but won't if someone needed to specify more than one version in the template file.

This also fixes an issue with the simple chart example.

Testing for the dynamic default export has been done manually.

Checklist:

Please update this checklist as you complete each item:

  • Tests have been included for all bug fixes or added functionality.
  • The changelog.rst has been updated with any significant changes.
  • GitHub Issues which may be closed by this Pull Request have been linked.
  • I have left irrelevant checklist items unchecked instead of removing them.

@rmorshea rmorshea changed the title ability to specify react version in template ability to specify version in template + no more exports_default Jun 19, 2022
@rmorshea rmorshea merged commit 8e3b0b1 into main Jun 19, 2022
@rmorshea rmorshea deleted the fix-mui-ex branch June 19, 2022 06:34
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.

None yet

1 participant