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

add 'spam' attribute to CheckContent object? #7

Open
markstos opened this issue Jul 18, 2013 · 0 comments
Open

add 'spam' attribute to CheckContent object? #7

markstos opened this issue Jul 18, 2013 · 0 comments

Comments

@markstos
Copy link

Michael,

What do you think about adding a 'spam' attribute to the CheckContent object, that would return 0, 1 or 2, to model the Mollom API?

It seems like for some cases it would be simpler to represent the result as one variable with 3 values, then 3 booleans. (Or, perhaps just name 'spam' and set it to 0, 1 or 2 directly model the Mollom API).

An example use-case might be passing the result back to JavaScript, where you can't pass the "$check" object.

(I first thought of naming this 'classification' and setting the values to 'spam', 'ham' or 'unsure'. Those names are clearer than the Mollom API, but perhaps it's best to consistent with it?)

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

1 participant