Skip to content

Composite pricing, updated (supersedes #830)#994

Merged
danbri merged 5 commits intoschemaorg:sdo-deimosfrom
mfhepp:CompositePricing
Mar 18, 2016
Merged

Composite pricing, updated (supersedes #830)#994
danbri merged 5 commits intoschemaorg:sdo-deimosfrom
mfhepp:CompositePricing

Conversation

@mfhepp
Copy link
Copy Markdown
Contributor

@mfhepp mfhepp commented Feb 18, 2016

This pull request updates #830. It adds examples for two domains and addresses the issue described in #829.

CompoundPriceSpecification is a needed enhancement from GoodRelations for cases in which a price consists of multiple components (e.g. 50 USD per day and 0.10 USD per mile for a rental car).

<div itemscope itemtype="http://schema.org/Offer">
<h1 itemprop="description">Our Private Bank Account Package: Low monthly fees and fair per-use charges.</h1>
<b itemprop="name">Private Bank Account Package</b>
<span itemprop="itemOffered" itemscope itemtype="BankAccount">
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The itemtype value must be an absolute URL.

danbri added a commit that referenced this pull request Mar 18, 2016
Composite pricing, updated (supersedes #830)
@danbri danbri merged commit e15f26d into schemaorg:sdo-deimos Mar 18, 2016
@danbri
Copy link
Copy Markdown
Contributor

danbri commented Mar 18, 2016

Merged as-is, but @mfhepp if you could fold in @unor 's various fixes into a followup PR that would be great.

@unor
Copy link
Copy Markdown
Contributor

unor commented Mar 18, 2016

I submitted a PR for the mentioned fixes as well as two fixes for issues I didn’t notice when reviewing it the first time: #1038

@mfhepp
Copy link
Copy Markdown
Contributor Author

mfhepp commented Mar 18, 2016

Thanks!

On 18 Mar 2016, at 23:24, unor notifications@github.com wrote:

I submitted a PR for the mentioned fixes as well as two fixes for issues I didn’t notice when reviewing it the first time: #1038


You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub

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