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

property name should differ from the class name in the examples #99

Closed
kidrane opened this issue Dec 31, 2013 · 2 comments
Closed

property name should differ from the class name in the examples #99

kidrane opened this issue Dec 31, 2013 · 2 comments

Comments

@kidrane
Copy link
Contributor

kidrane commented Dec 31, 2013

var Message = function (message) {
    this.message = message;
};

should be like the following, much less confusing for the beginers

var Message = function (text) {
    this.text = text;
};
@doug-martin doug-martin mentioned this issue May 18, 2014
doug-martin added a commit to doug-martin/nools that referenced this issue May 18, 2014
* Added new `===` and `!==` operators noolsjs#110
* Fix for issue noolsjs#109
* Updated Readme
    * Updated agenda groups examples for noolsjs#105
    * Changed class names not to match property names in readme noolsjs#99
doug-martin added a commit to doug-martin/nools that referenced this issue May 18, 2014
* Added new `===` and `!==` operators noolsjs#110
* Fix for issue noolsjs#109
* Updated Readme
    * Updated agenda groups examples for noolsjs#105
    * Changed class names not to match property names in readme noolsjs#99
@doug-martin
Copy link
Contributor

Thanks for the input I updated the readme with your suggestion

@mrgfisher
Copy link

I look forward to using your recent updates; I must say that Nooks is awesome and greatly enjoyed using it recently.

Regards,

Graham Fisher
Technical Consultant
Skype: mrgfisher
Phone: 077716 77726
Work: 0161 4080706

From: Doug Martinmailto:notifications@github.com
Sent: ?Sunday?, ?18? ?May? ?2014 ?23?:?02
To: C2FO/noolsmailto:nools@noreply.github.com

Closed #99#99.

Reply to this email directly or view it on GitHubhttps://github.com//issues/99#event-122284279.

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

3 participants