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

IBX-84: Added REST eZ Commerce for 3.3 #7

Merged
merged 1 commit into from
May 17, 2021
Merged

Conversation

Steveb-p
Copy link
Contributor

Question Answer
JIRA issue IBX-84
Type feature
Target Ibexa version v3.3.3
BC breaks no
Doc needed no (already provided)

Adds ezsystems/ezcommerce-rest as a dependency.

Checklist:

  • Provided PR description.
  • Tested the solution manually.
  • Checked that target branch is set correctly.
  • Asked for a review (ping @ibexa/engineering).

@bogusez bogusez self-assigned this May 12, 2021
@bogusez
Copy link
Contributor

bogusez commented May 13, 2021

I've found an issue (status code 500) for endpoint with updating / adding product to the basket: /api/ezp/v2/commerce/basket/7/line

Screenshot 2021-05-13 at 11 42 47

To resolve the problem we have to change payload data from:

{ "BasketLinesData": [ { "basketLineId": 5, "lineNumber": 1, "sku": "001", "quantity": "2", "isVariant": "0", "variantCode": "" } ] }

to

{ "BasketLinesData": [ { "sku": "001", "quantity": "2", "isVariant": "0", "variantCode": "" } ] }

Except this, we have also change header Content-Type data to: application/vnd.ez.api.BasketLineData+json

@adamwojs adamwojs merged commit 03f4a07 into 3.3 May 17, 2021
@adamwojs adamwojs deleted the ibx-84-ezcommerce-rest branch May 17, 2021 06:23
@adamwojs
Copy link
Member

@Steveb-p Could you please merge up changes?

@Steveb-p
Copy link
Contributor Author

Steveb-p commented May 17, 2021

Merging up postponed for a second, ezcommerce-rest needs a branch for 4.0 version.

Merged up in 1106fa7.

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

Successfully merging this pull request may close these issues.

None yet

4 participants