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(ui): react-datepicker inserting invalid require("react") calls in our bundle #6833

Merged
merged 2 commits into from
Jun 18, 2024

Conversation

AlessioGr
Copy link
Member

@AlessioGr AlessioGr commented Jun 18, 2024

See code comments for description

Fixes #6828

Webpack before this PR:
✓ Starting...
✓ Ready in 2.2s
○ Compiling / ...
✓ Compiled / in 1978ms (520 modules)
GET / 200 in 2082ms
○ Compiling /admin/[[...segments]] ...
✓ Compiled /admin/[[...segments]] in 7.2s (5105 modules)
[11:35:52] WARN: No email adapter provided. Email will be written to console. More info at https://payloadcms.com/docs/email/overview.
GET /admin 200 in 8932ms
GET /admin 200 in 232ms
○ Compiling /api/[...slug] ...
✓ Compiled /api/[...slug] in 2.1s (4096 modules)
GET /admin 200 in 2817ms

Webpack after this PR:
✓ Ready in 2.1s
○ Compiling / ...
✓ Compiled / in 2s (520 modules)
GET / 200 in 2115ms
○ Compiling /admin/[[...segments]] ...
✓ Compiled /admin/[[...segments]] in 7.3s (5105 modules)
[11:46:40] WARN: No email adapter provided. Email will be written to console. More info at https://payloadcms.com/docs/email/overview.
GET /admin/collections/pages 200 in 8927ms
GET /admin/collections/pages 200 in 119ms
GET /admin/collections/pages 200 in 160ms
○ Compiling /api/[...slug] ...
✓ Compiled /api/[...slug] in 945ms (4096 modules)

@AlessioGr AlessioGr merged commit b7d5a6a into beta Jun 18, 2024
49 of 50 checks passed
@AlessioGr AlessioGr deleted the fix/reactdatepicker-commonjs-hell branch June 18, 2024 15:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant