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

STRF-3168 - ItemAvailability schema for products #1174

Merged
merged 1 commit into from
Mar 5, 2018
Merged

STRF-3168 - ItemAvailability schema for products #1174

merged 1 commit into from
Mar 5, 2018

Conversation

mjschock
Copy link
Contributor

What?

  • fixes schema.org microdata ItemAvailability for product pages

Why?

  • the product.availability field maps to the Availability text field under a product's Other Details section and therefore does not necessarily satisfy the enumerated fields for ItemAvailability. now we're explicitly selecting one of those enums depending on the product's fields. note that this will always show InStock when 'Track inventory by options' is enabled, regardless of the stock level of a variant.

Tickets

Screenshots

  • before
    screen shot 2018-02-28 at 3 49 37 pm

  • after
    screen shot 2018-02-28 at 3 44 01 pm
    screen shot 2018-02-28 at 3 43 22 pm
    screen shot 2018-02-28 at 3 42 12 pm

ping @bigcommerce/storefront-team

@bigbot
Copy link

bigbot commented Feb 28, 2018

Autotagging @bigcommerce/storefront-team @davidchin

@junedkazi
Copy link
Contributor

@bigcommerce/catalog-api-dt @bookernath

CHANGELOG.md Outdated

## 1.13.2 (2018-02-28)
- Fix updateView firing when there are no default options. [#1172](https://github.com/bigcommerce/cornerstone/pull/1172)
- Don't clear bulk pricing rules when feature is disabled. [#1173](https://github.com/bigcommerce/cornerstone/pull/1173)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am not sure I get what are these changelog entries for ? Did we miss it when we did 1.3.2 ?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

File it as a separate PR please.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@junedkazi 👍

@mjschock mjschock merged commit b2a1989 into bigcommerce:master Mar 5, 2018
@mjschock mjschock deleted the STRF-3168 branch March 5, 2018 18:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants