Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Support buildstack changes for other modules #1446

Conversation

GuySartorelli
Copy link
Member

This is needed to avoid issues, e.g. with multiple copies of @emotion being loaded by react-select/creatable vs react-select vs react-select/async in different modules. If they're all exposed from the same source (admin), they all use the same copy of @emotion and therefore there are no conflicts or issues with css from different react-select components.

Parent issue

@GuySartorelli GuySartorelli changed the title Pulls/2/support buildstack changes Support buildstack changes for other modules Jan 19, 2023
@GuySartorelli GuySartorelli marked this pull request as draft January 19, 2023 01:27
@GuySartorelli
Copy link
Member Author

GuySartorelli commented Jan 19, 2023

Setting this to draft for now - I'll run a final yarn build when I'm sure I won't be making any other changes.
Edit: This can now be reviewed

I thought these would have come automatically, but it turns out they
need to be independently exposed and added to externals config.
Creating a new emotion cache for each provider can result in clashes
@GuySartorelli GuySartorelli force-pushed the pulls/2/support-buildstack-changes branch from 6b803eb to 4a1b488 Compare January 24, 2023 21:06
@GuySartorelli GuySartorelli marked this pull request as ready for review January 26, 2023 02:33
Copy link
Contributor

@sabina-talipova sabina-talipova left a comment

Choose a reason for hiding this comment

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

Look good for me

@sabina-talipova sabina-talipova merged commit b039eba into silverstripe:2 Jan 26, 2023
@sabina-talipova sabina-talipova deleted the pulls/2/support-buildstack-changes branch January 26, 2023 03:03
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants