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 upstream issue in Create-React-App #514

Merged
merged 5 commits into from
Jan 18, 2022
Merged

Conversation

XavierGeerinck
Copy link
Contributor

Description

There is a bug upstream in Create-React-App that doesn't initialize the MiniCssExtractPlugin correctly anymore as it is now requiring Types

Issue reference

Checklist

Signed-off-by: Xavier Geerinck <xavier.geerinck@gmail.com>
Signed-off-by: Xavier Geerinck <xavier.geerinck@gmail.com>
Signed-off-by: Xavier Geerinck <xavier.geerinck@gmail.com>
Signed-off-by: Xavier Geerinck <xavier.geerinck@gmail.com>
Signed-off-by: Xavier Geerinck <xavier.geerinck@gmail.com>
Copy link
Member

@berndverst berndverst left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Previous container build failures are now resolved. As far as I'm concerned this is good now :)

Copy link
Member

@msfussell msfussell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@msfussell msfussell merged commit 26ae0d0 into dapr:master Jan 18, 2022
@XavierGeerinck XavierGeerinck deleted the patch-1 branch January 18, 2022 17:22
Arashrohani pushed a commit to Arashrohani/quickstarts that referenced this pull request Feb 16, 2022
* Fix upstream issue in Create-React-App

Signed-off-by: Xavier Geerinck <xavier.geerinck@gmail.com>

* Add distributed calculator fix for upstream plugin

Signed-off-by: Xavier Geerinck <xavier.geerinck@gmail.com>

* Editing wrong file

Signed-off-by: Xavier Geerinck <xavier.geerinck@gmail.com>

* Undo edits

Signed-off-by: Xavier Geerinck <xavier.geerinck@gmail.com>

* Make the build process a bit more clear

Signed-off-by: Xavier Geerinck <xavier.geerinck@gmail.com>
Signed-off-by: Arash Rohani <arash.rohani@gmail.com>
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

3 participants