facebook-for-woocommerce icon indicating copy to clipboard operation
facebook-for-woocommerce copied to clipboard

Reorder and document feed fields

Open rcstr opened this issue 4 years ago • 1 comments

PR based on #1938 for branch feature/generate-feed-job-merged-implementations (#1924)

Changes proposed in this Pull Request:

PR documents and reorder feed columns. Order is based on column priority (required, option, etc). The docs taken in consideration are:

  • Required columns
    • https://www.facebook.com/business/help/120325381656392?id=725943027795860
  • Optional columns
    • https://developers.facebook.com/docs/marketing-api/catalog/reference/#da-commerce
  • Catalog template from Facebook

Closes #1894.

Fields not present in graph API

There are a few fields that aren't referenced in the FB docs, these are: variant, default_product, checkout_url, They aren't present in the docs neither the Catalog template.

I tested our generated feed file which contains these columns with the Facebook's Feed Debugger and no warnings, neither if I remove the columns.

Screen Shot 2021-05-10 at 16 01 05

Changelog

Dev - Document feed columns

rcstr avatar May 13 '21 01:05 rcstr

PR based on #1938 for branch feature/generate-feed-job-merged-implementations (#1924)

Thanks for clarifying this 😅

Let's chat (squad sync) about how we're using feature branches, and if this change (or others) can be shipped independently.

@rcstr In the meantime, it would be useful if you added some info to the PR description about how/where you want to ship this change. Or add that info to the issue - #1894.

Side note: if we're targeting a feature branch, we can be more proactive about removing the fields that aren't supported by API.

haszari avatar May 13 '21 01:05 haszari

Closing this as stale.

rawdreeg avatar Sep 30 '22 07:09 rawdreeg