AICC / CMI-5_Spec_Current

cmi5
http://adlnet.gov
Apache License 2.0
181 stars 91 forks source link

References to ‘xAPI Specification’ in Sections 8.1.2, 8.2.2 and 8.3 are awkward and inconsistent. #803

Closed MartinKoob closed 1 year ago

MartinKoob commented 1 year ago

There are three instances where the the xAPI specification is referred to in a way that is not consistent with other parts of the document and that appear either awkward or as an incomplete sentence.

Section 8.1.2 fetch

AU Usage: The AU MUST get the fetch value from the query string. The AU MUST make an HTTP POST to the fetch URL to retrieve the authorization token as defined in Section 8.2. The AU MUST place the authorization token in the Authorization headers of all HTTP requests made to the endpoint using the xAPI. The AU SHOULD NOT make more than one post to the fetch URL.

In the field "AU Usage" above the following phrase,

’using the xAPI.’

sounds awkward but is a complete sentence if you expand the acronym. E.g.

'… using the Experience API.'

Alternately the following sounds better with the acronym

’…using xAPI.’

but sounds awkward and appears to be an incomplete sentence with the acronym expanded.

‘… using Experience API.”

Section 8.2.2 Definition: auth-token

AU Usage:

The AU MUST get the auth-token value using an HTTP POST to the fetch URL. The AU MUST then place the authorization token in the Authorization headers of all HTTP requests made to the endpoint using the xAPI.

8.3 Other Launch Environments

Other launch environments are not currently implemented in this specification. cmi5 implementations for LMS's and AU's in these other environments will use the same REST communication interface as specified in xAPI specification. The xAPI specification does not specify launch mechanisms.

In this section above both the acronym and the expanded acronym sound awkward.

Suggested revisions

In other places in the document that use ‘the xAPI’ it is always followed by another term. E.g.

Sections 8.1.2 & 8.2.2 would read better as:

…using the xAPI Specification.

Sections 8.3 would read better as:

as specified in the xAPI specification.
MrBillMcDonald commented 1 year ago

Per the August 18, Meeting - the following was agreed:

  1. Update xAPI definition in definitions
  2. Update references to xAPI specification (both ADL and IEEE)
  3. Replace “xAPI specification” as “xAPI” (and make associated grammar changes)

Update Definition as follows:

Update References as follows: