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

LRS can support additional resources #601

Open
garemoko opened this issue Jan 19, 2015 · 6 comments
Open

LRS can support additional resources #601

garemoko opened this issue Jan 19, 2015 · 6 comments
Labels

Comments

@garemoko
Copy link
Contributor

Consider adding something to https://github.com/adlnet/xAPI-Spec/blob/master/xAPI.md#71-error-codes to say that the LRS can support additional resources not in the spec and add error code 405 to the list of example codes.

See https://github.com/adlnet/xAPI_LRS_Test/issues/24

@fugu13
Copy link
Contributor

fugu13 commented Jan 19, 2015

We don't really need to say that an LRS can support additional resources, that's just part of the nature of the web, but if people want to put it in, that's fine too. I'd actually think some positive language about possible future endpoints might be good, maybe "If additional endpoints are added to the spec, every path segment specified will start with a letter". That would mean LRS developers could safely stick their own resources in various places by starting with something other than a letter in part of the path.

@garemoko
Copy link
Contributor Author

That's an interesting approach. To follow the pattern from statement structure, could we say that future endpoint will never start with "extensions/" and possibly add in a SHOULD to locate additional end points there. I'm aware of at least one LRS already not following this, but if it's a SHOULD, that's ok.

@garemoko
Copy link
Contributor Author

garemoko commented Mar 3, 2015

This one also needs discussion on a call to determine:

  • Whether or not we want to commit to the broad approach outlined by @fugu13
  • The detail of how we'll implement it (see my suggestion and @fugu13's original suggestion as possibilities)

@andyjohnson
Copy link
Contributor

Per the 3/18/15 call, like both approaches, but no decision made on which one. Requirement would be a SHOULD*

@fugu13
Copy link
Contributor

fugu13 commented Mar 18, 2015

I don't think they need to be exclusive; we could easily allow for both options, and this is an area it's reasonable to be expansive

@garemoko
Copy link
Contributor Author

That PR never added 405 to the list of possible error codes. See adlnet-archive/grail-issue-tracker#55

@garemoko garemoko reopened this Dec 28, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants