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.

Sign up for GitHub

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

Jump to bottom

API auth shouldn't be required for posting comments #918

Closed
rjmackay opened this issue Oct 30, 2012 · 2 comments
Closed

API auth shouldn't be required for posting comments #918

rjmackay opened this issue Oct 30, 2012 · 2 comments
Labels
Bug
Milestone
Ushahidi v2.7

Comments

@rjmackay
Copy link
Contributor

API authenticated is being required for posting comments, even though comments are free for all via web.
rjmackay added a commit that referenced this issue Oct 31, 2012
@rjmackay
Clean up formatting on comments api #918
36fd00b
@rjmackay
Copy link
Contributor Author

@eyedol @dalezak pushed a fix for this, can you test it out?

@rjmackay
Copy link
Contributor Author

Also included a bunch of other clean up and fixes.. see 7633164 for details

rjmackay added a commit that referenced this issue Oct 31, 2012
@rjmackay
Comments API fixes #918
7633164 
* Don't require authentication to submit comments
* Only returned approved comments to anonymous requests
* Valdate comment_description field exists
* Check comment author / email fields are no empty
* Fix _get_comment_list to return more than one comment
* Redo comments query to use query builder
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug
Projects
None yet
Milestone
Ushahidi v2.7
Development

No branches or pull requests
1 participant
@rjmackay