This repository has been archived by the owner. It is now read-only.

Enhances Issue: #85 Add 'worksfor' to Person Schema #87

Closed
wants to merge 2 commits into
from

Conversation

Projects
None yet
3 participants
Contributor

inetbiz commented Nov 3, 2013

Needs testing and POedit work for locale

Collaborator

SleeplessByte commented Nov 3, 2013

Cool dude! Let's do the testing and localization and then merge this.

Contributor

inetbiz commented Nov 3, 2013

I'm installing a new theme in my MU. I'll upload my branch for testing and include screenshots from google rich snippet tool. The mystery I have right now is nesting that worksfor in person schema. Does it need to match the name listed in organization or what hrmm. We'll get it right sooner or later. There's a lot more that can be added to both person and organization

Contributor

inetbiz commented Nov 4, 2013

http://stackoverflow.com/questions/17186058/html5-microdata-itemref-to-another-itemscope-person-works-for-organization for digestion. My code did not display in the jquery form. I don't see that my commit was in my working copy either. I'm trying to recommit in my branch now.

Collaborator

SleeplessByte commented Jan 16, 2014

@inetbiz please see this commit and the binary differences between the png's

@stephenyeargin stephenyeargin modified the milestone: 1.1.0 Feb 7, 2015

Contributor

stephenyeargin commented Apr 8, 2015

Alright, I'm going to close this one and rebuild it. There's a lot going on that's not exactly related to the feature -- namely, the whitespace changes and encoding. However, the work itself looks good, so I'm going to do what I can to recover it.

stephenyeargin added a commit that referenced this pull request Apr 8, 2015

@stephenyeargin stephenyeargin removed this from the 1.1.0 milestone Apr 8, 2015

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