Skip to content

Composite pricing examples: various fixes#1038

Merged
danbri merged 2 commits intoschemaorg:sdo-deimosfrom
unor:composite-pricing-example-fixes
Mar 19, 2016
Merged

Composite pricing examples: various fixes#1038
danbri merged 2 commits intoschemaorg:sdo-deimosfrom
unor:composite-pricing-example-fixes

Conversation

@unor
Copy link
Copy Markdown
Contributor

@unor unor commented Mar 18, 2016

I fixed the points mentioned in #994 (itemtypitemtype; absolute URL value for itemtype attribute; missing </span>) and:

  • the Microdata examples had an additional </div>, which lead to parsing errors
  • the JSON-LD examples used a duplicated priceComponent property instead of using an array, which lead to parsing errors (please review if I used the array correctly, as I don’t know JSON-LD very well)

@unor
Copy link
Copy Markdown
Contributor Author

unor commented Mar 18, 2016

By the way, the used type http://schema.org/BankAccount (proposed in #965, I guess) is not yet part of schema.rdfa.

@mfhepp
Copy link
Copy Markdown
Contributor

mfhepp commented Mar 18, 2016

This is part of the FIBO proposal #965. We could replace the type in the example in order to remove this dependency.

danbri added a commit that referenced this pull request Mar 19, 2016
Composite pricing examples: various fixes
@danbri danbri merged commit b438105 into schemaorg:sdo-deimos Mar 19, 2016
@danbri
Copy link
Copy Markdown
Contributor

danbri commented Mar 19, 2016

Thanks @unor :) Merged, but let's change that example while the FIBO stuff awaits a bit more discussion - but what can we change it to?

@unor
Copy link
Copy Markdown
Contributor Author

unor commented Mar 21, 2016

BankAccount seems to have this type hierarchy (demo):

Thing > Intangible > Service > FinancialProduct > BankAccount

As FinancialProduct is new, too, Service would be the closest parent type.

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