-
Notifications
You must be signed in to change notification settings - Fork 141
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
Added operations to the readme #59
base: master
Are you sure you want to change the base?
Conversation
in the Customization section.
I really like this idea, I think it belongs in the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The README already refers to https://jsonlogic.com/add_operation.html which has lots of info.
It might be beneficial to leave the add_operation
out of the readme in favor of the readme being minimal?
Suggested changes commented below -- perhaps the examples from https://jsonlogic.com/add_operation.html could be put here? That is, assuming adding examples in the readme is desired.
return age < 65; | ||
} | ||
|
||
function greatherThanNumber(age) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is misleading. greatherThanNumber
is actually greaterThanOrEqualToNumber
.
You can perform custom operations on your dataset using `add_operation` | ||
|
||
```js | ||
function lessThanNumber(age) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The name lessThanNumber
suggests to me that the operation would let you pick which number it is less than. Really this is lessThan65
.
One additional note: I did test the examples to make sure they work, and they do in fact work. |
No description provided.