-
-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
feat: delete product image #5129
base: develop
Are you sure you want to change the base?
feat: delete product image #5129
Conversation
|
@bacarybruno is attempting to deploy a commit to the medusajs Team on Vercel. A member of the Team first needs to authorize it. |
701eb6c
to
cb66f80
Compare
08c146e
to
55de8df
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As this is a feature that includes updating our core project config, please create a feature request in our GitHub Discussions, so we can discuss the approach. We'd like to avoid having long-running PRs open.
This has long been a pain point. I would love to see this implemented. The approach makes sense to me. It gives people the option to keep existing behavior if for some reason they want to keep images on delete. |
Done here #5149 @olivermrbl 👍 do not hesitate to tell me if something else need to be done |
Details: #5149
This PR intends to remove product images and thumbnail when a product is updated or deleted.
Tasks:
medusa-file-local
medusa-file-s3
medusa-file-minio
: need testersmedusa-file-spaces
: need testers