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

[Bug 1074954] Add api to mark question as helpful. #2248

Closed
wants to merge 2 commits into from

Conversation

mythmon
Copy link
Contributor

@mythmon mythmon commented Dec 4, 2014

r?

I really need to write docs...

if not question.editable:
raise GenericAPIException(403, 'Question not editable')
if question.has_voted(request):
raise GenericAPIException(409, "Cannot vote twice")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: you used double quotes here and single quotes above

@rlr
Copy link
Contributor

rlr commented Dec 4, 2014

r+

@mythmon
Copy link
Contributor Author

mythmon commented Dec 4, 2014

2d73eb4 [Bug 1074954] Add api to mark questoin as helpful.

@mythmon mythmon closed this Dec 4, 2014
@mythmon mythmon deleted the api-mark-helpful-1074954 branch December 4, 2014 17:33
@mythmon mythmon changed the title [Bug 1074954] Add api to mark questoin as helpful. [Bug 1074954] Add api to mark question as helpful. Dec 4, 2014
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

Successfully merging this pull request may close these issues.

None yet

2 participants