seanomeara96 / go-bigcommerce Goto Github PK

You never added the include sub-resource structs to the product struct, so if you use the query param include to get the sub-resource custom_fields , in the all product request function, you cannot access the custom field sub-resource struct in the returned product struct since the field currently does not exist, but should as it is a valid option from bigcommerce.

https://developer.bigcommerce.com/docs/rest-catalog/products#get-all-products

The following sub-resources include:

• variants
• images
• custom_fields
• bulk_pricing_rules
• primary_images
• modifiers
• options
• videos

I forked your repo and made the change, but since this still seems as an active repo, I wanted to make a PR to introduce this change.

Really appreciate your work here, but there were some missing parameters, as well as the customer endpoints that did not exist. I issued two pull requests that have not gotten a response from you that will correct these issues.

I have made these corrections, in this forked repo below.

https://github.com/william1benn/go-bigcommerce

I am more than willing to submit all this work to this repo to avoid duplication of efforts.

for any POST - PUT request bigcommerce requires Content-Type headers for v3 api endpoints. Currently in client.go this header is missing.

req.Header.Set("Content-Type", "application/json")

See Parameters
https://developer.bigcommerce.com/docs/rest-catalog/products#create-a-product