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

Definers console warning if property option not recognized #112

Open
raix opened this issue Aug 30, 2012 · 2 comments
Open

Definers console warning if property option not recognized #112

raix opened this issue Aug 30, 2012 · 2 comments

Comments

@raix
Copy link

raix commented Aug 30, 2012

Feature req.
console warning if a define option not found (lib/resourceful/definers.js)
eg.:
Creature.property('life', Number, { default: 10, min: 0, max: 20 });
should be:
Creature.property('life', Number, { default: 10, minimum: 0, maximum: 20 });

A warning i dev mode? could be nice saying: "'life' Number property option 'min: 0' not recognized"

@Marak
Copy link
Contributor

Marak commented Sep 2, 2012

+1 This is a simple fix that would help a lot. Will add.

In the near future, we will be adding more tools and documentation to help with the creation of resource definitions.

Thanks for feedback.

@Marak
Copy link
Contributor

Marak commented Sep 2, 2012

Tangentially related issue #115 , since it seems we'd need a list of defined options to check against.

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