Include shipping weight details in product data

[Andrew998]

Added shipping weight details for Google Merchant, formatted as a string with 6 decimal places. Included condition to check for product weight before adding to data.

[vseager]

@Andrew998 Thanks for the contribution. This should respect the module config:

$this->getConfig('structureddata/product/include_weight')

Also, why rounding to 6 decimal places? Magneto rounds to 4 decimal places, so this seems unnecessary.

[Andrew998]

Thanks. I'm not too familiar with coding for Magento so all comments are welcome.

Is the process that I make those changes to the contribution?

[vseager]

Hi Andrew,

Yes please go ahead and update your merge request and we will review it again for you. If not, we can put it in our roadmap to implement at a later date.

[vseager]

The latest commit removes the check for a valid weight on the product. As well as checking if the config allows weight to be included, you must still check that a weight existing on the product. See the short description as an example of this logic.

[Andrew998]

Thank you. I'll get there eventually!