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

Validate that score properties are numbers. #63

Open
garemoko opened this issue Mar 10, 2014 · 1 comment
Open

Validate that score properties are numbers. #63

garemoko opened this issue Mar 10, 2014 · 1 comment

Comments

@garemoko
Copy link
Contributor

It'd be nice if TinCanJS ensured that all the score properties were numbers, not strings.

See LearningLocker/learninglocker#113

@brianjmiller
Copy link
Member

This would be a fairly significant change, again because of the lack of "setter" model. To date we haven't validated types of any of the values being set, except for where they are versioned and then only through run time exceptions.

Though we can certainly leave it open as a to do, and 1 more nail in the need for proper accessors.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants