Skip to content

refactor: remove unused assets, move remaining assets out of payload packages #12874

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 3 commits into from
Jun 23, 2025
Merged

refactor: remove unused assets, move remaining assets out of payload packages #12874

merged 3 commits into from
Jun 23, 2025
+10 −42

Conversation

AlessioGr
Copy link
Member

@AlessioGr AlessioGr commented Jun 19, 2025 •
edited
Loading

This PR removes the packages/payload/src/assets folder for the following reasons:

  • they were published to npm. Removing this decreases the install size of payload (excluding dependencies) from 6.22MB => 5.12MB, saving npm ≈100 GB in bandwidth every week 🎉
  • most assets were unused. The only used ones were moved to a different directory that does not get published to npm

This also updates some outdated asset URLs in our examples

@AlessioGr
refactor: remove unused assets, move remaining assets out of payload …
Loading
Loading status checks…
eea5465 
…package
@AlessioGr AlessioGr enabled auto-merge (squash) June 23, 2025 16:32
@AlessioGr AlessioGr disabled auto-merge June 23, 2025 16:32
@AlessioGr AlessioGr enabled auto-merge (squash) June 23, 2025 16:32
@AlessioGr AlessioGr merged commit bb17cc3 into main Jun 23, 2025
224 of 231 checks passed
@AlessioGr AlessioGr deleted the refactor/assets branch June 23, 2025 23:23
@github-actions GitHub Actions
Copy link
Contributor

🚀 This is included in version v3.44.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@paulpopus paulpopus

Assignees
No one assigned
Labels
created-by: Payload team
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@AlessioGr @paulpopus