adlnet / xAPI-Spec

The xAPI Specification describes communication about learner activity and experiences between technologies.
https://adlnet.gov/projects/xapi/
901 stars 405 forks source link

Stray OAuth Requirement #976

Closed andyjohnson closed 8 years ago

andyjohnson commented 8 years ago

Fixed a MUST in normative language - moved into a requirement.

andyjohnson commented 8 years ago

Fixes #965

ljwolford commented 8 years ago

+1

garemoko commented 8 years ago

Since there's only 1 requirement here, should we just have 1 list of Requirements for OAuth here: https://github.com/adlnet/xAPI-Spec/blob/ForStrayMust/xAPI-Communication.md#requirements-18 ? That was what I thought you meant in the issue.

andyjohnson commented 8 years ago

I like that suggestion, moved in the latest commit

andyjohnson commented 8 years ago

Lol, I guess that did leave the section empty. I got rid of the description header but didn't realize that was all in that section! Fixed.

garemoko commented 8 years ago

+1

ljwolford commented 8 years ago

+1