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

Updates IssuerOrg to Issuer Profile #71

Merged
merged 2 commits into from Aug 18, 2016
Merged

Conversation

ottonomy
Copy link
Contributor

I'm proposing a change to better account for non-organizational (i.e. individual) issuer entities.

I propose that we change the name of the IssuerOrganization data class to Issuer Profile. The linked data IRI https://w3id.org/openbadges/#Issuer should stay the same. Corrects one error in documentation between docs and core context file.

Benefits:

  • More explicitly support the fact that individual people may be issuers of Open Badges

Drawbacks:

  • None known.

Any objections to merging or questions?

@ottonomy
Copy link
Contributor Author

ottonomy commented Aug 4, 2016

Comments from @kayaelle on today's Standard WG Call:

  • Need to make additional adjustments to the Issuer Profile section to clarify language in the table's description field for the following properties: email, url, name, description. Any instance of "organization" in this table should be clarified to wording like "entity or organization"

@ottonomy
Copy link
Contributor Author

ottonomy commented Aug 5, 2016

I think I got all the modifications Kerri proposed, and corrected a couple miscapitalizations of IRIs for a few property identifiers defined in the context

Please provide your 👍s or objections here or on the mailing list thread. If there are no objections by end of day Monday the 8th, we'll hit the merge button. Otherwise, we'll continue discussion a little longer to see if we can build to consensus. This is a very minor change and has seen no opposition so far.

@ottonomy ottonomy merged commit 8d0c9d5 into 1EdTech:master Aug 18, 2016
amiller-ims pushed a commit that referenced this pull request Dec 11, 2020
Add trademark to Badge Connect
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.

None yet

1 participant