Skip to content

EAN13 must be included only in variations if they exists - #5

Open
misthero wants to merge 1 commit into
thoma202:masterfrom
misthero:patch-4
Open

EAN13 must be included only in variations if they exists#5
misthero wants to merge 1 commit into
thoma202:masterfrom
misthero:patch-4

Conversation

@misthero

Copy link
Copy Markdown

ebay refuses ean13 if the product has variations

ebay refuses ean13 if the product has variations
@mdelgrange

Copy link
Copy Markdown

Hi Misthero, thanks for your work.

We are currently working on a 1.12.2 which will add support for UPC & MPN (which becomes also mandatory), so there had been a bit of code work on this part. So we cannot accept you PR as is, but we will integrate your fix.

Thanks

@thoma202

thoma202 commented Feb 8, 2016

Copy link
Copy Markdown
Owner

Hi @misthero

Can you please send us the error you received when trying to send an EAN in the product and in the variations ? Because we did not see any error message when we tried it on our end.

Thanks,
Thoma

@misthero

misthero commented Feb 8, 2016

Copy link
Copy Markdown
Author

you can see it in the module configuration page -> view -> api logs, the message for me was in italian since it is connected to italian ebay.it

ShortMessage": "Per inserzioni con pi\u00c3\u00b9 varianti, i valori GTIN sono specificati a livello della variante.",
\n\t\t\t"LongMessage": "Per inserzioni con pi\u00c3\u00b9 varianti, i valori GTIN sono specificati a livello della variante. Rimuovi il valore GTIN dal livello dell'inserzione e fornisci un valore GTIN per ogni variante nell'inserzione.",
\n\t\t\t"ErrorCode": "21919420",
\n\t\t\t"SeverityCode": "Warning",
\n\t\t\t"ErrorParameters": [
\n\t\t\t\t{
\n\t\t\t\t\t"@attributes": {
\n\t\t\t\t\t\t"ParamID": "0"
\n\t\t\t\t\t},
\n\t\t\t\t\t"Value": "Per inserzioni con pi\u00c3\u00b9 varianti, i valori GTIN sono specificati a livello della variante."

that is in english something like this: LongMessage: for listings with variations GTIN values must be specified at variation level. Remove the GTIN value from the main advertisement and provide a GTIN value for each variant in your advertisement..

probably the exact english message will be different, I don't know i this is specific for ebay.it, anyway even when you manually create an item with variations in ebay you are allowed to enter EAN only in variations if they exists.

My patch fix this specific error only.

@thoma202

thoma202 commented Feb 8, 2016

Copy link
Copy Markdown
Owner

@misthero

Thanks for your return, as we see it, this is only a warning, this will not block the listing and revising of the list you have. Ebay only warns the user that this is not the correct behavior.

We've opened a support request to the eBay support to ask them what is the best thing for us to do. If this is becoming an error in the future or not and so on, as this might also be country specific.

We will come back to you to indicate you our decision.

@misthero

misthero commented Feb 8, 2016

Copy link
Copy Markdown
Author

yes, at the moment they are still listing products without EAN even in mandatory EAN restricted categories.

I don't know, but I suppose that in time they will become more strict, I see nothing wrong in fixing those warnings anyway, it is not too difficult.

Thank you for taking your time investigating the issue.

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

Successfully merging this pull request may close these issues.

3 participants