Skip to content

GoogleOptimizer throws exception when adding product via API #2315

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.

Sign up for GitHub

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

Jump to bottom
Closed
matthewwiggins opened this issue Nov 10, 2015 · 2 comments
Closed

GoogleOptimizer throws exception when adding product via API #2315

matthewwiggins opened this issue Nov 10, 2015 · 2 comments
Assignees
@yaronish
Labels
bug report Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development

Comments

@matthewwiggins
Copy link

If Google Contents Experiments is enabled, adding a product using the V1/products/ endpoint causes GoogleOptimizer/Observer/AbstractSave.php line 117 to throw InvalidArgumentException. With contents experiments disabled the product add completes successfully.

Example valid request json:

    "product": {
        "sku": "abc1",
        "name": "abc",
        "visibility": 4,
        "type_id": "simple",
        "price": 3.62,
        "status": 1,
        "attribute_set_id": 4
    }
@vrann vrann added the MX label Nov 10, 2015
@matthewwiggins matthewwiggins changed the title 2.0.0RC - GoogleOptimizer throws exception when adding product via API GoogleOptimizer throws exception when adding product via API Nov 10, 2015
@katmoon
Copy link
Contributor

katmoon commented Mar 12, 2016

Hi @matthewwiggins

Thank you for the reporting issue and sorry for the delay.
I have created internal ticket MAGETWO-50417 for this issue.

Thanks,
Kateryna.

@katmoon katmoon added the Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development label Mar 12, 2016
@mslabko
Copy link
Member

mslabko commented Apr 12, 2016

Hello @matthewwiggins
Issue was fixed on develop branch (d58c63e) and will be available in 2.1 version
Thank you for reporting!

@mslabko mslabko closed this as completed Apr 12, 2016
magento-engcom-team added a commit that referenced this issue Nov 16, 2018
@magento-engcom-team
ENGCOM-3418: #12970: [Forwardport] Can't add grouped product, with ou…
d03a9b3 
…t of stock sub product, to cart. #2315

 - Merge Pull Request magento-engcom/magento2ce#2315 from p-bystritsky/magento2ce:ISSUE-12970-FORWARDPORT
 - Merged commits:
   1. f631c90
magento-engcom-team pushed a commit that referenced this issue Nov 16, 2018
@slavvka
ENGCOM-3418: #12970: [Forwardport] Can't add grouped product, with ou…
76d8d00 
…t of stock sub product, to cart. #2315
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@yaronish yaronish

Labels
bug report Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
6 participants
@matthewwiggins @mslabko @vkholoshenko @katmoon @vrann @yaronish