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

Character Limits #4

Closed
FeLungs opened this issue Oct 31, 2022 · 5 comments
Closed

Character Limits #4

FeLungs opened this issue Oct 31, 2022 · 5 comments
Assignees

Comments

@FeLungs
Copy link
Owner

FeLungs commented Oct 31, 2022

Change Log Item Ids

s9, s10

1.0.3 text -> 9274.1.1 Update

The LRS SHOULD* NOT enforce character limits relating to response patterns.
->
Removed as a requirement

The LRS SHOULD* NOT limit the length of the correctResponsesPattern array for any interactionType
->
Removed as a requirement

Updates to xAPI Profile spec

  • ensure xAPI profile data doesn't impose character limits
  • add language to profile server section indicating implementations can impose character restrictions
    • should mirror similar language within xAPI base spec / standard
@FeLungs FeLungs added this to the Character Limits milestone Oct 31, 2022
@FeLungs
Copy link
Owner Author

FeLungs commented Jan 9, 2023

Where the specification does not include requirements relating to a particular facet of implementation, that detail can be considered to be outside of the scope of this specification. It is up to the implementer to determine a sensible approach. This specification tries to avoid vagueness and will usually give a rationale even if there no requirement in a given area.

^^^ from end of section 2.2 within Part 1

Given that clauses, should we mention character limits at all?

  • Could serve as a good example of what this clause means in practice
  • "For example, this specification does not place any limits on character limits on xAPI Profile fields but a Profile Server implementation MAY choose to include character limits for domain specific purposes like security or data storage"

@imartinezortiz
Copy link
Collaborator

Maybe copy the very same clause. Besides, domain specific can be confused with the specific domain of an application profile.

@FeLungs
Copy link
Owner Author

FeLungs commented Jan 13, 2023

Add reminder to Profile Server part that re-iterates "...specification does not include requirements relating to a particular facet of implementation..."

Reader may want to just go to section 3 and not have to read section 1 to know ^^^

@imartinezortiz
Copy link
Collaborator

This issue has been migrated to upstream repo adlnet#272

@imartinezortiz
Copy link
Collaborator

This issue has been migrated to upstream repo adlnet#271

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

No branches or pull requests

2 participants