Skip to content

Remove @shopify/web-worker Dependency from packages/upload-media #69816

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

Merged
merged 1 commit into from
Apr 4, 2025

Conversation

im3dabasia
Copy link
Contributor

What?

Closes #69254

Removes the deprecated @shopify/web-worker dependency from the packages/upload-media package.

Why?

The @shopify/web-worker package is no longer being maintained or supported according to its NPM page. Removing this unused dependency helps future-proof our codebase

How?

  1. Ran the following command to remove the dependency
npm uninstall @shopify/web-worker -w packages/upload-media
  1. Verified the package isn't used anywhere in the codebase by searching with :
grep -r "@shopify/web-worker" --include="*.js" --include="*.ts" --include="*.tsx" .
  1. No results were found in the search, confirming the package wasn't actively used
  2. Tested that media upload functionality continues to work correctly after removal

Copy link

github-actions bot commented Apr 4, 2025

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: im3dabasia <im3dabasia1@git.wordpress.org>
Co-authored-by: Mamaduka <mamaduka@git.wordpress.org>
Co-authored-by: swissspidy <swissspidy@git.wordpress.org>
Co-authored-by: SmushyTaco <smushytaco@git.wordpress.org>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

@Mamaduka Mamaduka requested a review from swissspidy April 4, 2025 11:13
@Mamaduka Mamaduka added [Type] Enhancement A suggestion for improvement. [Feature] Media Anything that impacts the experience of managing media labels Apr 4, 2025
Copy link
Member

@swissspidy swissspidy left a comment

Choose a reason for hiding this comment

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

As I said in #69254, the whole package is still work in progress, so this might just be a temporary change.

@swissspidy swissspidy merged commit 132e8c3 into WordPress:trunk Apr 4, 2025
68 checks passed
@github-actions github-actions bot added this to the Gutenberg 20.7 milestone Apr 4, 2025
@Mamaduka
Copy link
Member

Mamaduka commented Apr 4, 2025

Thanks, @swissspidy!

chriszarate pushed a commit to chriszarate/gutenberg that referenced this pull request Jul 1, 2025
…ess#69816)

Co-authored-by: im3dabasia <im3dabasia1@git.wordpress.org>
Co-authored-by: Mamaduka <mamaduka@git.wordpress.org>
Co-authored-by: swissspidy <swissspidy@git.wordpress.org>
Co-authored-by: SmushyTaco <smushytaco@git.wordpress.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Media Anything that impacts the experience of managing media [Type] Enhancement A suggestion for improvement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove @shopify/web-worker Dependency in @wordpress/upload-media
3 participants