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

Multiple pictures from Magento to Allegro #63

Open
michalzimniak opened this issue May 29, 2020 · 5 comments
Open

Multiple pictures from Magento to Allegro #63

michalzimniak opened this issue May 29, 2020 · 5 comments

Comments

@michalzimniak
Copy link

Is there any option to add more than one picture from magento directly to the auction?

@michalzimniak michalzimniak changed the title Multiple pictures from Magento to Allego Multiple pictures from Magento to Allegro May 29, 2020
@michalzimniak
Copy link
Author

Hi,

Can I help you with that somehow?

@michalzimniak
Copy link
Author

Hi,

Whats the problem to add multiple images to the allegro auction?

@cieslix
Copy link
Collaborator

cieslix commented Jun 15, 2020

Hi @michalzimniak,

Our client wanted to have only one product picture.
If you want to have multiple images take a look on the loop https://github.com/macopedia/magento2-allegro/blob/master/Ui/AllegroOffer/Form/CreateDataProvider.php#L92
and remove lines 89-94.

@cieslix cieslix closed this as completed Jun 15, 2020
@cieslix
Copy link
Collaborator

cieslix commented Jun 15, 2020

@michalzimniak if you would like you can submit Pull Request with option to add more images, but this has to be optional and has to be added with configuration input in magento backend.

@cieslix cieslix reopened this Jun 15, 2020
@michalzimniak
Copy link
Author

Hi, Tried to comment the loop from lines:

//        foreach ($images['items'] as $key => $image) {
//            if ($image['file'] !== $allegroImage) {
//                unset($images['items'][$key]);
//            }
//         }

but seems like this do not change anything, I mean if I want to mark more than one "Allegro Image" in Images and Videos Tab, only one possible to mark.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants