-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Include external resources when exporting ZIP #4843
Include external resources when exporting ZIP #4843
Conversation
Thank you so much for this! Will review tomorrow :) |
Build for latest commit 0fc7739 is at https://pr4843.build.csb.dev/s/new. |
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. Latest deployment of this branch, based on commit 0fc7739:
|
This looks really good, that is the only comment I have :D |
Obrigado @SaraVieira :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great!
Again, thank you so much! |
What kind of change does this PR introduce?
Closes #2268
Closes #4048
What is the current behavior?
External resources are not included inside
public/index.html
when you export the project as a ZIP file, so users have to manually fix that issue after exporting.What is the new behavior?
External resources are included inside
<head>
tag.What steps did you take to test this? This is required before we can merge, make sure to test the flow you've updated.
Checklist