Skip to content
This repository has been archived by the owner on Feb 19, 2020. It is now read-only.

Add support for code splitting and static assets #104

Merged
merged 7 commits into from Nov 1, 2019
Merged

Add support for code splitting and static assets #104

merged 7 commits into from Nov 1, 2019

Conversation

samuelmeuli
Copy link
Contributor

@samuelmeuli samuelmeuli commented Sep 24, 2019

Q                       A
Fixed Issues? Closes #61, closes #83
Patch: Bug Fix? No
Minor: New Feature? Yes
Major: Breaking Change? No
Tests Added + Pass? No
Documentation Provided Yes
Any Dependency Changes? Yes
License Apache License, Version 2.0

See the newly added doc files for a description of the changes.

Requires facebook/create-react-app#7721 and wcm-io-frontend/aem-clientlib-generator#23 to be merged and released first.

@grubyak
Copy link
Contributor

grubyak commented Sep 30, 2019

test-spa-react-react-app: Failed to run task: 'npm run build' failed -- did you see that?

@samuelmeuli
Copy link
Contributor Author

@grubyak Yeah, it's not working yet because it builds on the code of the currently unreleased PRs. Once they're released, we can update the dependencies and the tests should pass :)

@samuelmeuli
Copy link
Contributor Author

create-react-app v3.2 is released and includes our change.

@samuelmeuli
Copy link
Contributor Author

wcm-io-frontend/aem-clientlib-generator#23 has also been merged and released, so this PR can now be reviewed :)

@samuelmeuli samuelmeuli marked this pull request as ready for review October 30, 2019 12:50
@samuelmeuli samuelmeuli merged commit 6c713d4 into adobe:development Nov 1, 2019
@samuelmeuli samuelmeuli deleted the code-splitting branch November 1, 2019 09:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants