1EdTech / openbadges-specification

Specs related to Open Badges
183 stars 68 forks source link

Refine IMS Documentation Formatting #128

Open timothyfcook opened 7 years ago

timothyfcook commented 7 years ago

The update to the criteria object and the surrounding documentation is a bit obscure.

Suggest refinements to:

ottonomy commented 7 years ago

Yes, both id and narrative may be used in this class. At least one is required. Displayers will presumably be able to render some Markdown into HTML along with other badge metadata and and a link if there is one.

Examples of criteria with and without narratives are provided on the examples page linked from the Criteria class http://openbadgespec.org/examples/#Criteria

timothyfcook commented 7 years ago

Ok, part of the issue was that certain people (me) didn't see the "Example" link next to the Object Headers. Some considerations:

jbohrer commented 7 years ago

Agree with improving readability of Required? attribute. This is related to #107

ottonomy commented 7 years ago

@timothyfcook and @jbohrer: In this case, the intent of the existing draft is that issuers use Either a URI for criteria, a narrative embedded in the badge or both. That means that neither id nor narrative is a required property. As you develop a required column in the tables, make sure we don't neglect how this situation will be described.

In text,

BadgeClasses MUST contain either a criteria id, a criteria narrative, or both.

The existing documentation in the core document does not forcefully state this point. The current text reads:

Embedding criteria allows either enhancement of an external criteria page or increased portability and ease of use by allowing issuers to skip hosting the formerly-required external criteria page altogether.

timothyfcook commented 7 years ago

Going to update this to become a more general-purpose ticket to improve a few aspects of the overall documentation formatting:

Acceptance Criteria

  1. Add a "Required?" column to each of the object definitions
  2. Make Examples links separate from the headers
  3. Add the "Examples" page to the site tree/top navigation for the main documentation page
timothyfcook commented 7 years ago

PR should be done against the IMS 2.0 branch