The BorrowAction should allow Organizations to be the lender #1494

Closed
vholland opened this Issue Jan 25, 2017 · 6 comments

Comments

Projects
None yet
5 participants
@vholland
Contributor

vholland commented Jan 25, 2017

Currently, http://schema.org/lender only allows a Person as the expected type. It would be useful to allow Organizations to also be lenders.

@erimille

This comment has been minimized.

Show comment
Hide comment
@erimille

erimille Jan 25, 2017

+1

If specific examples are needed, I'd be happy to share from a Library perspective.

+1

If specific examples are needed, I'd be happy to share from a Library perspective.

@AymenLoukil

This comment has been minimized.

Show comment
Hide comment
@AymenLoukil

AymenLoukil Jan 25, 2017

I agree +1 . In other perspectives also (eg : financial organizations..)

I agree +1 . In other perspectives also (eg : financial organizations..)

@RichardWallis

This comment has been minimized.

Show comment
Hide comment
@RichardWallis

RichardWallis Jan 25, 2017

Contributor

I agree +1
Also for http://schema.org/borrower
Most other sub-properties of http://schema.org/participant should too.

Participant itself expects both Person and Organization.

Are we not looking at an inheritance issue here? Should not sub-properties inherit the expected properties of their super-property?

If the answer to these two questions are yes, we are actually looking at a fix to the logic in the coding of the Schema.org site to automatically cary forward expected types from a property onto its sub-properties.

Contributor

RichardWallis commented Jan 25, 2017

I agree +1
Also for http://schema.org/borrower
Most other sub-properties of http://schema.org/participant should too.

Participant itself expects both Person and Organization.

Are we not looking at an inheritance issue here? Should not sub-properties inherit the expected properties of their super-property?

If the answer to these two questions are yes, we are actually looking at a fix to the logic in the coding of the Schema.org site to automatically cary forward expected types from a property onto its sub-properties.

@danbri

This comment has been minimized.

Show comment
Hide comment
@danbri

danbri Jan 25, 2017

Contributor

Re borrower and "most other sub-properties", if this isn't applicable to allow sub-properties we should go case by case. I'll fix this one for now. Richard - on the larger question of what the semantics are for subproperty/range/domain and whether the site should display, can you raise an issue?

Contributor

danbri commented Jan 25, 2017

Re borrower and "most other sub-properties", if this isn't applicable to allow sub-properties we should go case by case. I'll fix this one for now. Richard - on the larger question of what the semantics are for subproperty/range/domain and whether the site should display, can you raise an issue?

@danbri danbri closed this in 289b669 Jan 25, 2017

@danbri

This comment has been minimized.

Show comment
Hide comment
@danbri

danbri Jan 25, 2017

Contributor

http://webschemas.org/lender should be updated (but there can be cache-based delays; in my browser I had to use http://webschemas.org//lender to see the fix).

Contributor

danbri commented Jan 25, 2017

http://webschemas.org/lender should be updated (but there can be cache-based delays; in my browser I had to use http://webschemas.org//lender to see the fix).

@AymenLoukil

This comment has been minimized.

Show comment
Hide comment
@AymenLoukil

AymenLoukil Jan 25, 2017

Yes it is showing both Person / Organization.

Yes it is showing both Person / Organization.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment