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

Sync and update Product images #132

Open
kossanah opened this issue Dec 7, 2024 · 0 comments · May be fixed by #133
Open

Sync and update Product images #132

kossanah opened this issue Dec 7, 2024 · 0 comments · May be fixed by #133
Labels
enhancement New feature or request

Comments

@kossanah
Copy link

kossanah commented Dec 7, 2024

Is your feature request related to a problem? Please describe.

Assumption: The ERP Item image are set as Public.
Upload the image when creating item on woocommerce.

Describe the solution you'd like

When an Image is uploaded to woocommerce REST api. copy of the image is copied over to woocommerce server, with an Id assigned. This ID must be refrenced for any future sync. If not a duplicate copy is made again on woocermmerce server and can cause server to be overloaded with multiple copy of same image. It could eventually lead to disk space getting filled due to hourly sync.

Describe alternatives you've considered

Ensure to make a request to woocommerce server to get associated image for a product and compare if the ERP version is newer.

Additional context

Additional logic maybe needed to synce multiple image of an Item, which may be required in some cases

@kossanah kossanah added the enhancement New feature or request label Dec 7, 2024
@kossanah kossanah changed the title <title>Sync and update Product images Sync and update Product images Dec 7, 2024
@kossanah kossanah linked a pull request Dec 8, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant