Add 'tax category' to the All Products report

[chahmedejaz]

⚠️ Please use clockify code #12476 Flower Farms

What? Why?

• Closes Add 'tax category' to the All Products report #13008
• This PR adds the tax category field in the All Products report

What should we test?

• As mentioned in the issue

Release notes

Changelog Category (reviewers may add a label for the release notes):

• User facing changes
• API changes (V0, V1, DFC or Webhook)
• Technical changes only
• Feature toggled

[chahmedejaz]

tax_category returns the default TaxCategory if the variant doesn't have one.
That's why we need to ensure that this tax category value maps to the variant's assigned tax category, rather than the default one.
This is the whole purpose of the addition of this field in the report.

[rioug]

Looks good 👍

[filipefurtad0]

Hey @chahmedejaz,

I've checked the acceptance criteria from the issue:

• Logging in as an enterprise and visiting to /admin/reports/products_and_inventory/all_products -> access the report page. The report renders as before:

• Verified that the "tax category" column is not selected by default:

• Selecting the column from the dropdown displays the respective tax category

• If there is no tax category associated with a product, then the report renders and None is displayed. Bonus points, this is a translatable string:

Ok, so indeed now the report easily allows identifying the products which are missing tax categories. However, as displayed in the pic above, the string is not translatable, on the report page (right side). We can see, that on the products page, this is the case, so the strings' translation is available on Transifex. I'll open an issue to address this. (This is also the case with the column Group Buy Unit Quantity).

Other than this minor glitch, the PR looks fine - I'll merge this now, so we include it on the latest release.